----- Original Message -----
> From: "Krishna Kumar" <[email protected]>
> To: "Bjorn Helgaas" <[email protected]>, "Timothy Pearson" 
> <[email protected]>
> Cc: "linuxppc-dev" <[email protected]>, "linux-kernel" 
> <[email protected]>, "linux-pci"
> <[email protected]>, "Madhavan Srinivasan" <[email protected]>, 
> "Michael Ellerman" <[email protected]>,
> "christophe leroy" <[email protected]>, "Naveen N Rao" 
> <[email protected]>, "Bjorn Helgaas"
> <[email protected]>, "Shawn Anastasio" <[email protected]>, 
> "Lukas Wunner" <[email protected]>
> Sent: Monday, July 7, 2025 3:01:00 AM
> Subject: Re: [PATCH v2 6/6] pci/hotplug/pnv_php: Enable third attention 
> indicator

> Thanks all for the review and Thanks a bunch to Timothy for fixing the PE 
> Freeze
> issue. The hotplug issues are like you fix N issue and N+1 th issue will come
> with new HW.
> 
> We had a meeting of around 1.5 -2.0 hr with demo, code review and log review 
> and
> we decided to let these fixes go ahead. I am consolidating what we discussed -
> 
> 
> 1. Let these fixes go ahead as it solves wider and much needed customer issue
> and its urgently needed for time being. We have verified in live demo that
> nothing is broken from legacy flow and
> 
> PE/PHB freeze, race condition, hung, oops etc has been solved correctly.
> Basically it fixes below issues -
> 
> root-port(phb) -> switch(having4 port)--> 2 nvme devices
> 
> 1st case - only removal of EP-nvme device (surprise hotplug disables msi at 
> root
> port), soft removal may work
> 2nd caseĀ  - If we remove switch itself (surprise hotplug disable msi at root
> port) .

Just a quick follow up to see if anything else is needed from my end before 
this merge can occur?

Thanks!

Reply via email to