Re: [RFC PATCH v3] PCI/portdrv: Only disable Bus Master on kexec reboot and connected PCI devices

From: Huacai Chen
Date: Wed Sep 16 2020 - 02:20:06 EST


Hi, Bjorn,

On Tue, Sep 15, 2020 at 4:38 PM Bjorn Helgaas <helgaas@xxxxxxxxxx> wrote:
>
> On Tue, Sep 15, 2020 at 09:36:13AM +0800, Huacai Chen wrote:
> > Hi, Tiezhu,
> >
> > On Mon, Sep 14, 2020 at 7:25 PM Tiezhu Yang <yangtiezhu@xxxxxxxxxxx> wrote:
> > >
> > > On 09/14/2020 05:46 PM, Huacai Chen wrote:
> > > > Hi, Tiezhu,
> > > >
> > > > On Mon, Sep 14, 2020 at 5:30 PM Tiezhu Yang <yangtiezhu@xxxxxxxxxxx> wrote:
> > > >> On 09/14/2020 04:52 PM, Huacai Chen wrote:
> > > >>> Hi, Tiezhu,
> > > >>>
> > > >>> How do you test kexec? kexec -e or systemctl kexec? Or both?
> > > >> kexec -l vmlinux --append="root=/dev/sda2 console=ttyS0,115200"
> > > >> kexec -e
> > > > So you haven't tested "systemctl kexec"?
> > >
> > > Yes, the distro I used is Loongnix which has not kexec service now.
> > > Is there any problem when use systemctl kexec? If you have more details,
> > > please let me know.
> > If you use systemctl kexec, the first part of kexec is the same as a
> > normal reboot/poweroff. So, there is no reason that reboot/poweroff is
> > bad but kexec is good.
> > Then, Bjorn, what is the best solution now? It seems like my first
> > version is OK (commit messages should be improved, of course).
>
> Sorry, there are too many conversations going on for me to keep track
> of what first version you're referring to. Please include a specific
> lore.kernel.org URL.
My first version means this one:
https://patchwork.kernel.org/patch/11695929/

Thanks,
Huacai
>
> Bjorn