Pete/Piet Delaney wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> gcc 4.1.1 is still a supported compiler by kernel.org; since we have
> what appears to be reasonable fix and it will prevent folks who happen
> to be using 4.1.1 from having weird problems it seems reasonable to
> apply the fix.
>
> I like to hear reasonable argument against including it.
>
>
Piet,
I do not have any problem with the original patch from a technical
angle. What I do see is a problem is that the patch does not match
what so ever to the current tree. This means that perhaps some other
prior patch is missing, or there is a non-obvious fork that happened
somewhere.
As an example in the original patch there was a section that removed the
following from kernel/kgdb.c.
- int __attribute__ ((weak))
- kgdb_arch_init(void)
- {
- return 0;
- }
As far as I can tell Tom's tree for the 2.6.18-testing did not contain
these extra functions that were supposed to be moved. I will gladly
consider the weak symbols fixes for the 4.1.0 compiler but someone needs
to rework the existing patch or find references to what happened to the
work in between.
Jason.
-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Kgdb-bugreport mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/kgdb-bugreport
