Messages in this thread | | | From | Deepa Dinamani <> | Date | Mon, 2 Dec 2019 15:13:06 -0800 | Subject | Re: [PATCH 7/7] fs: Do not overload update_time |
| |
On Mon, Dec 2, 2019 at 9:48 AM Christoph Hellwig <hch@infradead.org> wrote: > > On Fri, Nov 29, 2019 at 09:30:30PM -0800, Deepa Dinamani wrote: > > - int (*update_time)(struct inode *, struct timespec64 *, int); > > + int (*cb)(struct inode *, struct timespec64 *, int); > > > > - update_time = inode->i_op->update_time ? inode->i_op->update_time : > > + cb = inode->i_op->update_time ? inode->i_op->update_time : > > generic_update_time; > > > > - return update_time(inode, time, flags); > > + return cb(inode, time, flags); > > Just killing the pointless local variable cleans the function op, and > also avoids the expensive indirect call for the common case:
Will update this in v2. Thanks.
-Deepa
| |