locks: fix generic_delete_lease tracepoint to use victim pointer

Jeff Layton · Jeff Layton · commit a9b1b455c519 · 2015-03-14T09:45:35.000-04:00
It's possible that "fl" won't point at a valid lock at this point, so
use "victim" instead which is either a valid lock or NULL.

Signed-off-by: Jeff Layton &lt;jeff.layton@primarydata.com&gt;
diff --git a/fs/locks.c b/fs/locks.c
@@ -1728,7 +1728,7 @@ static int generic_delete_lease(struct file *filp, void *owner)
 			break;
 		}
 	}
-	trace_generic_delete_lease(inode, fl);
+	trace_generic_delete_lease(inode, victim);
 	if (victim)
 		error = fl->fl_lmops->lm_change(victim, F_UNLCK, &dispose);
 	spin_unlock(&ctx->flc_lock);

Original file line number	Diff line number	Diff line change
`@@ -1728,7 +1728,7 @@ static int generic_delete_lease(struct file filp, void owner)`
`1728`	`1728`	`break;`
`1729`	`1729`	`}`
`1730`	`1730`	`}`
`1731`		`- trace_generic_delete_lease(inode, fl);`
	`1731`	`+ trace_generic_delete_lease(inode, victim);`
`1732`	`1732`	`if (victim)`
`1733`	`1733`	`error = fl->fl_lmops->lm_change(victim, F_UNLCK, &dispose);`
`1734`	`1734`	`spin_unlock(&ctx->flc_lock);`