Re: API break, sysfs "capability" file
From: Christoph Hellwig
Date: Wed Apr 17 2024 - 11:14:12 EST
On Tue, Apr 16, 2024 at 08:44:55AM -0600, Keith Busch wrote:
> The patch that introduced this was submitted not because the API was
> stable; it was committed to encourage developers to update it as it
> changed because it is *not* stable. That's not the kind of interface you
> want exported for users to rely on, but no one should have to search
> commit logs to understand why the doc exists, so I think exporting it
> was just a mistake on the kernel side. To say this doc is "good"
> misunderstands what it was trying to accomplish, and what it ultimately
> created instead: technical debt.
Yes. It might be a problem with the documentation generation mess,
but something that is generated from a random code comment really
can't be an API document.
Anyway, instead of bickering about this, what does systemd actually
want to known? Because all these flags we talked about did slightly
different things all vaguely related to partition scanning.
We also have another GD_SUPPRESS_PART_SCAN bit that is used to
temporarily suppress partition scanning (and ublk now also abuses
it in really whacky way, sigh). I'm not really sure why userspace
would even care about any of this, but I'm open to come up with
something sane if we can actually define the semantics.