DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws;
  s=dk12062016; d=mindspring.com;
  b=eoUWzwHZPJZVJwYh3HFwl7fMNNP6ifLmsjaLM8IhB3lAju1IRmxeHftwa9nO2RKNh41BoBqLOA+keXJ8YTDX1B1DNbHrSpCVOymEFaCq0YLkTzkMoDA4k194yBR4Cu5QJccrdFhUEvr1V5o+3GtJc+0W+KInzQnNHlgJbRqFR6Wq9jwZ0PDA5/hVe9lf6JrPBKElT2h98Dm+2HADkLorxWLCvO8Uy/Xqwl6y4sJnsYbZFdWOy1kUi6gQODD2w7WzjXiHoGmi82mhAF4y8S2E1KcmZKwsohZ25qMU2Sm5yRwbQMdlU9UoRe1Z9rBDa2u1amVZ12LWKB3deDpD9LdTPg==;
  h=Received:From:To:Cc:References:In-Reply-To:Subject:Date:Message-ID:MIME-Version:Content-Type:Content-Transfer-Encoding:X-Mailer:Content-Language:Thread-Index:X-ELNK-Trace:X-Originating-IP;
From:   "Frank Filz" <ffilzlnx@mindspring.com>
To:     "'J. Bruce Fields'" <bfields@fieldses.org>,
        "'Scott Mayhew'" <smayhew@redhat.com>
Cc:     <jlayton@kernel.org>, <linux-nfs@vger.kernel.org>
References: <20190314211210.7454-1-smayhew@redhat.com> <20190314211210.7454-2-smayhew@redhat.com> <20190315204859.GB13567@fieldses.org>
In-Reply-To: <20190315204859.GB13567@fieldses.org>
Subject: RE: [pynfs PATCH 1/4] nfs4.1: add some reboot tests
Date:   Mon, 18 Mar 2019 07:30:20 -0700
Message-ID: <03cc01d4dd97$1df47ff0$59dd7fd0$@mindspring.com>
MIME-Version: 1.0
Content-Type: text/plain;
        charset="us-ascii"
Content-Transfer-Encoding: 7bit
Content-Language: en-us
Thread-Index: AQLJRflU5gZZ7y6x2nK4BtBk/Gn9owK9wYOoAXwcFbSkBtKUcA==
Sender: linux-nfs-owner@vger.kernel.org
Precedence: bulk

> On Thu, Mar 14, 2019 at 05:12:07PM -0400, Scott Mayhew wrote:
> > +def testRebootWithManyManyManyClients(t, env):
> > +    """Reboot with many many many clients
> > +
> > +    FLAGS: reboot
> > +    CODE: REBT2c
> > +    """
> > +    return doTestRebootWithNClients(t, env, 1000)
> 
> My test server uses a 15 second lease time, mainly just to speed up tests.
That's
> not enough for pynfs to send out reclaims for 1000 clients.
> 
> So I'm wondering whether that's a reasonable test or not.
> 
> On the one hand, we should be able to handle 1000 clients, and a 15 second
> lease is probably unrealistically short.  And maybe we could choose more
patient
> behavior for the server (currently it will wait at most 2 grace periods
while
> reclaims continue to arrive).
> 
> On the other hand, real clients will send their reclaims simultaneously
rather
> than one at a time.  And from a trace it looks like most of the time's
spent
> waiting for pynfs to send the next request rather than waiting for
replies.  So this
> is a bit unusual.
> 
> I'm inclined to drop the "many many many clients" tests.  It's easy enough
for
> someone doing reboot testing to patch the tests if they need to.
> 
> By the way, the longest round trip time I see is the RECLAIM_COMPLETE.
> I assume that's doing a commit to disk.  It looks like there's nothing on
the
> server to prevent processing RECLAIM_COMPLETEs in parallel so as long as
> that's true I suppose we're OK.

How about having the many many many clients tests under a different flag so
they are still available but easy to pick or not pick?

Considering that CID5 with the huge number of client-ids it creates but
doesn't clean up (so they all eventually expire) has caught bugs in Ganesha,
I like the idea of messy big tests being available for QE to run...

Frank