Re: ACPI modpost failure in 2.6.23-rc1-git2

[Andy Whitcroft <apw@shadowen.org>]

Andy Whitcroft wrote:
> Michal Piotrowski wrote:
>> Hi Andy,
>>
>> On 06/08/07, Andy Whitcroft <apw@shadowen.org> wrote:
>>> Between 2.6.23-rc1-git1 and 2.6.23-rc1-git2 we started getting the
>>> following fatal error during modpost on an x86_64 machine:
>>>
>>>   FATAL: drivers/acpi/video: sizeof(struct acpi_device_id)=20 is not a
>>>   modulo of the size of section __mod_acpi_device_table=48.
>>>   Fix definition of struct acpi_device_id in mod_devicetable.h
>>>   make[1]: *** [__modpost] Error 1
>>>
>>> This check seems to have been added as part of the following commit:
>>>
>>>   commit 29b71a1ca74491fab9fed09e9d835d840d042690
>>>   Author: Thomas Renninger <trenn@suse.de>
>>>   Date:   Mon Jul 23 14:43:51 2007 +0200
>>>
>>>       ACPI: autoload modules - Create ACPI alias interface
>>>
>>> Compiler version:
>>>   Reading specs from /usr/lib/gcc/i486-linux/3.4.4/specs
>>>   Configured with: ../src/configure -v
>>>   --enable-languages=c,c++,java,f77,pascal,objc,ada,treelang --prefix=/usr
>>>   --libexecdir=/usr/lib --with-gxx-include-dir=/usr/include/c++/3.4
>>>   --enable-shared --with-system-zlib --enable-nls
>>>   --without-included-gettext --program-suffix=-3.4 --enable-__cxa_atexit
>>>   --enable-libstdcxx-allocator=mt --enable-clocale=gnu
>>>   --enable-libstdcxx-debug --enable-java-gc=boehm --enable-java-awt=gtk
>>>   --disable-werror i486-linux
>>>   Thread model: posix
>>>   gcc version 3.4.4 20050314 (prerelease) (Debian 3.4.3-13sarge1)
>>>
>>> We are compiling a 64bit kernel on a machine with a pretty old
>>> userspace which is 32bit.
>>>
>>> Let me know if you need more info.
>>>
>>> -apw
>> similar to
>>
>> Subject         : modpost bug breaks ia64 cross compilation
>> References      : http://lkml.org/lkml/2007/7/27/30
>>                   http://lkml.org/lkml/2007/7/27/418
>> Last known good : ?
>> Submitter       : Jan Dittmer <jdi@l4x.org>
>> Caused-By       : Thomas Renninger <trenn@suse.de>
>>                   commit 29b71a1ca74491fab9fed09e9d835d840d042690
>> Handled-By      : ?
>> Patch           : http://lkml.org/lkml/2007/8/2/211
>> Status          : patch was suggested
>>
>> Please try this patch http://lkml.org/lkml/2007/8/2/211
> 
> Will do.  That is vile though.
> 
> It feels like we should be generating this information in the kernel
> build and extracting it from the built kernel.

Confirmed that this patch does indeed fix up my problem machine.

Its very ugly but it does work.

-apw