| Message ID | 148521639917.25631.6919429328589810158.stgit@djiang5-desk3.ch.intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Mon, Jan 23, 2017 at 05:06:39PM -0700, Dave Jiang wrote: > With only vmf being passed in, the pgoff in vmf that is passed in is for > the PTE and not the PMD. Fix up so the pgoff is corrected. > > Signed-off-by: Dave Jiang <dave.jiang@intel.com> Yep, this seems right to me. Thanks for catching it. Reviewed-by: Ross Zwisler <ross.zwisler@linux.intel.com>
diff --git a/include/trace/events/fs_dax.h b/include/trace/events/fs_dax.h index c566ddc..e26affa 100644 --- a/include/trace/events/fs_dax.h +++ b/include/trace/events/fs_dax.h @@ -30,7 +30,8 @@ DECLARE_EVENT_CLASS(dax_pmd_fault_class, __entry->vm_flags = vmf->vma->vm_flags; __entry->address = vmf->address; __entry->flags = vmf->flags; - __entry->pgoff = vmf->pgoff; + __entry->pgoff = linear_page_index(vmf->vma, + vmf->address & PMD_MASK); __entry->max_pgoff = max_pgoff; __entry->result = result; ),
With only vmf being passed in, the pgoff in vmf that is passed in is for the PTE and not the PMD. Fix up so the pgoff is corrected. Signed-off-by: Dave Jiang <dave.jiang@intel.com> --- include/trace/events/fs_dax.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)