Re: [PATCH 5/7] xfs: replace min & max with clamp() in xfs_max_open_zones()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Subject: Re: [PATCH 5/7] xfs: replace min & max with clamp() in xfs_max_open_zones()
From: Christoph Hellwig <hch@xxxxxx>
Date: Thu, 17 Jul 2025 07:16:19 +0200
Cc: "Darrick J. Wong" <djwong@xxxxxxxxxx>, Christoph Hellwig <hch@xxxxxx>, Carlos Maiolino <cem@xxxxxxxxxx>, Hans Holmberg <hans.holmberg@xxxxxxx>, linux-xfs@xxxxxxxxxxxxxxx, George Hu <integral@xxxxxxxxxxxxx>, John Garry <john.g.garry@xxxxxxxxxx>
In-reply-to: <3B511F05-E1FB-4396-B91F-769678B8E776@gmail.com>
References: <20250716125413.2148420-1-hch@lst.de> <20250716125413.2148420-6-hch@lst.de> <20250716160206.GL2672049@frogsfrogsfrogs> <3B511F05-E1FB-4396-B91F-769678B8E776@gmail.com>
User-agent: Mutt/1.5.17 (2007-11-01)

On Thu, Jul 17, 2025 at 12:52:14AM +0800, Alan Huang wrote:
> > Does clamp() handle the case where @max < @min properly?
> 
> No, it only has BUILD_BUG_ON_MSG(statically_true(ulo > uhi), “xxx")

Well, I guess we'll just need to drop this patch then, right?

Follow-Ups:
- Re: [PATCH 5/7] xfs: replace min & max with clamp() in xfs_max_open_zones()
  - From: Alan Huang

References:
- misc zoned allocators fixes and cleanups
  - From: Christoph Hellwig
- [PATCH 5/7] xfs: replace min & max with clamp() in xfs_max_open_zones()
  - From: Christoph Hellwig
- Re: [PATCH 5/7] xfs: replace min & max with clamp() in xfs_max_open_zones()
  - From: Darrick J. Wong
- Re: [PATCH 5/7] xfs: replace min & max with clamp() in xfs_max_open_zones()
  - From: Alan Huang

Prev by Date: Re: [PATCH] xfs: improve the xg_active_ref check in xfs_group_free
Next by Date: Re: [PATCH 5/7] xfs: replace min & max with clamp() in xfs_max_open_zones()
Previous by thread: Re: [PATCH 5/7] xfs: replace min & max with clamp() in xfs_max_open_zones()
Next by thread: Re: [PATCH 5/7] xfs: replace min & max with clamp() in xfs_max_open_zones()
Index(es):
- Date
- Thread

[Index of Archives] [XFS Filesystem Development (older mail)] [Linux Filesystem Development] [Linux Audio Users] [Yosemite Trails] [Linux Kernel] [Linux RAID] [Linux SCSI]