From: Daniel G. <da...@fp...> - 2005-01-27 22:56:28
|
Hi, Jake. During my work on the scheduler, I modified the kernel makefiles to build on my linux box (although it doesn't link), so that I could catch stupid compile errors without having to send the patches to my syllable box. In addition, I added -Werror to my CFLAGS. I'm using gcc 3.4.3, so this caught a bunch of warnings that don't show up on syllable. Attached are several patches that fix the most obvious of these warnings. seqlock_init.patch This one should definitely be applied. Seqlocks are being mis-initialized. syllable-warnings-config-h.patch syllable-warnings-device-h.patch syllable-warnings-image-h.patch syllable-warnings-irq-h.patch syllable-warnings-socket-h.patch syllable-warnings-if-h.patch syllable-warnings-net-h.patch These all fix "not a prototype" warnings. This is new to gcc 3.4, where it now wants a void argument on function prototypes that take no arguments. syllable-warnings-spinlock-h.patch atomic_t now has the volatile in it. syllable-warnings-sockios-h.patch No comment around #endif comment syllable-warnings-strace.patch This moves two static structures from a header to the single file they're used in. They should either be moved, or not marked static (but rather const), because gcc warns that static structs are not used. I'd vote for moving them, hence the patch. If some or all of these could be applied, that would make my life somewhat easier. Thanks, Daniel |