[PATCH 0/10] introduction: check pr_debug() arguments

introduction: check pr_debug() arguments

I was recently frustrated when I broke the arguments to a pr_debug() call and
the bug went unnoticed until I defined DEBUG.  I poked around a bit and found
that I wasn't alone in breaking pr_debug() arguments.

Instead of having pr_debug() hide broken arguments when DEBUG isn't defined,
let's make it an empty inline and have gcc check it's format specifier.

What follows are the patches that fix up the existing bad pr_debug() calls.
The worst flat out get syntax wrong or reference non-existant symbols.

With those out of the way, the final patch makes the change to pr_debug().  The
net result doesn't affect a allyesconfig x86-64 build.  My apologies to other
builds that will be exposed to broken pr_debug() arguments.  What a great
opportunity to fix them!

- z
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• Follow-Ups:
  • [PATCH 4/10] sysfs: use size_t length modifier in pr_debug format arguments
    • From: Zach Brown <[email protected]>
  • [PATCH 3/10] configfs: use size_t length modifier in pr_debug format argument
    • From: Zach Brown <[email protected]>
  • [PATCH 7/10] dell_rbu: fix pr_debug argument warnings
    • From: Zach Brown <[email protected]>
  • [PATCH 9/10] trident: use size_t length modifier in pr_debug format arguments
    • From: Zach Brown <[email protected]>
  • [PATCH 2/10] aio: use size_t length modifier in pr_debug format arguments
    • From: Zach Brown <[email protected]>
  • [PATCH 8/10] ifb: replace missing comma to separate pr_debug arguments
    • From: Zach Brown <[email protected]>
  • [PATCH 10/10] check pr_debug() arguments
    • From: Zach Brown <[email protected]>
  • [PATCH 6/10] tipar: repair nonexistant pr_debug argument use
    • From: Zach Brown <[email protected]>
  • [PATCH 5/10] umem: repair nonexistant bh pr_debug reference
    • From: Zach Brown <[email protected]>
  • [PATCH 1/10] futex: remove extra pr_debug format specifications
    • From: Zach Brown <[email protected]>

• Prev by Date: Re: [RFC] e-mail clients
• Next by Date: [PATCH 1/10] futex: remove extra pr_debug format specifications
• Previous by thread: Linux 2.6.17.12
• Next by thread: [PATCH 1/10] futex: remove extra pr_debug format specifications
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]