Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1328872

Re: [PATCH 1/5] ARM: add EXPORT_SYMBOL of hook_fault_code for PCI host modularization

From Arnd Bergmann <arnd@arndb.de>
Newsgroups linux.kernel
Subject Re: [PATCH 1/5] ARM: add EXPORT_SYMBOL of hook_fault_code for PCI host modularization
Date 2016-02-08 11:00 +0100
Message-ID <qZQLV-3tN-25@gated-at.bofh.it> (permalink)
References <qZHyW-5BD-5@gated-at.bofh.it> <qZHyW-5BD-9@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Sunday 07 February 2016 19:00:40 Paul Gortmaker wrote:
> In a discussion of a previous patch set[1], it was suggested that
> modularizing some of the PCI host support would be good to keep
> multi platform bzImage sizes smaller.
> 
> Two of the files that are candidates for conversion to tristate
> from bool are:
> 
> drivers/pci/host/pci-imx6.c
> drivers/pci/host/pci-keystone.c
> 
> However, doing the conversion reveals that they are going to fail
> at modpost time since hook_fault_code isn't currently exported.
> 
> Since we are now going to export it we also need to remove the
> __init tag, as the fcn needs to be present at insmod time.
> 
> [1] https://lkml.kernel.org/r/20160108203102.GH5354@localhost
> 

If we want to make hook_fault_code() visible, we at least also need
an unhook_fault_code() function that removes the fault handler
when unloading the module, and preferably does so in a safe way.

The usage of hook_fault_code() at least in the imx driver is also
problematic, as it just ignores all "imprecise external abort"
faults, whether they come from PCI or not.

	Arnd

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH 1/5] ARM: add EXPORT_SYMBOL of hook_fault_code for PCI host modularization Paul Gortmaker <paul.gortmaker@windriver.com> - 2016-02-08 01:10 +0100
  Re: [PATCH 1/5] ARM: add EXPORT_SYMBOL of hook_fault_code for PCI host modularization Arnd Bergmann <arnd@arndb.de> - 2016-02-08 11:00 +0100
    Re: [PATCH 1/5] ARM: add EXPORT_SYMBOL of hook_fault_code for PCI  host modularization Paul Gortmaker <paul.gortmaker@windriver.com> - 2016-02-08 17:50 +0100
    Re: [PATCH 1/5] ARM: add EXPORT_SYMBOL of hook_fault_code for PCI  host modularization Russell King - ARM Linux <linux@arm.linux.org.uk> - 2016-02-08 18:30 +0100
  Re: [PATCH 1/5] ARM: add EXPORT_SYMBOL of hook_fault_code for PCI  host modularization Russell King - ARM Linux <linux@arm.linux.org.uk> - 2016-02-08 18:40 +0100

csiph-web