Re: [PATCH] pciehp: Disable/enable link during slot power off/on

From: Jesse Barnes
Date: Mon Dec 05 2011 - 14:20:55 EST


On Fri, 02 Dec 2011 15:10:08 -0800
Yinghai Lu <yinghai.lu@xxxxxxxxxx> wrote:

>
> One system have repeater in system board to support gen2 hotplug.
>
> Found when EM is removed from some slots, /var/log/message will be full of
> "card present/not present" warning.
>
> It turns out root complex still try to train the link to repeater because
> repeater is not reset.
>
> This patch will disable link to make repeater could reset properly.
> Also could kill AER during EM removal.
>
> Recently when testing hotplug on one system under development, found if boot
> the system without EM, and later hotplug does not work with Linux.
> But other OS is ok.
> The root cause is that bios left link disabled when slot is empty,
> and other OS is playing link disable bit in link ctrl during power on/off.
>
> So We could do the same thing to disable/enable link during power off/on.
>
> Signed-off-by: Yinghai Lu <yinghai.lu@xxxxxxxxxx>
>

Kenji-san, any comment or chance to test?

Thanks,
--
Jesse Barnes, Intel Open Source Technology Center

Attachment: signature.asc
Description: PGP signature