Re: [PATCH 06/11] CIFS: Respect MaxMpxCount field

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

 



2012/2/29 Steve French <smfrench@xxxxxxxxx>:
> Attempt to summarize after discussions with jra and Chris:
>
> - don't make default case to Samba worse performance than it already is
> - don't "fix" by adding restrictions on echo retries and blocking
> locks when it is not broken (windows allows these, and counts blocking
> locks against the pid not against the session).   For servers which
> allow few mpx, perhaps those for 10 or fewer - if you think they are
> buggy, I don't mind treating echo and blocking locks differently than
> we do now - but echo and blocking locks don't need to be restricted
> the way Pavel suggested to windows and samba and any normal server
> (and there is some risk in restricting them that way).   We could
> probably limit the number of blocking locks (out of resource error,
> returning ENOLOCK) when there are 50 (e.g.) on a process (or 10 if the
> server supports maxmpx of 10, and zero if maxmpx is less than 3 or
> less than 10).
> - for other smb requests (the ones we enforce today) enforce the limit
> the server returns (rather than 50 always)
>

But for SMB2 we need to count both echos and blocking locks. It even
more compicated because the number of credits is dynamic: we need to
increment/decrement the maximum number of parallel blocking locks to
make sure we don't exceed available credits.

-- 
Best regards,
Pavel Shilovsky.
--
To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux