Ralph Böhme via samba-technical
2018-05-02 10:05:30 UTC
Hi!
Stumpled across this one while looking at the MSG_SMB_UNLOCK broadcast storm
issue.
MSG_SMB_UNLOCK should be send to smbd that are waiting on blocked
byte-range-locks when a lock holder died.
In smbd_cleanupd_unlock() we do this twice: once via a broadcast and
then again via brl_revalidate() to processes that are actually recorded
in brlock.tdb.
As brl_revalidate() should already take care of signaling anyone who
would be interested in the message, there's no need to broadcast.
Attached patch removes the broadcast.
Please review carefully and push if ok. I'm still suspecting that I'm missing a
subtlety around MSG_SMB_UNLOCK...
Thanks!
-slow
Stumpled across this one while looking at the MSG_SMB_UNLOCK broadcast storm
issue.
MSG_SMB_UNLOCK should be send to smbd that are waiting on blocked
byte-range-locks when a lock holder died.
In smbd_cleanupd_unlock() we do this twice: once via a broadcast and
then again via brl_revalidate() to processes that are actually recorded
in brlock.tdb.
As brl_revalidate() should already take care of signaling anyone who
would be interested in the message, there's no need to broadcast.
Attached patch removes the broadcast.
Please review carefully and push if ok. I'm still suspecting that I'm missing a
subtlety around MSG_SMB_UNLOCK...
Thanks!
-slow
--
Ralph Boehme, Samba Team https://samba.org/
Samba Developer, SerNet GmbH https://sernet.de/en/samba/
GPG Key Fingerprint: FAE2 C608 8A24 2520 51C5
59E4 AA1E 9B71 2639 9E46
Ralph Boehme, Samba Team https://samba.org/
Samba Developer, SerNet GmbH https://sernet.de/en/samba/
GPG Key Fingerprint: FAE2 C608 8A24 2520 51C5
59E4 AA1E 9B71 2639 9E46