Re: pahole BRAIN FART ALERT on F12

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Em Tue, Nov 24, 2009 at 08:28:31AM -0500, Mike Snitzer escreveu:
> On Mon, Nov 23 2009 at  7:10pm -0500,
> Arnaldo de Melo <acme@xxxxxxxxxx> wrote:
> > Em Mon, Nov 23, 2009 at 05:56:15PM -0500, Mike Snitzer escreveu:
> > > On Mon, Nov 23 2009 at  5:49pm -0500,
> > > Arnaldo de Melo <acme@xxxxxxxxxx> wrote:
> > > > Em Mon, Nov 23, 2009 at 05:17:03PM -0500, Mike Snitzer escreveu:
> > > > > Unfortunately, when I run pahole against lvm2 I'm getting many BRAIN
> > > > > FART ALERTs on F12 x86_64 (dwarves-1.7-5.x86_64).  I had a look at your
> > > > > OLS 2007 "7 dwarves" paper and figured I'd try your minimalist
> > > > > swiss_cheese example, this is what I get:
> > > > > $ pahole swiss_cheese
> > > > > struct cheese {
> > > > >         char                       name[17];             /*     0    17 */
> > > > >         short int                  age;                  /*     0     2 */
> > > > >         char                       type;                 /*     0     1 */
> > <SNIP>
> > > > >         /* BRAIN FART ALERT! 48 != 17 + 0(holes), diff = 31 */
> > > > > };
> > > > You should, I think this is related to some recent elfutils changes that
> > > > Mark Wieelard warned me about it but I hadn't time yet to act upon, will
> > > > do so tomorrow, thanks for the report!
> >  
> > > BTW, I did try the latest git version from:
> > > git://git.kernel.org/pub/scm/linux/kernel/git/acme/pahole.git
> > > It didn't work either.
> > 
> > Can you try with the attached patch applied against the git repo
> > version?
> 
> Works great, thanks!
> 
> Would you still like me to file an F12 bug?

Please, as I'll have to go the process for updating the package... :-)

- Arnaldo
--
To unsubscribe from this list: send the line "unsubscribe dwarves" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux