Re: [RFC] use pci_find_ext_capability everywhere

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Great, thanks a lot for testing, Tomasz.  I'll put the fix into my linux-next 
branch.

Jesse

On Wednesday, October 8, 2008 10:08 am Tomasz Czernecki wrote:
> Jesse, Yanmin,
>
> I've confirmed - AER is loaded and working fine now w/ Jesse's patch.
>
> Tomasz
>
> On Tue, Oct 7, 2008 at 4:14 PM, Tomasz Czernecki <czernecki@xxxxxxxxx> 
wrote:
> > Thanks Jesse - recompiling now..  I'll have results by tomorrow
> > morning - have some things to attend to this afternoon..
> >
> > Thanks.
> > Tomasz
> >
> > On Tue, Oct 7, 2008 at 1:47 PM, Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx> 
wrote:
> >> On Tuesday, October 7, 2008 10:34 am Tomasz Czernecki wrote:
> >>> Hi Jesse,
> >>>
> >>> I applied the commands you specified, applied the
> >>> "pci-use-find-ext-cap-3.patch", and started compiling, but then
> >>> encountered an error.  The following is the last half-screen or so of
> >>> the output.  Should I have been using the 1115-001.bin patch instead?
> >>>
> >>>   AS      arch/x86/lib/putuser.o
> >>>   AS      arch/x86/lib/semaphore_32.o
> >>>   CC      arch/x86/lib/string_32.o
> >>>   CC      arch/x86/lib/strstr_32.o
> >>>   AS      arch/x86/lib/thunk_32.o
> >>>   CC      arch/x86/lib/usercopy_32.o
> >>>   AR      arch/x86/lib/lib.a
> >>>   LD      vmlinux.o
> >>>   MODPOST vmlinux.o
> >>> WARNING: modpost: Found 2 section mismatch(es).
> >>> To see full details build your kernel with:
> >>> 'make CONFIG_DEBUG_SECTION_MISMATCH=y'
> >>>   GEN     .version
> >>>   CHK     include/linux/compile.h
> >>>   UPD     include/linux/compile.h
> >>>   CC      init/version.o
> >>>   LD      init/built-in.o
> >>>   LD      .tmp_vmlinux1
> >>> drivers/built-in.o:(__ksymtab_gpl+0xa0): undefined reference to
> >>> `pci_find_aer_capability'
> >>> make: *** [.tmp_vmlinux1] Error 1
> >>> suselinux103x86:/usr/src/pci-2.6 #
> >>
> >> Ah I missed that there was an EXPORT_SYMBOL of this function in
> >> aerdrv_core.c (this is why EXPORT_SYMBOL belongs next to the function
> >> exported!).  Updated patch attached, but it would be fastest for you to
> >> just hand edit your partially built tree, removing the
> >> EXPORT_SYMBOL_GPL(pci_find_aer_capability); line at the bottom of
> >> aerdrv_core.c
> >>
> >> Thanks,
> >> Jesse



-- 
Jesse Barnes, Intel Open Source Technology Center
--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux