Launchpad has imported 14 comments from the remote bug at https://bugzilla.redhat.com/show_bug.cgi?id=1240682.
If you reply to an imported comment from within Launchpad, your comment will be sent to the remote bug automatically. Read more about Launchpad's inter-bugtracker facilities at https://help.launchpad.net/InterBugTracking. ------------------------------------------------------------------------ On 2015-07-07T14:02:52+00:00 Ruben wrote: Description of problem: I was trying to run the following simple probe: ruben@wodan: ~$ sudo stap -v -e 'probe vfs.read {printf("read performed\n"); exit()}' Pass 1: parsed user script and 122 library script(s) using 237480virt/49484res/7192shr/42152data kb, in 190usr/70sys/250real ms. Pass 2: analyzed script: 1 probe(s), 1 function(s), 4 embed(s), 0 global(s) using 376932virt/183612res/8544shr/181604data kb, in 2170usr/1540sys/4358real ms. Pass 3: translated to C into "/tmp/stapbdpxn3/stap_a0ec17f995e8f89d672d8c2eb7fe7c24_1693_src.c" using 376932virt/183860res/8792shr/181604data kb, in 10usr/390sys/417real ms. In file included from include/linux/mutex.h:15:0, from /tmp/stapbdpxn3/stap_a0ec17f995e8f89d672d8c2eb7fe7c24_1693_src.c:25: include/linux/spinlock_types.h:55:14: error: ‘__ARCH_SPIN_LOCK_UNLOCKED’ undeclared here (not in a function) .raw_lock = __ARCH_SPIN_LOCK_UNLOCKED, \ ^ include/linux/spinlock_types.h:79:15: note: in expansion of macro ‘__RAW_SPIN_LOCK_INITIALIZER’ { { .rlock = __RAW_SPIN_LOCK_INITIALIZER(lockname) } } ^ include/linux/spinlock_types.h:82:16: note: in expansion of macro ‘__SPIN_LOCK_INITIALIZER’ (spinlock_t ) __SPIN_LOCK_INITIALIZER(lockname) ^ include/linux/mutex.h:111:18: note: in expansion of macro ‘__SPIN_LOCK_UNLOCKED’ , .wait_lock = __SPIN_LOCK_UNLOCKED(lockname.wait_lock) \ ^ include/linux/mutex.h:117:27: note: in expansion of macro ‘__MUTEX_INITIALIZER’ struct mutex mutexname = __MUTEX_INITIALIZER(mutexname) ^ /tmp/stapbdpxn3/stap_a0ec17f995e8f89d672d8c2eb7fe7c24_1693_src.c:26:8: note: in expansion of macro ‘DEFINE_MUTEX’ static DEFINE_MUTEX(module_refresh_mutex); ^ scripts/Makefile.build:258: recipe for target '/tmp/stapbdpxn3/stap_a0ec17f995e8f89d672d8c2eb7fe7c24_1693_src.o' failed make[1]: *** [/tmp/stapbdpxn3/stap_a0ec17f995e8f89d672d8c2eb7fe7c24_1693_src.o] Error 1 Makefile:1386: recipe for target '_module_/tmp/stapbdpxn3' failed make: *** [_module_/tmp/stapbdpxn3] Error 2 WARNING: kbuild exited with status: 2 Pass 4: compiled C into "stap_a0ec17f995e8f89d672d8c2eb7fe7c24_1693.ko" in 8650usr/5580sys/15546real ms. Pass 4: compilation failed. [man error::pass4] Version-Release number of selected component (if applicable): systemtap-2.8-1.fc23.x86_64 ruben@wodan: ~$ uname -r 4.2.0-0.rc0.git4.1.fc23.x86_64 Reply at: https://bugs.launchpad.net/ubuntu/+source/systemtap/+bug/1545330/comments/0 ------------------------------------------------------------------------ On 2015-07-07T14:24:29+00:00 David wrote: I'm seeing the same thing, looking into it now. Reply at: https://bugs.launchpad.net/ubuntu/+source/systemtap/+bug/1545330/comments/1 ------------------------------------------------------------------------ On 2015-07-07T15:18:03+00:00 David wrote: It looks like this was caused by the addition of qspinlocks in the following kernel commit: ==== commit a33fda35e3a7655fb7df756ed67822afb5ed5e8d Author: Waiman Long <waiman.l...@hp.com> Date: Fri Apr 24 14:56:30 2015 -0400 locking/qspinlock: Introduce a simple generic 4-byte queued spinlock ==== With this change we need to include linux/module.h before including linux/mutex.h. To fix your immediate problem, add the following text to /usr/share/systemtap/runtime/linux/runtime_defines.h: ==== #ifndef _LINUX_RUNTIME_DEFINES_H_ #define _LINUX_RUNTIME_DEFINES_H_ #include <linux/module.h> #include <linux/mutex.h> #endif /* _LINUX_RUNTIME_DEFINES_H_ */ ==== I'm not 100% sure if that's the final fix, but it will get you going again. Reply at: https://bugs.launchpad.net/ubuntu/+source/systemtap/+bug/1545330/comments/2 ------------------------------------------------------------------------ On 2015-07-07T15:48:32+00:00 David wrote: The final fix ended up being a small tweak to our code generation - commit 320e1ec. <https://www.sourceware.org/git/gitweb.cgi?p=systemtap.git;a=commitdiff;h=320e1ecb16427b5769f0f5a097d80823ee1fb765> Reply at: https://bugs.launchpad.net/ubuntu/+source/systemtap/+bug/1545330/comments/3 ------------------------------------------------------------------------ On 2015-07-07T17:16:49+00:00 Ruben wrote: Thanks for the quick fix David! Reply at: https://bugs.launchpad.net/ubuntu/+source/systemtap/+bug/1545330/comments/4 ------------------------------------------------------------------------ On 2015-09-14T21:55:12+00:00 Marcelo wrote: Hi David, would it be possible to do a F21 update too? It's also broken by this bug. Thanks. Reply at: https://bugs.launchpad.net/ubuntu/+source/systemtap/+bug/1545330/comments/5 ------------------------------------------------------------------------ On 2015-09-15T21:13:05+00:00 David wrote: (In reply to Marcelo Ricardo Leitner from comment #5) > Hi David, would it be possible to do a F21 update too? It's also broken by > this bug. Thanks. Thanks for letting us know. I'll see if we can't get an update out soon for f21 (and f22). Reply at: https://bugs.launchpad.net/ubuntu/+source/systemtap/+bug/1545330/comments/6 ------------------------------------------------------------------------ On 2015-09-16T16:25:47+00:00 David wrote: (In reply to Marcelo Ricardo Leitner from comment #5) > Hi David, would it be possible to do a F21 update too? It's also broken by > this bug. Thanks. Marcelo, Have you got a reproducer for this bug on f21? The one in comment #1 seems to work fine there. Reply at: https://bugs.launchpad.net/ubuntu/+source/systemtap/+bug/1545330/comments/7 ------------------------------------------------------------------------ On 2015-09-16T17:22:50+00:00 Marcelo wrote: Oops. Then it depends on kernel version. I'm running a custom one, 4.2.0-rc8+, and the one in comment #0 reproduces it in here (greped for brevity): [root@localhost ~]# stap -v -e 'probe vfs.read {printf("read performed\n"); exit()}' 2>&1 | grep error include/linux/spinlock_types.h:55:14: error: ‘__ARCH_SPIN_LOCK_UNLOCKED’ undeclared here (not in a function) Pass 4: compilation failed. [man error::pass4] [root@localhost ~]# uname -r 4.2.0-rc8+ [root@localhost ~]# cat /etc/fedora-release Fedora release 21 (Twenty One) [root@localhost ~]# rpm -q systemtap systemtap-2.8-1.fc21.x86_64 I also cannot reproduce it with 4.1.6-100.fc21.x86_64. Now checking, the fix seems to be compatible with both, just yeah not strictly needed for f21.. I hope it's okay to include it anyway. Reply at: https://bugs.launchpad.net/ubuntu/+source/systemtap/+bug/1545330/comments/8 ------------------------------------------------------------------------ On 2015-09-17T13:52:55+00:00 Fedora wrote: systemtap-2.8-2.fc21 has been submitted as an update to Fedora 21. https://bodhi.fedoraproject.org/updates/FEDORA-2015-16095 Reply at: https://bugs.launchpad.net/ubuntu/+source/systemtap/+bug/1545330/comments/9 ------------------------------------------------------------------------ On 2015-09-18T05:20:38+00:00 Fedora wrote: systemtap-2.8-2.fc21 has been pushed to the Fedora 21 testing repository. If problems still persist, please make note of it in this bug report.\nIf you want to test the update, you can install it with \n su -c 'yum --enablerepo=updates-testing update systemtap'. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-16095 Reply at: https://bugs.launchpad.net/ubuntu/+source/systemtap/+bug/1545330/comments/10 ------------------------------------------------------------------------ On 2015-10-11T21:39:34+00:00 Fedora wrote: systemtap-2.9-1.fc21 has been submitted as an update to Fedora 21. https://bodhi.fedoraproject.org/updates/FEDORA-2015-9ef098b6d4 Reply at: https://bugs.launchpad.net/ubuntu/+source/systemtap/+bug/1545330/comments/11 ------------------------------------------------------------------------ On 2015-10-13T18:21:16+00:00 Fedora wrote: systemtap-2.9-1.fc21 has been pushed to the Fedora 21 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with $ su -c 'dnf --enablerepo=updates-testing update systemtap' You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-9ef098b6d4 Reply at: https://bugs.launchpad.net/ubuntu/+source/systemtap/+bug/1545330/comments/12 ------------------------------------------------------------------------ On 2015-10-23T16:19:49+00:00 Fedora wrote: systemtap-2.9-1.fc21 has been pushed to the Fedora 21 stable repository. If problems still persist, please make note of it in this bug report. Reply at: https://bugs.launchpad.net/ubuntu/+source/systemtap/+bug/1545330/comments/13 ** Changed in: systemtap (Fedora) Status: Unknown => Fix Released ** Changed in: systemtap (Fedora) Importance: Unknown => Undecided -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1545330 Title: [wily][regression] systemtap script compilation broken by new kernels To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1545330/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
