Re: [Qemu-devel] [edk2] NVMe question

[Laszlo Ersek]

On 06/02/15 09:19, Tian, Feng wrote:
> Thanks for the info.
> 
>  
> 
> I didn’t test it on NVMe-Qemu, I just run the change on a real
> production. I will do bigger scope test to see if it’s valuable to add
> it back.

This email thread is a perfect example why top-posting is a horrible
practice.

The NVMe device model was developed for QEMU by Intel's own Keith Busch
(added to the address list). Now Keith will have to read this email in
reverse. (A good portion of which consists of confidentiality notices,
posted to a public mailing list.) Good luck.

Keith, here's the thread, if it helps:
http://thread.gmane.org/gmane.comp.bios.tianocore.devel/15073

The issue seems to be that

https://github.com/tianocore/edk2/commit/d6c55989

removed the nonzero initialization of Cc.Iosqes (submission queue size?)
and Cc.Iocqes (completion queue size?) in function
NvmeEnableController(). And the removal of these field initializations
seems to cause the early sanity check in QEMU's nvme_start_ctrl() to
fail -- my guess at least.

The question is now if QEMU is right (according to the NVMe spec) to
require those fields, or if edk2 is right not to initialize them.

(On the side, let me point out that even if this edk2 change was
justified, it definitely should have been a separate patch, because it
has nothing to do with "Expose EFI_NVM_EXPRESS_PASS_THRU protocol".)

Let me break it down for the 1000th time:
- don't top post
- write focused patches

PLEASE! It's not 1980 any longer.

Thanks
Laszlo

> 
> Thanks
> 
> Feng
> 
>  
> 
> *From:* Anbazhagan, Baraneedharan [mailto:address@hidden
> *Sent:* Tuesday, June 02, 2015 06:22
> *To:* Tian, Feng; address@hidden
> *Subject:* RE: NVMe question
> 
>  
> 
> Thanks for the update. NVMe used to work in QEMU and started failing now
> with this change. May be NVMe emulation issue in QEMU.
> 
>  
> 
> -Baranee
> 
>  
> 
> CONFIDENTIALITY NOTICE: The information contained in this e-mail and any
> accompanying documents may contain information which is HP confidential
> or otherwise protected from disclosure. This transmission may also be
> protected by the attorney-client privilege, the attorney work-product
> privilege, or both. If you are not the intended recipient of this
> message, or if this message has been addressed to you in error, please
> immediately alert the sender by reply e-mail and then delete this
> message, including any attachments. Any dissemination, distribution or
> other use of the contents of this message by anyone other than the
> intended recipient is strictly prohibited.
> 
>  
> 
> *From:* Tian, Feng [mailto:address@hidden
> *Sent:* Sunday, May 31, 2015 9:54 PM
> *To:* Anbazhagan, Baraneedharan; address@hidden
> <mailto:address@hidden>
> *Cc:* Tian, Feng
> *Subject:* RE: NVMe question
> 
>  
> 
> Hi, Baraneedharan
> 
>  
> 
> Why I removed these two fields initialization is because NVMe spec 7.6.1
> doesn’t say it’s mandatory.
> 
>  
> 
> Do you see any real impact on this?
> 
>  
> 
> Thanks
> 
> Feng
> 
>  
> 
> *From:* Anbazhagan, Baraneedharan [mailto:address@hidden
> *Sent:* Saturday, May 30, 2015 06:21
> *To:* Tian, Feng; address@hidden
> <mailto:address@hidden>
> *Subject:* NVMe question
> 
>  
> 
> With recent changes to NvmExpressDxe module, NvmeEnableController
> doesn’t set I/O completion and submission queue size. Why default values
> are removed?  
> 
>  
> 
>  
> 
> CONFIDENTIALITY NOTICE: The information contained in this e-mail and any
> accompanying documents may contain information which is HP confidential
> or otherwise protected from disclosure. This transmission may also be
> protected by the attorney-client privilege, the attorney work-product
> privilege, or both. If you are not the intended recipient of this
> message, or if this message has been addressed to you in error, please
> immediately alert the sender by reply e-mail and then delete this
> message, including any attachments. Any dissemination, distribution or
> other use of the contents of this message by anyone other than the
> intended recipient is strictly prohibited.
> 
>  
> 
> 
> 
> ------------------------------------------------------------------------------
> 
> 
> 
> _______________________________________________
> edk2-devel mailing list
> address@hidden
> https://lists.sourceforge.net/lists/listinfo/edk2-devel
>