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

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

 



On Mon, Jul 14, 2025 at 03:59:27AM -0700, Christoph Hellwig wrote:
> On Mon, Jul 14, 2025 at 08:58:59AM +0100, John Garry wrote:
> > > @@ -1133,9 +1133,7 @@ xfs_max_open_zones(
> > >   	/*
> > >   	 * Cap the max open limit to 1/4 of available space
> > >   	 */
> >
> > Maybe you can keep this comment, but it was pretty much describing the code
> > and not explaining the rationale.
> 
> Let me send an incremental patch to explain the rational as long as I
> can still remember it :)

Sounds good, I'll wait and merge both together.

> 
> 




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


  Powered by Linux