On Mon, Jan 16, 2023 at 9:24 AM Anatoli Antonovitch <anatoli.antonovitch@xxxxxxx> wrote: > > Thanks Bjorn, > > The extensive testing has been done for hot-plug, hot-unplug the device and ACPI S3 suspend/resume cycles on system with AER enabled. > > The patch has bee resent. Thanks. Just FYI, the mailing lists generally reject HTML email, so your response doesn't appear in the archives: https://lore.kernel.org/linux-pci/20230105184355.9829-1-a.antonovitch@xxxxxxxxx/ See http://vger.kernel.org/majordomo-info.html, https://en.wikipedia.org/wiki/Posting_style#Interleaved_style, https://people.kernel.org/tglx/notes-about-netiquette . Not a big deal for this response, but it is important for more substantive conversations. > On 2023-01-05 14:18, Bjorn Helgaas wrote: > > On Thu, Jan 05, 2023 at 01:43:55PM -0500, Anatoli Antonovitch wrote: > > From: Anatoli Antonovitch <anatoli.antonovitch@xxxxxxx> > > It is to avoid any potential issues when S3 resume but at the same time we want to hot-unplug. > Need to do some more testing to see if it is still necessary. > > I'm not sure how to interpret this. > > I guess I'll wait for you to do the testing and repost this if it does > turn out to be necessary? > > I don't want to merge a patch with a commit log that says "need to do > more testing." > > Bjorn
