Linux-NVME Archive on lore.kernel.org
 help / color / mirror / Atom feed
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).


  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