public inbox for systemtap@sourceware.org
 help / color / mirror / Atom feed
* master's current HEAD still uses file->f_dentry, removed in Linux 3.19
@ 2016-04-15  9:35 Giovanni Gherdovich
  2016-04-15 11:41 ` Giovanni Gherdovich
  0 siblings, 1 reply; 2+ messages in thread
From: Giovanni Gherdovich @ 2016-04-15  9:35 UTC (permalink / raw)
  To: systemtap

Hello,

was this patch from William Cohen (Dec 2014)
https://sourceware.org/ml/systemtap/2014-q4/msg00257.html
ever applied?

I cloned from git://sourceware.org/git/systemtap.git, and on the tag
release-3.0 (two weeks ago) grepping for f_dentry on the file
runtime/linux/task_finder2.c still gives results.

f_dentry was a macro defined in the Linux kernel (precisely,
include/linux/fs.h) removed in the 3.19 release. With the SystemTap
version I have, even a minimal test script like

stap -ve 'probe begin { log("hello world") exit () }'

fails compilation with

error: 'struct file' has no member named 'f_dentry'

The patch I link above fixes this, but I can't find traces of it in the
git history. Is there something I'm missing?

Regards,
Giovanni

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: master's current HEAD still uses file->f_dentry, removed in Linux 3.19
  2016-04-15  9:35 master's current HEAD still uses file->f_dentry, removed in Linux 3.19 Giovanni Gherdovich
@ 2016-04-15 11:41 ` Giovanni Gherdovich
  0 siblings, 0 replies; 2+ messages in thread
From: Giovanni Gherdovich @ 2016-04-15 11:41 UTC (permalink / raw)
  To: systemtap

On Fri, 2016-04-15 at 11:35 +0200, Giovanni Gherdovich wrote:
> 
> [...] With the SystemTap
> version I have, even a minimal test script like
> 
> stap -ve 'probe begin { log("hello world") exit () }'
> 
> fails compilation with
> 
> error: 'struct file' has no member named 'f_dentry'

Sorry, my bad.
I was running SystemTap 2.5; I upgraded to 3.0 and the problem if
fixed.

The rev I missed is

commit: 547f6291433b574186e82f4afe233b00b7eb242e
date: Tue, 6 Jan 2015 10:07:01 -0600
title: Fixed compile problem on 3.18 kernels, caused by removal of
       dentry macro.
https://sourceware.org/git/?p=systemtap.git;a=commit;h=547f6291433b5741
86e82f4afe233b00b7eb242e

appeared in 2.7.

Regards,
Giovanni

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-04-15 11:41 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-04-15  9:35 master's current HEAD still uses file->f_dentry, removed in Linux 3.19 Giovanni Gherdovich
2016-04-15 11:41 ` Giovanni Gherdovich

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).