This is the mail archive of the
systemtap@sourceware.org
mailing list for the systemtap project.
[Bug testsuite/4254] testsuite/buildok/twenty.stp needs change for 2.6.21-rc1 and above kernel.
- From: "pierre dot peiffer at bull dot net" <sourceware-bugzilla at sourceware dot org>
- To: systemtap at sources dot redhat dot com
- Date: 22 Mar 2007 12:34:37 -0000
- Subject: [Bug testsuite/4254] testsuite/buildok/twenty.stp needs change for 2.6.21-rc1 and above kernel.
- References: <20070322121204.4254.srinivasa@in.ibm.com>
- Reply-to: sourceware-bugzilla at sourceware dot org
------- Additional Comments From pierre dot peiffer at bull dot net 2007-03-22 12:34 -------
Subject: Re: New: testsuite/buildok/twenty.stp needs
change for 2.6.21-rc1 and above kernel.
I've sent a correction for that few weeks ago and Frank Ch. Eigler has
integrated it few days ago.
=> now available in CVS
P.
srinivasa at in dot ibm dot com a écrit :
> Since there has been change in the kernel, testsuite/buildok/twent.stp doesn't
> work and hence fails.
> We need to change this testcase to make it work.
>
> Earlier testcase looks like this.
> ===================================================
> #! stap -gp4
>
> # Test for writing to target variables, PR 1131
>
> probe kernel.function("vfs_readdir")
> {
> $file->f_dentry->d_name->len = 1
> }
> =============================================
> Which has to be changed to this
> ========================================
> #! stap -gp4
>
> # Test for writing to target variables, PR 1131
>
> probe kernel.function("vfs_readdir")
> {
> $file->f_path->dentry->d_name->len = 1;
> }
> =================================================
>
> Due to this change in include/linux/file.h
> ===================================
> struct file {
> /*
> * fu_list becomes invalid after file_free is called and queued via
> * fu_rcuhead for RCU freeing
> */
> union {
> struct list_head fu_list;
> struct rcu_head fu_rcuhead;
> } f_u;
> struct path f_path; ====> change
> #define f_dentry f_path.dentry
> #define f_vfsmnt f_path.mnt
> const struct file_operations *f_op;
> atomic_t f_count;
> unsigned int f_flags;
> mode_t f_mode;
> loff_t f_pos;
> =============================================
>
> Please let me know your comments on this.
>
--
http://sourceware.org/bugzilla/show_bug.cgi?id=4254
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.