From mboxrd@z Thu Jan 1 00:00:00 1970 From: Benjamin Herrenschmidt Subject: Re: [PATCH v4 2/5] powerpc/eeh: Add warning message in eeh_dev_open() Date: Fri, 08 Aug 2014 13:50:12 +1000 Message-ID: <1407469812.4508.1.camel@pasglop> References: <1407379640-13065-1-git-send-email-gwshan@linux.vnet.ibm.com> <1407379640-13065-3-git-send-email-gwshan@linux.vnet.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Cc: linuxppc-dev@lists.ozlabs.org, kvm@vger.kernel.org, aik@ozlabs.ru, alex.williamson@redhat.com To: Gavin Shan Return-path: Received: from gate.crashing.org ([63.228.1.57]:34616 "EHLO gate.crashing.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755262AbaHHDze (ORCPT ); Thu, 7 Aug 2014 23:55:34 -0400 In-Reply-To: <1407379640-13065-3-git-send-email-gwshan@linux.vnet.ibm.com> Sender: kvm-owner@vger.kernel.org List-ID: On Thu, 2014-08-07 at 12:47 +1000, Gavin Shan wrote: > The patch adds one warning message in eeh_dev_open() in case the > PCI device can't be marked as passed through. > > Suggested-by: Alexey Kardashevskiy > Signed-off-by: Gavin Shan > --- Acked-by: Benjamin Herrenschmidt Alex, are you taking this or should I ? > arch/powerpc/kernel/eeh.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/arch/powerpc/kernel/eeh.c b/arch/powerpc/kernel/eeh.c > index 59a64f8..5d73a49 100644 > --- a/arch/powerpc/kernel/eeh.c > +++ b/arch/powerpc/kernel/eeh.c > @@ -1162,8 +1162,11 @@ int eeh_dev_open(struct pci_dev *pdev) > > /* No EEH device or PE ? */ > edev = pci_dev_to_eeh_dev(pdev); > - if (!edev || !edev->pe) > + if (!edev || !edev->pe) { > + pr_warn_once("%s: PCI device %s not supported\n", > + __func__, pci_name(pdev)); > goto out; > + } > > /* Increase PE's pass through count */ > atomic_inc(&edev->pe->pass_dev_cnt);