On Wed, Oct 18, 2023 at 02:40:41PM -0500, Bjorn Helgaas wrote:
> On Tue, Oct 17, 2023 at 10:42:49AM +0300, Ido Schimmel wrote:
> > Spectrum-{1,2,3,4} devices report that a D3hot->D0 transition causes a
> > reset (i.e., they advertise NoSoftRst-). However, this transition seems
> > to have no effect on the device: It continues to be operational and
> > network ports remain up. Advertising this support makes it seem as if a
> > PM reset is viable for these devices. Mark it as unavailable to skip it
> > when testing reset methods.
> >
> > Before:
> >
> > # cat /sys/bus/pci/devices/0000\:03\:00.0/reset_method
> > pm bus
> >
> > After:
> >
> > # cat /sys/bus/pci/devices/0000\:03\:00.0/reset_method
> > bus
> >
> > Signed-off-by: Ido Schimmel <idosch@xxxxxxxxxx>
>
> Acked-by: Bjorn Helgaas <bhelgaas@xxxxxxxxxx>
>
> Hopefully since these are NVIDIA parts and you work at NVIDIA, this is
> stronger than "this transition *seems* to have no effect" :)
Yes. Reworded to "this transition does not have any effect on the
device" and kept your tag.
FYI, new devices will not advertise support for PM reset so I don't
expect this list to grow.
