Re: [PATCH V5 3/3] PCI: Mask and unmask hotplug interrupts during reset

From: Sinan Kaya
Date: Tue Jul 03 2018 - 09:31:33 EST


On 7/3/2018 9:25 AM, Sinan Kaya wrote:
>> what if we only have conditional enumeration ? (leaving removing devices followed by SBR as is) ?
>>
Sorry, I think I didn't quite get your question.

Are you asking about the enumeration following link up while keeping the
current code as is?

Issue is observing hotplug link down event in the middle of AER recovery as in
my previous reply.

If we mask hotplug interrupts before secondary bus reset via my patch,
then hotplug driver will not observe both link up and link down interrupts.

If we don't mask hotplug interrupts, we have a race condition.

I hope I answered your question.

--
Sinan Kaya
Qualcomm Datacenter Technologies, Inc. as an affiliate of Qualcomm Technologies, Inc.
Qualcomm Technologies, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project.