lkml.org 
[lkml]   [2008]   [Jul]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] x86: PIC, L-APIC and I/O APIC debug information
On Mon, 21 Jul 2008, Mikael Pettersson wrote:

> It's not even remotely C-like, which will cause confusion
> for anyone or anything trying to read and understand the code.

Not being C-like sounds like an advantage as the reader would be forced
to check what it really means, but I can see what you mean, and the fact
we have got worse constructs elsewhere does not look like a terribly good
argument either.

> Something like the following looks saner to me, would it work for you?
>
> #ifdef debug
> #define apicdebug /* empty */
> #define __apicinit /* empty */
> #else
> #define apicdebug static
> #define __apicinit __init
> #endif
> ...
> apicdebug void __apicinit print_IO_APIC(void)
> { .. }

Hmm, suddenly out of a single line, quite a lot of stuff has emerged. I
am not convinced and I am now thinking of something else instead.

However these debugging facilities are not absolutely necessary for the
essential functionality of the patch, so I will simply remove them for
reconsideration and send a regenerated patch containing the rest only.

Ingo, please drop this one from anywhere you may have it -- I will send a
replacement shortly.

Maciej


\
 
 \ /
  Last update: 2008-07-23 02:15    [W:1.602 / U:0.172 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site