Re: [PATCH 2/9] sector_t format string
From: Andrew Morton
Date: Thu Aug 10 2006 - 15:56:36 EST
On Thu, 10 Aug 2006 23:44:40 +0400
Alexey Dobriyan <adobriyan@xxxxxxxxx> wrote:
> On Thu, Aug 10, 2006 at 12:17:47PM -0700, Joel Becker wrote:
> > On Wed, Aug 09, 2006 at 11:40:19PM -0700, Andrew Morton wrote:
> > > On Wed, 09 Aug 2006 18:20:43 -0700
> > > Mingming Cao <cmm@xxxxxxxxxx> wrote:
> > >
> > > > Define SECTOR_FMT to print sector_t in proper format
> > >
> > > We've thus-far avoided doing this. In fact a similar construct in
> > > device-mapper was recently removed.
> > Yeah, OCFS2 had similar formats, and we were asked to change
> > them to naked casts before inclusion. Seems quite consistent with the
> > rest of the kernel.
> printk("%S", sector_t);
> kill at least one kitten?
It would be really nice to be able to define local enhancements like this
to printf. It would solve lots of these problems quite nicely.
Bus alas, there's no way (afaik) to teach __attribute__((format)) about
them, so gcc will warn.
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/