Re: Regarding usb dwc3 core shutdown callback
From: Sandeep Maheswaram
Date: Thu Aug 26 2021 - 05:41:57 EST
On 8/12/2021 2:37 PM, Felipe Balbi wrote:
(why isn't this email plain/text? Content Type was set to multipart
alternative, please configure your email client correctly :-)
While at that, also make sure to break lines at 80-columns)
Sandeep Maheswaram <sanm@xxxxxxxxxxxxxx> writes:
Right, as should be expected when we find regressions
Earlier I have posted the patch for usb dwc3 core shutdown callback
and it was reverted due to issues.
If that's the case, then sure. Please validate the condition, though,
As we already have shutdown callback in xhci plat driver where we halt
the controller, so there will be no transactions with usb devices.
So I think dwc3 core shutdown may not be required at least when we are
using host mode. Let me know your opinion about this.
and kindly report back on your findings
I have enabled couple of logs in shutdown path and see no URBs enqueued
after xhci shut down.
Hope this is enough for validation . Please suggest if anything more I
[ 312.862737] systemd-journald: Received SIGTERM from PID 1 (init).
[ 312.866332] xhci-hcd xhci-hcd.0.auto: xhci_urb_enqueue
[ 312.875205] xhci-hcd xhci-hcd.0.auto: xhci_urb_enqueue
[ 312.951045] Unsafe core_pattern used with fs.suid_dumpable=2.
[ 312.951045] Pipe handler or fully qualified core dump path required.
[ 312.951045] Set kernel.core_pattern before fs.suid_dumpable.
[ 312.981364] ip_local_port_range: prefer different parity for
[ 313.086729] Bluetooth: qca_download_firmware() hci0: QCA Downloading
[ 313.194059] init: console-ttyMSM0 main process (516) killed by TERM
[ 313.317109] kvm: exiting hardware virtualization
[ 313.338988] platform video-firmware.0: Removing from iommu group 14
[ 313.345975] platform wifi-firmware.0: Removing from iommu group 12
[ 313.372244] xhci-hcd xhci-hcd.0.auto: -->xhci_shutdown
[ 313.377591] xhci-hcd xhci-hcd.0.auto: // Halt the HC
[ 313.382811] xhci-hcd xhci-hcd.0.auto: xhci_shutdown completed -
status = 1
[ 313.398238] Bluetooth: qca_uart_setup() hci0: QCA setup on UART is
[ 313.405583] debugfs: Directory 'ibs' with parent 'hci0' already present!
[ 313.439006] Bluetooth: qca_hw_error() hci0: mem_dump_status: 0
[ 313.445073] Bluetooth: qca_send_crashbuffer() hci0: crash the soc to
collect controller dump
[ 313.595791] arm-smmu 15000000.iommu: disabling translation
[ 313.601831] arm-smmu 5040000.iommu: disabling translation
[ 313.608559] reboot: Restarting system