Re: [PATCH net-next] drivers/net/vxge: Use pr_<level>, fix loggingmacros
From: Joe Perches
Date: Tue Jul 27 2010 - 17:43:50 EST
On Tue, 2010-07-27 at 16:38 -0500, Jon Mason wrote:
> On Tue, Jul 27, 2010 at 01:43:08PM -0700, Joe Perches wrote:
> > Use direct printing of logging messages to save text.
> I have a similar patch queued internally (pending testing).
Great.
[]
> Why not make the entire vxge_debug part of vxge_debug_ll? If
> disabled, that code can be removed completely as it is unnecessary.
> Also, why not call printk directly from vxge_debug? This code had too
> many levels of indirection before, remove all of them (as that is way
> I did in the internal patch).
'cause I don't really want to figure out the nesting,
I just want it to be smaller and easier to understand.
> I did not have any of these changes in my patch. If you want to push
> this seperately, I ack it.
I'll resend separately.
cheers, Joe
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/