Re: [Patch v1 0/7] mpt3sas: Hot-Plug Surprise removal support on IOC.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



[cc += linux-pci, benh]

On Fri, Aug 31, 2018 at 7:37 AM Suganath Prabu S <suganath-prabu.subramani@xxxxxxxxxxxx> wrote:
> Posting below set of patches to support PCIe Hot Plug surprise removal,
> and few defect fixes.

Please cross-post to linux-pci in the future.


Regarding [PATCH 1/7] mpt3sas: Introduce mpt3sas_base_pci_device_is_unplugged:
https://www.spinics.net/lists/linux-scsi/msg122962.html

* mpt3sas_base_pci_device_is_unplugged() is a duplication of the existing
  pci_device_is_present().

* Just reading the vendor ID may not be sufficient to detect unplug,
  it may also read as "all ones" if the link is down due to error
  recovery by DPC.


Regarding [PATCH 2/7] mpt3sas: Add HBA hot plug watchdog thread:
https://www.spinics.net/lists/linux-scsi/msg122963.html

* I don't see why you need to poll for the device's removal from a
  watchdog thread.  pciehp will invoke your driver's ->remove hook
  once the device is gone.

* A recent discussion initiated by Benjamin Herrenschmidt came to the
  conclusion that device removal should be treated as a type of
  error state (either pci_channel_io_perm_failure or another, newly
  introduced state).  It will then be possible to detect the device's
  inaccessibility with pci_channel_offline().  Please help work towards
  such a future solution in the PCI core instead of solutions localized
  to a single device driver.  Sorry, the discussion was lengthy, it is
  available here:
  https://www.spinics.net/lists/linux-pci/msg75425.html

Thanks,

Lukas



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]

  Powered by Linux