* [Bug tapsets/14642] New: pfiles.stp doesn't compile anymore because of missing FD_ISSET
@ 2012-09-28 11:57 mjw at redhat dot com
2012-09-28 17:51 ` [Bug tapsets/14642] " jistone at redhat dot com
2012-09-29 8:58 ` mjw at redhat dot com
0 siblings, 2 replies; 3+ messages in thread
From: mjw at redhat dot com @ 2012-09-28 11:57 UTC (permalink / raw)
To: systemtap
http://sourceware.org/bugzilla/show_bug.cgi?id=14642
Bug #: 14642
Summary: pfiles.stp doesn't compile anymore because of missing
FD_ISSET
Product: systemtap
Version: unspecified
Status: NEW
Severity: normal
Priority: P2
Component: tapsets
AssignedTo: systemtap@sourceware.org
ReportedBy: mjw@redhat.com
Classification: Unclassified
pfiles.stp doesn't compile anymore with my current kernel 3.5.4-1.fc17.x86_64
It is complaining:
error: implicit declaration of function ‘FD_ISSET’
[-Werror=implicit-function-declaration]
I think this has something to do with this bug report:
"__NFDBITS conflict between sys/select.h and linux/posix_types.h"
https://bugzilla.redhat.com/show_bug.cgi?id=837641
But this is for a kernel space script, and that report is for user land code.
What should pfiles.stp include to get the correct FD_* definitions?
--
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
^ permalink raw reply [flat|nested] 3+ messages in thread
* [Bug tapsets/14642] pfiles.stp doesn't compile anymore because of missing FD_ISSET
2012-09-28 11:57 [Bug tapsets/14642] New: pfiles.stp doesn't compile anymore because of missing FD_ISSET mjw at redhat dot com
@ 2012-09-28 17:51 ` jistone at redhat dot com
2012-09-29 8:58 ` mjw at redhat dot com
1 sibling, 0 replies; 3+ messages in thread
From: jistone at redhat dot com @ 2012-09-28 17:51 UTC (permalink / raw)
To: systemtap
http://sourceware.org/bugzilla/show_bug.cgi?id=14642
Josh Stone <jistone at redhat dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |jistone at redhat dot com
--- Comment #1 from Josh Stone <jistone at redhat dot com> 2012-09-28 17:51:29 UTC ---
(In reply to comment #0)
> pfiles.stp doesn't compile anymore with my current kernel 3.5.4-1.fc17.x86_64
> It is complaining:
>
> error: implicit declaration of function ‘FD_ISSET’
> [-Werror=implicit-function-declaration]
>
> I think this has something to do with this bug report:
> "__NFDBITS conflict between sys/select.h and linux/posix_types.h"
> https://bugzilla.redhat.com/show_bug.cgi?id=837641
>
> But this is for a kernel space script, and that report is for user land code.
It's true, that commit 8ded2bbc removed the FD_ISSET macro from the kernel.
Even before that though, it was simply a wrapper for __FD_ISSET, which was
removed earlier in commit cf420048.
> What should pfiles.stp include to get the correct FD_* definitions?
The previous __FD_ISSET was very simple:
static inline int __FD_ISSET(unsigned long __fd, const __kernel_fd_set
*__fdsetp)
{
return test_bit(__fd, __fdsetp->fds_bits);
}
It should be fine for pfiles.stp to use test_bit the same way.
--
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
^ permalink raw reply [flat|nested] 3+ messages in thread
* [Bug tapsets/14642] pfiles.stp doesn't compile anymore because of missing FD_ISSET
2012-09-28 11:57 [Bug tapsets/14642] New: pfiles.stp doesn't compile anymore because of missing FD_ISSET mjw at redhat dot com
2012-09-28 17:51 ` [Bug tapsets/14642] " jistone at redhat dot com
@ 2012-09-29 8:58 ` mjw at redhat dot com
1 sibling, 0 replies; 3+ messages in thread
From: mjw at redhat dot com @ 2012-09-29 8:58 UTC (permalink / raw)
To: systemtap
http://sourceware.org/bugzilla/show_bug.cgi?id=14642
Mark Wielaard <mjw at redhat dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |FIXED
--- Comment #2 from Mark Wielaard <mjw at redhat dot com> 2012-09-29 08:58:14 UTC ---
commit 95b4f5c10dd5bdf63a32b62d1e8fd08f9cf80a2e
Author: Mark Wielaard <mjw@redhat.com>
Date: Sat Sep 29 10:24:38 2012 +0200
PR14642 pfiles.stp: Use test_bit directly instead of FD_ISSET.
FD_ISSET might not be defined on all kernels.
Tested against 3.5.4-1.fc17.x86_64 and 2.6.18-308.1.1.el5 (i686).
--
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2012-09-29 8:58 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-09-28 11:57 [Bug tapsets/14642] New: pfiles.stp doesn't compile anymore because of missing FD_ISSET mjw at redhat dot com
2012-09-28 17:51 ` [Bug tapsets/14642] " jistone at redhat dot com
2012-09-29 8:58 ` mjw at redhat dot com
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).