Hello Indraneel,

I think I fixed this bug in git repo, see attached patch. I was only
able to test it with arm emulator and cross-compiler and probes located
in the kernel itself, not in modules, so if you could test it with
native compilation and probes in modules it will be very good.

This patch skips saving of build-id for kernels like one for ARM. I
think the previous patch only worked because build-id check was skipped
because of check in sym.c:261 (comment says "shouldn't happen").

Regards,
Eugeniy Meshcheryakov

14 грудня 2009 о 17:55 +0530 Indraneel Mukherjee написав(-ла):
> On Mon, Dec 14, 2009 at 4:30 PM, Eugeniy Meshcheryakov <eugen@debian.org> wrote:
> > Yes, I know about that problem. But I'm not sure whether proposed patch
> > fixes anything. Were you able to _use_ modules made with that patch applied?
> 
> Yes, i was able to _use_ SystemTap on ARM after applying that patch +
> your fix. I ran the following script and it worked absolutely fine:
> 
> global reads
> probe kernel.function("do_writepages") {
>         reads[execname()]++
>         }
> probe timer.s(3)
> {
>         foreach (count in reads)
>                 printf("%s : %d \n", count, reads[count])
> }
> 
> OUTPUT of script:
> pdflush : 4
> pdflush : 4
> pdflush : 8
> pdflush : 8
> pdflush : 12
> pdflush : 12
> pdflush : 12
> ...
> ...
> > Or does it fixes only compilation? My guess is that module
> > initialization will fail because build-id does not match.
> 
> No it does not fail.
> 
> Regards,
> Indro
> 
> >
> > 14 грудня 2009 о 16:11 +0530 Indraneel Mukherjee написав(-ла):
> >> Thanks for the fix. Also, there was another patch that i had to apply
> >> to get Systemtap working for ARM. I checked the latest git and the fix
> >> is not available for ARM. The fix is here:
> >> http://sources.redhat.com/ml/systemtap/2009-q4/msg00574.html
> >> Without the above fix, the ARM(arm-linux is non-relocatable) stap FAILS.
> >>
> >> This is related to
> >> http://www.mail-archive.com/debian-bugs-dist@lists.debian.org/msg690166.html
> >>
> >> Shouldn't this fix be added to the Systemtap tree to get it right for ARM?
> >
> > -----BEGIN PGP SIGNATURE-----
> > Version: GnuPG v1.4.10 (GNU/Linux)
> >
> > iEYEARECAAYFAksmGt0ACgkQKaC6+zmozOLZtwCfasFxn9Q8LgZgLQ59sKIEyJT3
> > dEMAoJC1rOkL/NN+mABa5p16hDqXMMV1
> > =ud4E
> > -----END PGP SIGNATURE-----
> >
> >