Re: [PATCH] check stx_dio_offset_align to determine if DIO is supported

[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]



On Mon, Sep 23, 2024 at 12:01:22PM +0100, Filipe Manana wrote:
> On Mon, Sep 23, 2024 at 10:06 AM Wu Bo <wubo.oduw@xxxxxxxxx> wrote:
> >
> > According to the statx manual, if stx_dio_offset_align is 0, it
> > indicates that Direct I/O (DIO) is not supported on this file.
> 
> There's already a patch for this:
> 
> https://lore.kernel.org/fstests/20240911035333.364961-1-chao@xxxxxxxxxx/
> 
> (but not merged yet)

Thanks Filipe. Yes, this's fixed by Chao Yu. You can check it in
"patches-in-queue" branch, it'll be merged to for-next soon (this
week I hope).

Thanks,
Zorro

> 
> Thanks.
> 
> >
> > Signed-off-by: Wu Bo <wubo.oduw@xxxxxxxxx>
> > ---
> >  src/min_dio_alignment.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/src/min_dio_alignment.c b/src/min_dio_alignment.c
> > index 131f6023..c50ae8c3 100644
> > --- a/src/min_dio_alignment.c
> > +++ b/src/min_dio_alignment.c
> > @@ -28,7 +28,7 @@ static int min_dio_alignment(const char *mntpnt, const char *devname)
> >         fd = open(mntpnt, O_TMPFILE | O_RDWR | O_EXCL, 0600);
> >         if (fd >= 0 &&
> >             xfstests_statx(fd, "", AT_EMPTY_PATH, STATX_DIOALIGN, &stx) == 0 &&
> > -           (stx.stx_mask & STATX_DIOALIGN))
> > +           (stx.stx_mask & STATX_DIOALIGN) && stx.stx_dio_offset_align)
> >                 return stx.stx_dio_offset_align;
> >
> >         /*
> > --
> > 2.34.1
> >
> >
> 





[Index of Archives]     [Linux Filesystems Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux