RE: [PATCH 1/1] nvme-fabrics: Generate uevent on namespace AENs

From: "Rao, Vinay" <Vinay.Rao@dell.com>
To: Hannes Reinecke <hare@suse.de>, Keith Busch <kbusch@kernel.org>
Cc: Chaitanya Kulkarni <Chaitanya.Kulkarni@wdc.com>,
	"Belanger, Martin" <Martin.Belanger@dell.com>,
	"Hayes, Stuart" <Stuart.Hayes@dell.com>,
	"Tarikere, Madhu" <Madhu.Tarikere@dell.com>,
	Christoph Hellwig <hch@lst.de>,
	Martin Belanger <nitram_67@hotmail.com>,
	"linux-nvme@lists.infradead.org" <linux-nvme@lists.infradead.org>,
	"axboe@fb.com" <axboe@fb.com>,
	"sagi@grimberg.me" <sagi@grimberg.me>
Subject: RE: [PATCH 1/1] nvme-fabrics: Generate uevent on namespace AENs
Date: Wed, 30 Jun 2021 14:00:33 +0000	[thread overview]
Message-ID: <BYAPR19MB2935A9169D186DD3C05979668F019@BYAPR19MB2935.namprd19.prod.outlook.com> (raw)
In-Reply-To: <097e56b6-2b13-e00c-ef84-509fa0e887ca@suse.de>


-----Original Message-----
From: Hannes Reinecke <hare@suse.de> 
Sent: Thursday, June 24, 2021 4:19 PM
To: Keith Busch; Rao, Vinay
Cc: Chaitanya Kulkarni; Belanger, Martin; Hayes, Stuart; Tarikere, Madhu; Christoph Hellwig; Martin Belanger; linux-nvme@lists.infradead.org; axboe@fb.com; sagi@grimberg.me
Subject: Re: [PATCH 1/1] nvme-fabrics: Generate uevent on namespace AENs


[EXTERNAL EMAIL] 

On 4/30/21 6:21 PM, Keith Busch wrote:
> On Fri, Apr 30, 2021 at 05:14:55AM +0000, Rao, Vinay wrote:
>> + Madhu
>>
>> Hi Chaitanya,
>>
>> Some of the AEN's like ANA state change and Name Space attribute change events are of interest outside the NVME driver. 
>>
>> It would be good to consider these events to be propagated up to user 
>> space. I am not yet convinced on how propagating this above to user space is problematic.
> 
> The namespace attribute change event pairs with the Changed Namespace 
> List log, and reading that log will change the result for a subsequent 
> reader. User space racing with the kernel on a log access when there 
> are read side effects creates non-deterministic behavior, and that is 
> problematic.
> 
It would, if we were relying on the Changed Namespace List log.
But in the end we're just reading it to clear the AEN flag; the contents are completely immaterial to us as we're doing a full namespace scan afterwards anyway.
So I don't think that that'll be a problem for us.

What will be problematic, though, is if we were to allow userspace to register an AEN; as soon as the controller allows more than one pending AERs there nothing preventing it from doing so.
If he does our internal states will be messed up as we're not receiving all events. I'd rather error out any attempt here, but then we can't do a full passthrough, and I guess things like SPDK might become unhappy.
> Wanted to check if there is a timeline by when we can expect the AEN's to be propagated to user space. 


Cheers,

Hannes
-- 
Dr. Hannes Reinecke		           Kernel Storage Architect
hare@suse.de			                  +49 911 74053 688
SUSE Software Solutions Germany GmbH, Maxfeldstr. 5, 90409 Nürnberg HRB 36809 (AG Nürnberg), GF: Felix Imendörffer
_______________________________________________
Linux-nvme mailing list
Linux-nvme@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-nvme