bug#18406: O_NOATIME patch

2020-09-04 Thread Zev Weiss
On Fri, Sep 04, 2020 at 11:12:07AM CDT, Jim Meyering wrote: On Tue, Sep 1, 2020 at 4:22 AM Paul Eggert wrote: On 9/11/14 1:13 PM, Paul Eggert wrote: > Thanks, but there's no need for that; just have 'grep' complain if the option is > used and O_NOATIME == 0. On looking into this more today, O

bug#18406: O_NOATIME patch

2020-09-04 Thread Jim Meyering
On Tue, Sep 1, 2020 at 4:22 AM Paul Eggert wrote: > On 9/11/14 1:13 PM, Paul Eggert wrote: > > Thanks, but there's no need for that; just have 'grep' complain if the > > option is > > used and O_NOATIME == 0. > > On looking into this more today, O_NOATIME seems to be just a best-effort > thing >

bug#18406: O_NOATIME patch

2020-08-31 Thread Paul Eggert
On 9/11/14 1:13 PM, Paul Eggert wrote: Thanks, but there's no need for that; just have 'grep' complain if the option is used and O_NOATIME == 0. On looking into this more today, O_NOATIME seems to be just a best-effort thing as some GNU/Linux filesystems ignore it, so grep should just join th

bug#18406: O_NOATIME patch

2014-09-11 Thread starlight . 2014q3
Another argument in favor of adding O_NOATIME support to a limited set of utilities (just 'grep' IMO) is that recent 'ext4' file system behavior defaults to a mode where ATIME is updated only once relative to a given MTIME. For O_NOATIME to be of use, one must care enough about ATIME to add 'stric

bug#18406: O_NOATIME patch

2014-09-11 Thread starlight . 2014q3
At 13:13 9/11/2014 -0700, Paul Eggert wrote: >> If the patch is of interest I'm willing >> to improve it by having the feature >> present conditionally on the appearance of >> HAVE_WORKING_O_NOATIME >> in 'config.h'. > >Thanks, but there's no need for that; just >have 'grep' complain if the opt

bug#18406: O_NOATIME patch

2014-09-11 Thread Eric Blake
On 09/11/2014 02:13 PM, Paul Eggert wrote: >> If the patch is of interest I'm willing >> to improve it by having the feature >> present conditionally on the appearance of >> HAVE_WORKING_O_NOATIME >> in 'config.h'. > > Thanks, but there's no need for that; just have 'grep' complain if the > op

bug#18406: O_NOATIME patch

2014-09-11 Thread Paul Eggert
If the patch is of interest I'm willing to improve it by having the feature present conditionally on the appearance of HAVE_WORKING_O_NOATIME in 'config.h'. Thanks, but there's no need for that; just have 'grep' complain if the option is used and O_NOATIME == 0. I'm of two minds about thi

bug#18406: O_NOATIME patch

2014-09-04 Thread starlight . 2014q3
Wrote a quick (but clean) patch to have O_NOATIME applied on file opens. I find this handy for bulk find/grep when I'd prefer not to update atime. If the patch is of interest I'm willing to improve it by having the feature present conditionally on the appearance of HAVE_WORKING_O_NOATIME in 'co