tapset + testsuite corrections

Pierre Peiffer pierre.peiffer@bull.net
Thu Feb 15 14:36:00 GMT 2007


Frank Ch. Eigler a écrit :
> 
> Since we have some interest in keeping tapsets working with older
> kernels, we generally need to use kernel version-sensitive
> #if/#else/#endif conditional compilation in embedded-C code. 

Yes, I had it in mind and took care of this where it was needed.
So normally, it should work for both old and new kernels.

> (An
> analogous %( %? %: %) construct exists for script code, and is
> documented in the man pages.)

Thanks for the tip, I did not discover this yet.

> 
> That is fine.  Please also see src/HACKING for additional guidelines,
> such as designation of a copyright holder for your changes.

Ok, thanks.

I've attached the same patch, with updates, and unified style this time.
Here is the Changelog:

	* tapset/LKET/nfs.stp, tapset/nfs.stp, tapset/vfs.stp: local variables
	f_dentry renamed, because conflicting with a new #define in kernel
	header linux/fs.h in 2.6.20.
	* tapset/rpc.stp: struct rpc_xprt modified since kernel 2.6.19.
	* testsuite/buildok/twenty.stp: updated with new struct file since
	kernel 2.6.20
	* testsuite/semok/seventeen.stp: updated with new function name
	since kernel 2.6.19

-- 
Pierre Peiffer
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fixup_kernel_2620.patch
Type: text/x-patch
Size: 9919 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/systemtap/attachments/20070215/3ee5f91e/attachment.bin>


More information about the Systemtap mailing list