From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-ob0-f194.google.com ([209.85.214.194]:34049 "EHLO mail-ob0-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752340AbcFTSDU (ORCPT ); Mon, 20 Jun 2016 14:03:20 -0400 MIME-Version: 1.0 In-Reply-To: <1466382443-11063-1-git-send-email-deepa.kernel@gmail.com> References: <1466382443-11063-1-git-send-email-deepa.kernel@gmail.com> From: Linus Torvalds Date: Mon, 20 Jun 2016 11:03:01 -0700 Message-ID: Subject: Re: [PATCH v2 00/24] Delete CURRENT_TIME and CURRENT_TIME_SEC macros To: Deepa Dinamani Cc: linux-fsdevel , Linux Kernel Mailing List , Arnd Bergmann , Thomas Gleixner , "Theodore Ts'o" , Al Viro , y2038@lists.linaro.org, "adilger.kernel@dilger.ca" , Adrian Hunter , Anna Schumaker , buchino@cisco.com, ceph-devel , Chris Mason , Changman Lee , Artem Bityutskiy , David Sterba , Alex Elder , Eric Paris , Greg Kroah-Hartman , hiralpat@cisco.com, Ilya Dryomov , Jan Kara , Jaegeuk Kim , Josef Bacik , jejb@linux.vnet.ibm.com, jfs-discussion@lists.sourceforge.net, Joel Becker , John Stultz , linux-audit@redhat.com, linux-btrfs , "linux-ext4@vger.kernel.org" , "Linux F2FS DEV, Mailing List" , linux-mtd , Linux NFS Mailing List , Linux SCSI List , lustre-devel@lists.lustre.org, "Martin K. Petersen" , Mark Fasheh , ocfs2-devel@oss.oracle.com, Paul Moore , Sage Weil , Steve French , Dave Kleikamp , sramars@cisco.com, Trond Myklebust , "Yan, Zheng" Content-Type: text/plain; charset=UTF-8 Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Sun, Jun 19, 2016 at 5:26 PM, Deepa Dinamani wrote: > The series is aimed at getting rid of CURRENT_TIME and CURRENT_TIME_SEC macros. This version now looks ok to me. I do have a comment (or maybe just a RFD) for future work. It does strike me that once we actually change over the inode times to use timespec64, the calling conventions are going to be fairly horrendous on most 32-bit architectures. Gcc handles 8-byte structure returns (on most architectures) by returning them as two 32-bit registers (%edx:%eax on x86). But once it is timespec64, that will no longer be the case, and the calling convention will end up using a pointer to the local stack instead. So for 32-bit code generation, we *may* want to introduce a new model of doing set_inode_time(inode, ATTR_ATIME | ATTR_MTIME); which basically just does inode->i_atime = inode->i_mtime = current_time(inode); but with a much easier calling convention on 32-bit architectures. But that is entirely orthogonal to this patch-set, and should be seen as a separate issue. And maybe it doesn't end up helping anyway, but I do think those "simple" direct assignments will really generate pretty disgusting code on 32-bit architectures. That whole inode->i_atime = inode->i_mtime = CURRENT_TIME; model really made a lot more sense back in the ancient days when inode times were just simply 32-bit seconds (not even timeval structures). Linus