From: John Garry <john.g.garry@oracle.com>
To: Christoph Hellwig <hch@infradead.org>
Cc: Chaitanya Kulkarni <chaitanyak@nvidia.com>,
"alan.adamson@oracle.com" <alan.adamson@oracle.com>,
"linux-nvme@lists.infradead.org" <linux-nvme@lists.infradead.org>
Subject: Re: Issue with AWUPF when using multiple controllers in a subsystem
Date: Thu, 10 Apr 2025 11:11:20 +0100 [thread overview]
Message-ID: <ca30c1ea-409e-4656-835f-31d29354c53a@oracle.com> (raw)
In-Reply-To: <Z_eNLCn_e1vFOu9m@infradead.org>
On 10/04/2025 10:19, Christoph Hellwig wrote:
> On Thu, Apr 10, 2025 at 10:09:53AM +0100, John Garry wrote:
>> This, combined with no dedicated NVMe command to issue a write atomically
>> (which could error for out-of-limits size/crossing boundary), is pretty
>> concerning.
>
> Agreed. Given that Oracle is actually a major user of NVMe, can you
> bring that to the working group's attention? That works much better
> than a random Linux maintainer.
Sure, I'll ask someone.
>
>> As an aside, I found the scope of the boundary definition to be quite vague
>> as well.
>
> It used to be really horrible, but got a major rework for multiple
> atomicy in NVMe 2.2 (or was it 2.1?).
I am not really talking about multiple atomicy here, but something more
basic...
> Which version did you check?
I am specifically referring to NSFEAT.NSABP, which is now described in
NVM Command Set Specification 1.1.
I just found it odd that NSABP is related to boundary, but describes
whether NAWUN et al fields are used or not (but not whether NABSPF is
valid). Having said that, I'd think that NABSPF is always valid from
2.1.4.4 Atomic Boundaries "The namespace supports Atomic Boundaries if
NABSN or NABSPF are set to non-zero values".
But from "Figure 4: Atomicity Parameters for Single Atomicity Mode",
"Namespace Atomic Boundary Parameters" cell, it tells to refer to
"Identify Namespace data structure" which describes NSFEAT.NSABP;
however it does also describe NABSPF.
I am getting the feeling that the kernel driver should not check
NSFEAT.NSABP on whether NABSPF is valid (which it does today).
next prev parent reply other threads:[~2025-04-10 11:18 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-04-04 22:42 Issue with AWUPF when using multiple controllers in a subsystem alan.adamson
2025-04-08 21:08 ` Chaitanya Kulkarni
2025-04-09 9:01 ` Christoph Hellwig
2025-04-10 9:09 ` John Garry
2025-04-10 9:19 ` Christoph Hellwig
2025-04-10 10:11 ` John Garry [this message]
2025-04-08 21:08 ` Chaitanya Kulkarni
2025-04-09 9:05 ` Christoph Hellwig
2025-04-10 1:33 ` Chaitanya Kulkarni
2025-04-10 8:32 ` Christoph Hellwig
2025-04-10 9:07 ` Keith Busch
2025-04-10 9:17 ` Christoph Hellwig
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ca30c1ea-409e-4656-835f-31d29354c53a@oracle.com \
--to=john.g.garry@oracle.com \
--cc=alan.adamson@oracle.com \
--cc=chaitanyak@nvidia.com \
--cc=hch@infradead.org \
--cc=linux-nvme@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox