public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Wu, Hao A" <hao.a.wu@intel.com>
To: "Gao, Liming" <gaoliming@byosoft.com.cn>,
	"devel@edk2.groups.io" <devel@edk2.groups.io>,
	"sulikun@loongson.cn" <sulikun@loongson.cn>,
	"Demeter, Miki" <miki.demeter@intel.com>
Subject: Re: [edk2-devel] empty USB DVD disk hang Xhci
Date: Tue, 25 Jul 2023 02:18:42 +0000	[thread overview]
Message-ID: <DM6PR11MB4025C6F67540A1868960B5DACA03A@DM6PR11MB4025.namprd11.prod.outlook.com> (raw)
In-Reply-To: <01ac01d9be99$86abcca0$940365e0$@byosoft.com.cn>


[-- Attachment #1.1: Type: text/plain, Size: 3936 bytes --]

Sorry, I do not have much resource to review this in detail. But the patch generally looks good to me:
Acked-by: Hao A Wu hao.a.wu@intel.com<mailto:hao.a.wu@intel.com>

However, I hope the below question can be answered (or add related information to the commit log message) before mering:
Likun, could you help to elaborate a bit more on why the below statement will return NULL pointer for “USB DVD with empty disk”?
EPRing        = (TRANSFER_RING *)(UINTN)Xhc->UsbDevContext[SlotId].EndpointTransferRing[Dci-1];


Best Regards,
Hao Wu

From: gaoliming <gaoliming@byosoft.com.cn>
Sent: Tuesday, July 25, 2023 9:44 AM
To: devel@edk2.groups.io; sulikun@loongson.cn; Demeter, Miki <miki.demeter@intel.com>
Cc: Wu, Hao A <hao.a.wu@intel.com>
Subject: 回复: [edk2-devel] empty USB DVD disk hang Xhci

Likun:
 This change is good to me.  Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn<mailto:gaoliming@byosoft.com.cn>>

Hao:
 Have you time to review this fix?

Thanks
Liming
发件人: devel@edk2.groups.io<mailto:devel@edk2.groups.io> <devel@edk2.groups.io<mailto:devel@edk2.groups.io>> 代表 苏丽坤
发送时间: 2023年7月18日 9:49
收件人: gaoliming@byosoft.com.cn<mailto:gaoliming@byosoft.com.cn>; miki.demeter@intel.com<mailto:miki.demeter@intel.com>; devel@edk2.groups.io<mailto:devel@edk2.groups.io>
主题: [edk2-devel] empty USB DVD disk hang Xhci


Hi

    We find a bug in xhci, USB DVD boot with emtpy disk will hang Xhci while Ehci can pass.

    The hang log is shown as : MdeModulePkg/Bus/Pci/XhciDxe/XhciSched.c(1847): TrsRing != ((void *) 0)

    We suggest a way to solve this bug is shown as: https://github.com/suling-123/edk2 the latest commit,   MdeModulePkg: Solve boot hang xhci driver when use USB DVD with empty disk, 950bc8781d81b96b0c7944e7ac947382b1bc0c06

[cid:image001.png@01D9BEE1.61EECFA0]


本邮件及其附件含有龙芯中科的商业秘密信息,仅限于发送给上面地址中列出的个人或群组。禁止任何其他人以任何形式使用(包括但不限于全部或部分地泄露、复制或散发)本邮件及其附件中的信息。如果您错收本邮件,请您立即电话或邮件通知发件人并删除本邮件。
This email and its attachments contain confidential information from Loongson Technology , which is intended only for the person or entity whose address is listed above. Any use of the information contained herein in any way (including, but not limited to, total or partial disclosure, reproduction or dissemination) by persons other than the intended recipient(s) is prohibited. If you receive this email in error, please notify the sender by phone or email immediately and delete it.


本邮件及其附件含有龙芯中科的商业秘密信息,仅限于发送给上面地址中列出的个人或群组。禁止任何其他人以任何形式使用(包括但不限于全部或部分地泄露、复制或散发)本邮件及其附件中的信息。如果您错收本邮件,请您立即电话或邮件通知发件人并删除本邮件。
This email and its attachments contain confidential information from Loongson Technology , which is intended only for the person or entity whose address is listed above. Any use of the information contained herein in any way (including, but not limited to, total or partial disclosure, reproduction or dissemination) by persons other than the intended recipient(s) is prohibited. If you receive this email in error, please notify the sender by phone or email immediately and delete it.



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#107215): https://edk2.groups.io/g/devel/message/107215
Mute This Topic: https://groups.io/mt/100343121/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



[-- Attachment #1.2: Type: text/html, Size: 11245 bytes --]

[-- Attachment #2: image001.png --]
[-- Type: image/png, Size: 63621 bytes --]

  reply	other threads:[~2023-07-25  2:18 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-07-18  1:48 [edk2-devel] empty USB DVD disk hang Xhci 苏丽坤
2023-07-25  1:44 ` 回复: " gaoliming via groups.io
2023-07-25  2:18   ` Wu, Hao A [this message]
2023-07-27  3:21     ` 苏丽坤
2023-07-27  7:23       ` Chao Li
2023-08-02  5:29         ` 回复: " gaoliming via groups.io
2023-08-03  2:44           ` Chao Li
2023-08-03 11:43             ` 苏丽坤

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-list 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=DM6PR11MB4025C6F67540A1868960B5DACA03A@DM6PR11MB4025.namprd11.prod.outlook.com \
    --to=devel@edk2.groups.io \
    /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