[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAPybu_3kqqtfv1XyhruS3AMiRbd=wdMUr4rJL1UmDG_+z2m9jA@mail.gmail.com>
Date: Thu, 20 Oct 2011 10:45:01 +0200
From: Ricardo Ribalda Delgado <ricardo.ribalda@...il.com>
To: Cyrill Gorcunov <gorcunov@...il.com>
Cc: Pekka Enberg <penberg@...helsinki.fi>, a.p.zijlstra@...llo.nl,
paulus@...ba.org, mingo@...e.hu, acme@...stprotocols.net,
anton@...ba.org, daahern@...co.com, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] tools/perf: Increase symbol KSYM_NAME_LEN size
That is 4gb size :S..... I have tried making a dummy program
int tttttt......ttttt (int a, int b){
return a+b;
}
int main(int arg, char *[]argv){
return tttt....tttt(1,2);
}
and it has worked at least until t^16384 both in gcc and in gdb, and I
guess it would have continue working until much more.
Right now, I think we can increase it until 512, to be on the safe
side, and keep it simple. What do you think?
best regards
On Thu, Oct 20, 2011 at 10:14, Cyrill Gorcunov <gorcunov@...il.com> wrote:
> On Thu, Oct 20, 2011 at 10:54:40AM +0300, Pekka Enberg wrote:
> ...
>>
>> Is there some specified maximum length for symbols (e.g. in ELF spec)?
>> I'm OK with the patch but I'd prefer we didn't bump up the number
>> blindly there's something "official" number we can use.
>>
>> Pekka
>
> Elf restricts offsets in string table by 4 bytes, so probably the option
> is to have complete .strtab being read somewhere and use indices when needed.
> (note I didn't check where perf tool need those names).
>
> Cyrill
>
--
Ricardo Ribalda
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists