[lustre-devel] [PATCH 069/151] lustre: ptlrpc: use lu_extent in layout_intent
James Simmons
jsimmons at infradead.org
Tue Oct 1 10:54:29 PDT 2019
> > From: Jinshan Xiong <jinshan.xiong at gmail.com>
> >
> > This way it will be easier to check and print the extent
> > of layout.
> >
> > WC-bug-id: https://jira.whamcloud.com/browse/LU-9771
> > Lustre-commit: c1f450270dda ("LU-9971 ptlrpc: use lu_extent in layout_intent")
> > Signed-off-by: Jinshan Xiong <jinshan.xiong at gmail.com>
> > Reviewed-on: https://review.whamcloud.com/29087
> > Reviewed-by: Lai Siyao <lai.siyao at whamcloud.com>
> > Reviewed-by: Bobi Jam <bobijam at hotmail.com>
> > Signed-off-by: James Simmons <jsimmons at infradead.org>
> > ---
> > include/uapi/linux/lustre/lustre_idl.h | 3 +--
> > include/uapi/linux/lustre/lustre_user.h | 2 +-
> > 2 files changed, 2 insertions(+), 3 deletions(-)
>
>
> hi James,
> most of this patch is missing - it seems to have been merged into the
> next patch by mistake.
> I'm walking through patches checking that code still compiles after
> each patch, and this one doesn't.
>
> NeilBrown
Ugh. My tree is on a NFS partition so some times when I apply a patch
while on one node the entire change doesn't show up on another node.
Do you need me to resend the patch?
> >
> > diff --git a/include/uapi/linux/lustre/lustre_idl.h b/include/uapi/linux/lustre/lustre_idl.h
> > index a088f4a..b4f7aec 100644
> > --- a/include/uapi/linux/lustre/lustre_idl.h
> > +++ b/include/uapi/linux/lustre/lustre_idl.h
> > @@ -2825,8 +2825,7 @@ enum {
> > struct layout_intent {
> > __u32 li_opc; /* intent operation for enqueue, read, write etc */
> > __u32 li_flags;
> > - __u64 li_start;
> > - __u64 li_end;
> > + struct lu_extent li_extent;
> > } __packed;
> >
> > /**
> > diff --git a/include/uapi/linux/lustre/lustre_user.h b/include/uapi/linux/lustre/lustre_user.h
> > index 7d9f0d2..d4372d9 100644
> > --- a/include/uapi/linux/lustre/lustre_user.h
> > +++ b/include/uapi/linux/lustre/lustre_user.h
> > @@ -426,7 +426,7 @@ struct lu_extent {
> > __u64 e_end;
> > };
> >
> > -#define DEXT "[ %#llx , %#llx )"
> > +#define DEXT "[%#llx, %#llx)"
> > #define PEXT(ext) (ext)->e_start, (ext)->e_end
> >
> > static inline bool lu_extent_is_overlapped(struct lu_extent *e1,
> > --
> > 1.8.3.1
>
More information about the lustre-devel
mailing list