[PATCH 0/5] ide debugging macros

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: <Bartlomiej@...>, <Zolnierkiewicz@...>, <bzolnier@...>
Cc: <linux-kernel@...>, <linux-ide@...>, Borislav Petkov <petkovbb@...>
Date: Sunday, August 17, 2008 - 1:23 pm

Hi Bart,

here's something i've been wanting to do for a long time: debugging macros. The
reason for it is that i got tired of adding debug printk's everytime i'm testing
something so here we go.

The debugging macro is similar to the old ones but is one for all drivers
(currently only ide-floppy), is nice on branch prediction and is controlled by a
drive->debug_mask switch which is a module parameter and as such can be set at
module load time, of course. I've been thinking of adding also a sysfs attribute
too but can't seem to find quite the justification for it so no sysfs for now :)

In addition, one can still optimize away all the debug calls in the old manner
and i'm sure those will be removed completely too when ide generic conversion is
done.

Please tell me what you think, what can be changed/improved and after we've
figured out the details I'll do the other drivers too.

Thanks.

 drivers/ide/ide-cd.c     |   17 ++---
 drivers/ide/ide-floppy.c |  168 +++++++++++++++++++++++++---------------------
 drivers/ide/ide-tape.c   |   23 ++----
 include/linux/ide.h      |   31 ++++++++-
 4 files changed, 135 insertions(+), 104 deletions(-)
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 0/5] ide debugging macros, Borislav Petkov, (Sun Aug 17, 1:23 pm)
Re: [PATCH 0/5] ide debugging macros, Bartlomiej Zolnierkiewicz..., (Mon Aug 18, 5:08 pm)
Re: [PATCH 0/5] ide debugging macros, Borislav Petkov, (Tue Aug 19, 2:23 am)
Re: [PATCH 0/5] ide debugging macros, Borislav Petkov, (Tue Aug 19, 2:15 am)
Re: [PATCH 0/5] ide debugging macros, Bartlomiej Zolnierkiewicz..., (Mon Aug 18, 5:29 pm)
[PATCH 5/5] ide-floppy: add a debug_mask module parameter, Borislav Petkov, (Sun Aug 17, 1:23 pm)
[PATCH 1/5] ide: unify conversion macros, Borislav Petkov, (Sun Aug 17, 1:23 pm)
[PATCH 3/5] ide: add a driver-wide debugging macro, Borislav Petkov, (Sun Aug 17, 1:23 pm)
Re: [PATCH 3/5] ide: add a driver-wide debugging macro, Joe Perches, (Sun Aug 17, 2:13 pm)
Re: [PATCH 3/5] ide: add a driver-wide debugging macro, Borislav Petkov, (Sun Aug 17, 2:24 pm)
[PATCH 2/5] ide: add drive-&gt;debug_mask switch, Borislav Petkov, (Sun Aug 17, 1:23 pm)