Re: [PATCH pynfs] CSESS22: increase the limit above latest knfsd limit

[Calum Mackay <calum.mackay@oracle.com>]

On 20/12/2024 5:23 pm, Jeff Layton wrote:
> With the latest code in Chuck's nfsd-testing branch, the limit on the
> number of fore-channel session slots has been increased to 2048.
> Increase the TOO_MANY_SLOTS variable to 2049.
> 
> Signed-off-by: Jeff Layton <jlayton@kernel.org>
> ---
> Maybe we should increase this to a much higher number instead?  I half
> wonder if we ought to just eliminate this test altogether. It seems sort
> of arbitrary.
> ---

I've applied this as is, thanks Jeff; will have a further think about 
whether the test is still useful.

cheers,
c.


>   nfs4.1/server41tests/st_create_session.py | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/nfs4.1/server41tests/st_create_session.py b/nfs4.1/server41tests/st_create_session.py
> index 43166443ef6f86a6b44de70f3793bd93a330bc9d..740f8bf274c4eeb3ff9da20c0de241a31f80ac26 100644
> --- a/nfs4.1/server41tests/st_create_session.py
> +++ b/nfs4.1/server41tests/st_create_session.py
> @@ -395,7 +395,7 @@ def testMaxreqs(t, env):
>       """
>       # Assuming this is too large for any server; increase if necessary:
>       # but too huge will eat many memory for replay_cache, be careful!
> -    TOO_MANY_SLOTS = 500
> +    TOO_MANY_SLOTS = 2049
>   
>       c = env.c1.new_client(env.testname(t))
>       # CREATE_SESSION with fore_channel = TOO_MANY_SLOTS
> 
> ---
> base-commit: fa786a7217128e6e5c9ef20da9dd6069f5f593e7
> change-id: 20241220-master-2b43693225b0
> 
> Best regards,