From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx2.suse.de ([195.135.220.15]:51486 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756488AbdAJKox (ORCPT ); Tue, 10 Jan 2017 05:44:53 -0500 Date: Tue, 10 Jan 2017 11:44:51 +0100 From: Jan Kara To: Fabian Frederick Cc: Jan Kara , linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org Subject: Re: [PATCH 06/12 linux-next] udf: add udf_ajust_time Message-ID: <20170110104451.GF4991@quack2.suse.cz> References: <1483736037-25111-1-git-send-email-fabf@skynet.be> <1483736037-25111-7-git-send-email-fabf@skynet.be> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1483736037-25111-7-git-send-email-fabf@skynet.be> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Fri 06-01-17 21:53:54, Fabian Frederick wrote: > Merge timespec affectations > > Signed-off-by: Fabian Frederick Thanks. I've just renamed the function to udf_trim_crtime() which better describes what it does. Honza > --- > fs/udf/inode.c | 25 +++++++++++-------------- > 1 file changed, 11 insertions(+), 14 deletions(-) > > diff --git a/fs/udf/inode.c b/fs/udf/inode.c > index b5893f4..f563e97 100644 > --- a/fs/udf/inode.c > +++ b/fs/udf/inode.c > @@ -1612,6 +1612,14 @@ static int udf_sync_inode(struct inode *inode) > return udf_update_inode(inode, 1); > } > > +static void udf_adjust_time(struct udf_inode_info *iinfo, struct timespec time) > +{ > + if (iinfo->i_crtime.tv_sec > time.tv_sec || > + (iinfo->i_crtime.tv_sec == time.tv_sec && > + iinfo->i_crtime.tv_nsec > time.tv_nsec)) > + iinfo->i_crtime = time; > +} > + > static int udf_update_inode(struct inode *inode, int do_sync) > { > struct buffer_head *bh = NULL; > @@ -1738,20 +1746,9 @@ static int udf_update_inode(struct inode *inode, int do_sync) > efe->objectSize = cpu_to_le64(inode->i_size); > efe->logicalBlocksRecorded = cpu_to_le64(lb_recorded); > > - if (iinfo->i_crtime.tv_sec > inode->i_atime.tv_sec || > - (iinfo->i_crtime.tv_sec == inode->i_atime.tv_sec && > - iinfo->i_crtime.tv_nsec > inode->i_atime.tv_nsec)) > - iinfo->i_crtime = inode->i_atime; > - > - if (iinfo->i_crtime.tv_sec > inode->i_mtime.tv_sec || > - (iinfo->i_crtime.tv_sec == inode->i_mtime.tv_sec && > - iinfo->i_crtime.tv_nsec > inode->i_mtime.tv_nsec)) > - iinfo->i_crtime = inode->i_mtime; > - > - if (iinfo->i_crtime.tv_sec > inode->i_ctime.tv_sec || > - (iinfo->i_crtime.tv_sec == inode->i_ctime.tv_sec && > - iinfo->i_crtime.tv_nsec > inode->i_ctime.tv_nsec)) > - iinfo->i_crtime = inode->i_ctime; > + udf_adjust_time(iinfo, inode->i_atime); > + udf_adjust_time(iinfo, inode->i_mtime); > + udf_adjust_time(iinfo, inode->i_ctime); > > udf_time_to_disk_stamp(&efe->accessTime, inode->i_atime); > udf_time_to_disk_stamp(&efe->modificationTime, inode->i_mtime); > -- > 2.7.4 > > -- Jan Kara SUSE Labs, CR