Messages in this thread | | | Date | Wed, 04 Jul 2018 21:07:35 +0800 | From | Huaisheng Ye <> | Subject | Re: [PATCH 3/3] fs/dax: Assigning NULL to gfn of dax_direct_access if useless |
| |
---- On Wed, 04 Jul 2018 19:30:12 +0800 Jan Kara <jack@suse.cz> wrote ---- > On Wed 04-07-18 14:40:58, Huaisheng Ye wrote: > > From: Huaisheng Ye <yehs1@lenovo.com> > > > > Some functions within fs/dax don't need to get gfn from direct_access. > > Assigning NULL to gfn of dax_direct_access is more intuitive and simple > > than offering a useless local variable. > > > > Signed-off-by: Huaisheng Ye <yehs1@lenovo.com> > > I like this. You can add: > > Reviewed-by: Jan Kara <jack@suse.cz> > > for the series. > > Honza > I am so happy you like them, thank you very much.
--- Cheers, Huaisheng
| |