Re: [PATCH 14/15] nvme: enable FDP support

From: Keith Busch
Date: Tue Nov 19 2024 - 17:49:33 EST


On Tue, Nov 19, 2024 at 07:24:27PM +0100, Christoph Hellwig wrote:
> On Tue, Nov 19, 2024 at 11:17:36AM -0700, Keith Busch wrote:
> > > + if (le32_to_cpu(configs[result.fdpcidx].nrg) > 1) {
> > > + dev_warn(ns->ctrl->device, "FDP NRG > 1 not supported\n");
> >
> > Why not support multiple reclaim groups?
>
> Can you come up with a sane API for that?

Haven't really thought about it. If it's there, it's probably useful for
RU's that are not "Persistently Isolated". But let's not worry about it
now, we can just say you don't get to use write streams for these.

> And can you find devices in
> the wild that actually support it?

I haven't come across any, no.

But more about the return codes for pretty much all the errors here.
They'll prevent the namespace from being visible, but I think you just
want to set the limits to disable write streams instead. Otherwise it'd
be a regression since namespaces configured this way are currently
usable.