From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: From: David Laight To: 'Christoph Hellwig' , Oza Pawandeep CC: Bjorn Helgaas , Philippe Ombredanne , Thomas Gleixner , "Greg Kroah-Hartman" , Kate Stewart , "linux-pci@vger.kernel.org" , "linux-kernel@vger.kernel.org" , Dongdong Liu , "Keith Busch" , Wei Zhang , Sinan Kaya , Timur Tabi Subject: RE: [PATCH v10 2/7] PCI/AER: factor out error reporting from AER Date: Thu, 22 Feb 2018 18:42:32 +0000 Message-ID: <0cd107dfe3ec4dda89457b112e8f79e5@AcuMS.aculab.com> References: <1519315332-26852-1-git-send-email-poza@codeaurora.org> <1519315332-26852-3-git-send-email-poza@codeaurora.org> <20180222183448.GB6267@infradead.org> In-Reply-To: <20180222183448.GB6267@infradead.org> Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: From: Christoph Hellwig > > - pci_uevent_ers(dev, PCI_ERS_RESULT_DISCONNECT); > > - /* TODO: Should kernel panic here? */ > > - pci_info(dev, "AER: Device recovery failed\n"); > > -} > > +EXPORT_SYMBOL(pci_find_aer_service); > > EXPORT_SYMBPOL_GPL please. Is this a function that a 'normal' PCIe device driver would need to call? If so it is a PITA requiring all drivers that want AER to be GPL. David