On Tue, Aug 15, 2017 at 12:04:50AM +0800, Qiuxu Zhuo wrote: > Sun, Yunying reported the following failure on Denverton micro-server: > > EDAC DEBUG: pnd2_init: > EDAC DEBUG: pnd2_probe: > EDAC DEBUG: dnv_rd_reg: Read b_cr_tolud_pci=00000000_80000000 > EDAC DEBUG: dnv_rd_reg: Read b_cr_touud_lo_pci=00000000_80000000 > EDAC DEBUG: dnv_rd_reg: Read b_cr_touud_hi_pci=00000000_00000004 > EDAC DEBUG: dnv_rd_reg: Read b_cr_asym_mem_region0_mchbar=00000000_00000000 > EDAC DEBUG: dnv_rd_reg: Read b_cr_asym_mem_region1_mchbar=00000000_00000000 > EDAC DEBUG: dnv_rd_reg: Read b_cr_mot_out_base_mchbar=00000000_00000000 > EDAC DEBUG: dnv_rd_reg: Read b_cr_mot_out_mask_mchbar=00000000_00000000 > EDAC pnd2: Failed to register device with error -19. > > On Denverton micro-server, the presence of the P2SB bridge PCI device is > enabled or disabled by the item 'RelaxSecConf' in BIOS setup menu. When > 'RelaxSecConf' is enabled, the P2SB PCI device is present and the pnd2_edac > EDAC driver also uses it to get BAR. Hiding the P2SB PCI device caused the > pnd2_edac EDAC driver failed to get BAR then reported the above failure. > > Therefor, store the presence state of P2SB PCI device before unhiding it > for reading BAR and restore the presence state after reading BAR. > > Signed-off-by: Qiuxu Zhuo <qiuxu.zhuo@xxxxxxxxx> > Reported-and-tested-by: Yunying Sun <yunying.sun@xxxxxxxxx> Applied to for-next, thanks! I don't want to apply it to for-current or stable without Jean's ack. So, let's get more test coverage first. It can still be backported later. Hope this is fine with you.
Attachment:
signature.asc
Description: PGP signature