On Thu, Jul 05, 2007 at 01:34:20AM +0400, Alexey Dobriyan wrote:
> 1) Drop __KERNEL__ out of profile.h. It contains only internal kernel stuff
> and
>not in exported headers list
> 2) Put profile.c under CONFIG_PROFILING. You enabled profiling in config, you
>will get it. Removes condition
On 08/07/07, Adrian Bunk <[EMAIL PROTECTED]> wrote:
On Thu, Jul 05, 2007 at 01:50:27AM +0200, Jesper Juhl wrote:
> On 04/07/07, Alexey Dobriyan <[EMAIL PROTECTED]> wrote:
>> 1) Drop __KERNEL__ out of profile.h. It contains only internal kernel
>> stuff and
>>not in exported headers list
>
> E
On Thu, Jul 05, 2007 at 01:50:27AM +0200, Jesper Juhl wrote:
> On 04/07/07, Alexey Dobriyan <[EMAIL PROTECTED]> wrote:
>> 1) Drop __KERNEL__ out of profile.h. It contains only internal kernel
>> stuff and
>>not in exported headers list
>
> Even if it's not in the list of exported headers, does
On Thu, Jul 05, 2007 at 01:50:27AM +0200, Jesper Juhl wrote:
> One tiny comment below.
> >+#define prof_on 0
> >+static inline void profile_init(void)
> >+{
> >+}
>
> Just to be pedantic; don't we want a blank line between functions
> here, even if they are empty?
Dunno, it's boilerplate code, no
On Thu, 5 Jul 2007, Denis Vlasenko wrote:
> On Thursday 05 July 2007 01:50, Jesper Juhl wrote:
> > > Removes conditional branch from schedule(). Code savings on my
> > >usual config:
> > >
> > >textdata bss dec hex filename
> > > 2921871 179895 180224 3281
On Thursday 05 July 2007 01:50, Jesper Juhl wrote:
> > Removes conditional branch from schedule(). Code savings on my
> >usual config:
> >
> >textdata bss dec hex filename
> > 2921871 179895 180224 3281990 321446 vmlinux before
> > 2920141
On 04/07/07, Alexey Dobriyan <[EMAIL PROTECTED]> wrote:
1) Drop __KERNEL__ out of profile.h. It contains only internal kernel stuff and
not in exported headers list
Even if it's not in the list of exported headers, does it really hurt
to retain that extra safeguard?
2) Put profile.c under
Alexey Dobriyan <[EMAIL PROTECTED]> writes:
> 2) Put profile.c under CONFIG_PROFILING. You enabled profiling in config, you
>will get it. Removes conditional branch from schedule(). Code savings on my
>usual config:
>
> textdata bss dec hex filename
> 292187
1) Drop __KERNEL__ out of profile.h. It contains only internal kernel stuff and
not in exported headers list
2) Put profile.c under CONFIG_PROFILING. You enabled profiling in config, you
will get it. Removes conditional branch from schedule(). Code savings on my
usual config:
t
9 matches
Mail list logo