Re: Revert "tpci: unbind driver in test_assign_resources()"

On Tue, Aug 4, 2026 at 9:12 PM Andrea Cervesato
<[email protected]> wrote:
>
> Hi Tang,
>
> > On 30/7/26 4:53 pm, Cyril Hrubis wrote:
> > > Hi!
> > >>> We recently confirmed this commit does cause a flood of I/O errors on
> > >>> the root filesystem in our testing, leaving the machine unusable until
> > >>> reboot.
> > >>>
> > >>> Given that, we'd appreciate the community helping to decide soon
> > >>> whether the revert is warranted.
> > >>
> > >> Are you 100% sure that is caused by the patch you want to revert?
> > >> Because the patch looked ok to me when I reviewed. We need to have
> > >> more context and info around the issue which has been generated in
> > >> order to avoid any workaround for a system/kernel bug.
> > >
> > > AFAIK the testing module grabs first device available on the system and
> > > uses it for testing. Which means that if you are unlucky it's your
> > > network card, or disk controller and you force reload it on a live
> > > system.
> > >
> > > And honestly the PCI test should be either rewritten and cleaned up to
> > > do something more sensible or just removed. As it is it's too fragile.
> > >
> >
> > Thank you for your feedback. It is very helpful to us.
> >
> > --
> > Best Regards,
> > Yi
> >
>
> Are you going to work on a new patch? what we do with this one?

Hi Andrea,

Sorry, I’m not a PCI expert, so for now I’ve only reverted this commit
in our company testing. I’ll follow your new patchset.

Best regards,
Yi

>
> Regards,
> --
> Andrea Cervesato
> SUSE QE Automation Engineer Linux
> [email protected]

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp