Re: [PATCH] lib/vsprintf: add %pT[C012] format specifier

[Pavel Machek]

> > > I'm not nacking this, just stating my view.
> > 
> > And I believe Andrew clearly stated his view, on the very topic you
> > asked him on.
> 
> I believe Andrew's view:
> 
> On Sat, 2013-12-28 at 12:08 -0800, Andrew Morton wrote:
> > On Sat, 28 Dec 2013 11:53:25 -0800 Joe Perches <joe@xxxxxxxxxxx> wrote:
> > > Tell me again, what's wrong with using p or current?
> > > printk("%pt", current);
> > 
> > Nothing much.  It's just that all these callsites are generating the
> > code to pass an argument which the callee already has access to. 
> > Optimizing that will reduce text size a bit.
> 
> was that the argument passing was the primary issue.

Yes. He dislikes passing argument callee has already access
to. "current". This patch does not do this, it just passes the NULL as
a marker... and to keep printf() checkers happy.

I believe this is way better than alternatives.

> Now that that's not done, this code actually uses a
> different concept, that "NULL" is special when using
> %pT.  If the argument isn't eliminated all together,
> I think that new concept should be avoided.
> 
> The additional cost of using current vs NULL is ~zero.

The additional cost of current vs NULL is cca 8 bytes per caller. Test
for NULL is cca 4 bytes, maybe 20 bytes total. I believe it is worth
it.

								Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
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/