All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dave Chinner <david@fromorbit.com>
To: Pavel Shilovsky <pshilovsky@samba.org>
Cc: "fstests@vger.kernel.org" <fstests@vger.kernel.org>,
	Christoph Hellwig <hch@infradead.org>,
	linux-fsdevel <linux-fsdevel@vger.kernel.org>,
	linux-cifs <linux-cifs@vger.kernel.org>,
	"samba-technical@samba.org" <samba-technical@samba.org>,
	David Disseldorp <ddiss@suse.de>,
	Steve French <smfrench@gmail.com>
Subject: Re: [PATCH v3 0/3] CIFS support for XFS test suite
Date: Wed, 3 Sep 2014 21:42:16 +1000	[thread overview]
Message-ID: <20140903114216.GP20518@dastard> (raw)
In-Reply-To: <CAKywueRns+hehrZ3dQ8a8XnaHbd6tNh1BnZAwsTxNTV_w6V6SQ@mail.gmail.com>

On Wed, Sep 03, 2014 at 10:23:47AM +0400, Pavel Shilovsky wrote:
> 2014-08-29 23:42 GMT+04:00 Pavel Shilovsky <pshilovsky@samba.org>:
> > These are patches that adds CIFS support to XFS tests.
> >
> > The first patch changes generic/013 test to use mount options provided for $TEST_DEV for remount. The second patch adds "-cifs" command line argument and CIFS specific variables. The third patch disables tests that need POSIX behavior from filesystem for unix cifs mounts.
> >
> > With these patches applied, most of generic/shared $TEST_DEV tests pass for the recent kernel cifs module.
> >
> > Changes since v2:
> > 1) patch #3 uses separate _require_test_{feature} functions rather than generic _require_test_posix_ext
> > 2) patch #2 adds UDF notes to README
> >
> > Pavel Shilovsky (3):
> >   generic/013: encapsulate remount during cleanup
> >   common: add cifs support
> >   cifs: skip tests that need POSIX support for nounix mounts
> >
> >  README            |  5 +++--
> >  check             |  2 ++
> >  common/config     | 31 +++++++++++++++++++++++++++----
> >  common/rc         | 55 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
> >  tests/generic/005 |  8 ++------
> >  tests/generic/013 |  2 +-
> >  tests/generic/023 |  1 +
> >  tests/generic/024 |  1 +
> >  tests/generic/025 |  1 +
> >  tests/generic/131 |  1 +
> >  10 files changed, 94 insertions(+), 13 deletions(-)
> >
> > --
> > 1.9.1
> >
> 
> Any comments on the series?

Patience. It will be friday before I get to it....

Cheers,

Dave.
-- 
Dave Chinner
david@fromorbit.com

WARNING: multiple messages have this Message-ID (diff)
From: Dave Chinner <david-FqsqvQoI3Ljby3iVrkZq2A@public.gmane.org>
To: Pavel Shilovsky <pshilovsky-eUNUBHrolfbYtjvyW6yDsg@public.gmane.org>
Cc: "fstests-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<fstests-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	Christoph Hellwig <hch-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org>,
	linux-fsdevel
	<linux-fsdevel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	linux-cifs <linux-cifs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	"samba-technical-eUNUBHrolfbYtjvyW6yDsg@public.gmane.org"
	<samba-technical-eUNUBHrolfbYtjvyW6yDsg@public.gmane.org>,
	David Disseldorp <ddiss-l3A5Bk7waGM@public.gmane.org>,
	Steve French <smfrench-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Subject: Re: [PATCH v3 0/3] CIFS support for XFS test suite
Date: Wed, 3 Sep 2014 21:42:16 +1000	[thread overview]
Message-ID: <20140903114216.GP20518@dastard> (raw)
In-Reply-To: <CAKywueRns+hehrZ3dQ8a8XnaHbd6tNh1BnZAwsTxNTV_w6V6SQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>

On Wed, Sep 03, 2014 at 10:23:47AM +0400, Pavel Shilovsky wrote:
> 2014-08-29 23:42 GMT+04:00 Pavel Shilovsky <pshilovsky-eUNUBHrolfbYtjvyW6yDsg@public.gmane.org>:
> > These are patches that adds CIFS support to XFS tests.
> >
> > The first patch changes generic/013 test to use mount options provided for $TEST_DEV for remount. The second patch adds "-cifs" command line argument and CIFS specific variables. The third patch disables tests that need POSIX behavior from filesystem for unix cifs mounts.
> >
> > With these patches applied, most of generic/shared $TEST_DEV tests pass for the recent kernel cifs module.
> >
> > Changes since v2:
> > 1) patch #3 uses separate _require_test_{feature} functions rather than generic _require_test_posix_ext
> > 2) patch #2 adds UDF notes to README
> >
> > Pavel Shilovsky (3):
> >   generic/013: encapsulate remount during cleanup
> >   common: add cifs support
> >   cifs: skip tests that need POSIX support for nounix mounts
> >
> >  README            |  5 +++--
> >  check             |  2 ++
> >  common/config     | 31 +++++++++++++++++++++++++++----
> >  common/rc         | 55 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
> >  tests/generic/005 |  8 ++------
> >  tests/generic/013 |  2 +-
> >  tests/generic/023 |  1 +
> >  tests/generic/024 |  1 +
> >  tests/generic/025 |  1 +
> >  tests/generic/131 |  1 +
> >  10 files changed, 94 insertions(+), 13 deletions(-)
> >
> > --
> > 1.9.1
> >
> 
> Any comments on the series?

Patience. It will be friday before I get to it....

Cheers,

Dave.
-- 
Dave Chinner
david-FqsqvQoI3Ljby3iVrkZq2A@public.gmane.org

  reply	other threads:[~2014-09-03 11:42 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-29 19:42 [PATCH v3 0/3] CIFS support for XFS test suite Pavel Shilovsky
2014-08-29 19:42 ` Pavel Shilovsky
2014-08-29 19:42 ` [PATCH v3 1/3] generic/013: encapsulate remount during cleanup Pavel Shilovsky
2014-08-29 19:42   ` Pavel Shilovsky
2014-08-29 19:42 ` [PATCH v3 2/3] common: add cifs support Pavel Shilovsky
2014-08-29 19:42   ` Pavel Shilovsky
2014-08-29 19:42 ` [PATCH v3 3/3] cifs: skip tests that need POSIX support for nounix mounts Pavel Shilovsky
2014-09-03  6:23 ` [PATCH v3 0/3] CIFS support for XFS test suite Pavel Shilovsky
2014-09-03 11:42   ` Dave Chinner [this message]
2014-09-03 11:42     ` Dave Chinner

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20140903114216.GP20518@dastard \
    --to=david@fromorbit.com \
    --cc=ddiss@suse.de \
    --cc=fstests@vger.kernel.org \
    --cc=hch@infradead.org \
    --cc=linux-cifs@vger.kernel.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=pshilovsky@samba.org \
    --cc=samba-technical@samba.org \
    --cc=smfrench@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.