Re: [PATCH] xfs: update BDI {io,ra}_pages values based on the RT device limits
"Darrick J. Wong" <[email protected]>
| Newsgroups | org.kernel.vger.linux-xfs,org.kernel.vger.linux-fsdevel |
|---|---|
| Message-ID | <20260721151852.GA7358@frogsfrogsfrogs> |
On Tue, Jul 21, 2026 at 11:23:29AM +0200, Carlos Maiolino wrote: > On Tue, Jul 21, 2026 at 06:30:11AM +0200, Christoph Hellwig wrote: > > On Tue, Jul 21, 2026 at 08:07:40AM +0900, Damien Le Moal wrote: > > > > + * Update the BDI values to the max of the data and RT device to cover our > > > > > > s/data/main ? > > > I am still confused about the proper name for the device holding metadata :) > > > > > > Other than this, this looks OK to me. > > > > Historically the main device is called the data device, which is > > also reflected in all kinds of field naming. But yes, it is confusing > > especially for zoned xfs setups. Maybe I should do a little global > > naming sweep.. > > > > > > +1, please keep /data/ here, this makes much more sense in xfs lingo... > Perhaps we should start naming them to something like ndata (for normal > data) and zdata for zoned data, but that's talk for another thread. I've been trying to be consistent at calling them the 'datadev' and 'rtdev' (no spaces) so it's more obvious that I'm using a jargon. But 'rt' makes no sense these days and I never did come up with a good retcon for it though. "RelaTive"? Because the rtdev is "related" to the datadev by metadata? :D --D