* [PATCH] generic/095: update warning whitelist @ 2016-08-01 9:02 Christoph Hellwig 2016-08-02 23:23 ` Dave Chinner 0 siblings, 1 reply; 4+ messages in thread From: Christoph Hellwig @ 2016-08-01 9:02 UTC (permalink / raw) To: fstests; +Cc: xfs The recent direct I/O path refactor changed the function name in the racing read case. Update the test case to handle the new name in addition to the old one. Signed-off-by: Christoph Hellwig <hch@lst.de> --- tests/generic/095 | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/tests/generic/095 b/tests/generic/095 index c656ef1..4754992 100755 --- a/tests/generic/095 +++ b/tests/generic/095 @@ -125,9 +125,11 @@ $FIO_PROG $fio_config >>$seqres.full 2>&1 filter_xfs_dmesg() { local warn1="WARNING:.*fs/xfs/xfs_file\.c:.*xfs_file_dio_aio_write.*" - local warn2="WARNING:.*fs/xfs/xfs_file\.c:.*xfs_file_read_iter.*" + local warn2="WARNING:.*fs/xfs/xfs_file\.c:.*xfs_file_dio_aio_read.*" + local warn3="WARNING:.*fs/xfs/xfs_file\.c:.*xfs_file_read_iter.*" sed -e "s#$warn1#Intentional warnings in xfs_file_dio_aio_write#" \ - -e "s#$warn2#Intentional warnings in xfs_file_read_iter#" + -e "s#$warn2#Intentional warnings in xfs_file_dio_aio_read#" \ + -e "s#$warn3#Intentional warnings in xfs_file_read_iter#" } # umount before checking dmesg in case umount triggers any WARNING or Oops -- 2.1.4 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] generic/095: update warning whitelist 2016-08-01 9:02 [PATCH] generic/095: update warning whitelist Christoph Hellwig @ 2016-08-02 23:23 ` Dave Chinner 2016-08-03 7:39 ` Zorro Lang 0 siblings, 1 reply; 4+ messages in thread From: Dave Chinner @ 2016-08-02 23:23 UTC (permalink / raw) To: Christoph Hellwig; +Cc: fstests, xfs On Mon, Aug 01, 2016 at 11:02:56AM +0200, Christoph Hellwig wrote: > The recent direct I/O path refactor changed the function name in the > racing read case. Update the test case to handle the new name in > addition to the old one. > > Signed-off-by: Christoph Hellwig <hch@lst.de> > --- > tests/generic/095 | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/tests/generic/095 b/tests/generic/095 > index c656ef1..4754992 100755 > --- a/tests/generic/095 > +++ b/tests/generic/095 > @@ -125,9 +125,11 @@ $FIO_PROG $fio_config >>$seqres.full 2>&1 > filter_xfs_dmesg() > { > local warn1="WARNING:.*fs/xfs/xfs_file\.c:.*xfs_file_dio_aio_write.*" > - local warn2="WARNING:.*fs/xfs/xfs_file\.c:.*xfs_file_read_iter.*" > + local warn2="WARNING:.*fs/xfs/xfs_file\.c:.*xfs_file_dio_aio_read.*" > + local warn3="WARNING:.*fs/xfs/xfs_file\.c:.*xfs_file_read_iter.*" > sed -e "s#$warn1#Intentional warnings in xfs_file_dio_aio_write#" \ > - -e "s#$warn2#Intentional warnings in xfs_file_read_iter#" > + -e "s#$warn2#Intentional warnings in xfs_file_dio_aio_read#" \ > + -e "s#$warn3#Intentional warnings in xfs_file_read_iter#" > } Looks fine. Reviewed-by: Dave Chinner <dchinner@redhat.com> -- Dave Chinner david@fromorbit.com ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] generic/095: update warning whitelist 2016-08-02 23:23 ` Dave Chinner @ 2016-08-03 7:39 ` Zorro Lang 2016-08-03 7:55 ` Eryu Guan 0 siblings, 1 reply; 4+ messages in thread From: Zorro Lang @ 2016-08-03 7:39 UTC (permalink / raw) To: Dave Chinner; +Cc: Christoph Hellwig, fstests, xfs On Wed, Aug 03, 2016 at 09:23:05AM +1000, Dave Chinner wrote: > On Mon, Aug 01, 2016 at 11:02:56AM +0200, Christoph Hellwig wrote: > > The recent direct I/O path refactor changed the function name in the > > racing read case. Update the test case to handle the new name in > > addition to the old one. > > > > Signed-off-by: Christoph Hellwig <hch@lst.de> > > --- > > tests/generic/095 | 6 ++++-- > > 1 file changed, 4 insertions(+), 2 deletions(-) > > > > diff --git a/tests/generic/095 b/tests/generic/095 > > index c656ef1..4754992 100755 > > --- a/tests/generic/095 > > +++ b/tests/generic/095 > > @@ -125,9 +125,11 @@ $FIO_PROG $fio_config >>$seqres.full 2>&1 > > filter_xfs_dmesg() > > { > > local warn1="WARNING:.*fs/xfs/xfs_file\.c:.*xfs_file_dio_aio_write.*" > > - local warn2="WARNING:.*fs/xfs/xfs_file\.c:.*xfs_file_read_iter.*" > > + local warn2="WARNING:.*fs/xfs/xfs_file\.c:.*xfs_file_dio_aio_read.*" > > + local warn3="WARNING:.*fs/xfs/xfs_file\.c:.*xfs_file_read_iter.*" > > sed -e "s#$warn1#Intentional warnings in xfs_file_dio_aio_write#" \ > > - -e "s#$warn2#Intentional warnings in xfs_file_read_iter#" > > + -e "s#$warn2#Intentional warnings in xfs_file_dio_aio_read#" \ > > + -e "s#$warn3#Intentional warnings in xfs_file_read_iter#" Hi, 1) xfs_file_dio_aio_read() was splited from xfs_file_read_iter() by below commit: bbc5a74 xfs: split xfs_file_read_iter into buffered and direct I/O helpers 2) xfs_file_read_iter() came from xfs_file_aio_read() as below commit: b4f5d2c xfs: switch to ->read_iter() Some Linux distro (e.g: RHEL-7) still use the old kernel with old function xfs_file_aio_read(), generic/095 always fails on it. So how about filter xfs_file_aio_read too? Thanks, Zorro > > } > > Looks fine. > > Reviewed-by: Dave Chinner <dchinner@redhat.com> > > -- > Dave Chinner > david@fromorbit.com > > _______________________________________________ > xfs mailing list > xfs@oss.sgi.com > http://oss.sgi.com/mailman/listinfo/xfs ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] generic/095: update warning whitelist 2016-08-03 7:39 ` Zorro Lang @ 2016-08-03 7:55 ` Eryu Guan 0 siblings, 0 replies; 4+ messages in thread From: Eryu Guan @ 2016-08-03 7:55 UTC (permalink / raw) To: Zorro Lang; +Cc: Dave Chinner, Christoph Hellwig, fstests, xfs On Wed, Aug 03, 2016 at 03:39:18PM +0800, Zorro Lang wrote: > On Wed, Aug 03, 2016 at 09:23:05AM +1000, Dave Chinner wrote: > > On Mon, Aug 01, 2016 at 11:02:56AM +0200, Christoph Hellwig wrote: > > > The recent direct I/O path refactor changed the function name in the > > > racing read case. Update the test case to handle the new name in > > > addition to the old one. > > > > > > Signed-off-by: Christoph Hellwig <hch@lst.de> > > > --- > > > tests/generic/095 | 6 ++++-- > > > 1 file changed, 4 insertions(+), 2 deletions(-) > > > > > > diff --git a/tests/generic/095 b/tests/generic/095 > > > index c656ef1..4754992 100755 > > > --- a/tests/generic/095 > > > +++ b/tests/generic/095 > > > @@ -125,9 +125,11 @@ $FIO_PROG $fio_config >>$seqres.full 2>&1 > > > filter_xfs_dmesg() > > > { > > > local warn1="WARNING:.*fs/xfs/xfs_file\.c:.*xfs_file_dio_aio_write.*" > > > - local warn2="WARNING:.*fs/xfs/xfs_file\.c:.*xfs_file_read_iter.*" > > > + local warn2="WARNING:.*fs/xfs/xfs_file\.c:.*xfs_file_dio_aio_read.*" > > > + local warn3="WARNING:.*fs/xfs/xfs_file\.c:.*xfs_file_read_iter.*" > > > sed -e "s#$warn1#Intentional warnings in xfs_file_dio_aio_write#" \ > > > - -e "s#$warn2#Intentional warnings in xfs_file_read_iter#" > > > + -e "s#$warn2#Intentional warnings in xfs_file_dio_aio_read#" \ > > > + -e "s#$warn3#Intentional warnings in xfs_file_read_iter#" > > Hi, > > 1) xfs_file_dio_aio_read() was splited from xfs_file_read_iter() by > below commit: > > bbc5a74 xfs: split xfs_file_read_iter into buffered and direct I/O helpers > > 2) xfs_file_read_iter() came from xfs_file_aio_read() as below commit: > > b4f5d2c xfs: switch to ->read_iter() > > Some Linux distro (e.g: RHEL-7) still use the old kernel with old function > xfs_file_aio_read(), generic/095 always fails on it. So how about filter > xfs_file_aio_read too? I think you can just send another (follow-up) patch for review. Eryu ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2016-08-03 7:55 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-08-01 9:02 [PATCH] generic/095: update warning whitelist Christoph Hellwig 2016-08-02 23:23 ` Dave Chinner 2016-08-03 7:39 ` Zorro Lang 2016-08-03 7:55 ` Eryu Guan
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox