Message-ID: <41695F85.A0000E3D@tv-sign.ru>
Date: Sun, 10 Oct 2004 20:12:53 +0400
From: Oleg Nesterov <oleg@tv-sign.ru>
MIME-Version: 1.0
To: Andi Kleen <ak@muc.de>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [RFC] __initdata strings
References: <2NNXM-1fZ-5@gated-at.bofh.it> <m38yae1ss7.fsf@averell.firstfloor.org>
Content-Type: text/plain; charset=koi8-r
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 801
Lines: 25

Andi Kleen wrote:
> 
> There is a more generic way to do this with gcc extensions. Something like
> 
> #define __i(x) ({ static char __str[] __initdata = x; __str; })

I can't see any difference with:

#define I_STRING(str)  \
({                                             \
       static char data[] __initdata = (str);  \
       data;                                   \
})

> But I'm not sure the few bytes saved are worth the code uglification.

Probably you are right, but i think it would be few kilobytes,
there are so many __init functions.

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