diff options
author | David Teigland <teigland@redhat.com> | 2008-12-08 17:14:10 -0600 |
---|---|---|
committer | David Teigland <teigland@redhat.com> | 2008-12-23 10:16:32 -0600 |
commit | 03339696314fffb95dafb349b84243358e945ce6 (patch) | |
tree | 4fcdacf43dbe0adaeba79374d54395b5b4f0c035 /fs/dlm/user.c | |
parent | d61e9aac96317a43c192f1faabfa95d4d675b7ce (diff) | |
download | op-kernel-dev-03339696314fffb95dafb349b84243358e945ce6.zip op-kernel-dev-03339696314fffb95dafb349b84243358e945ce6.tar.gz |
dlm: remove extra blocking callback check
Just before delivering a blocking callback (bast), the dlm_astd
thread checks again that the granted mode of the lkb actually
blocks the mode requested by the bast. The idea behind this was
originally that the granted mode may have changed since the bast
was queued, making the callback now unnecessary. Reasons for
removing this extra check are:
- dlm_astd doesn't lock the rsb before reading the lkb grmode, so
it's not technically safe (this removes the long standing FIXME)
- after running some tests, it doesn't appear the check ever actually
eliminates a bast
- delivering an unnecessary blocking callback isn't a bad thing and
can happen anyway
Signed-off-by: David Teigland <teigland@redhat.com>
Diffstat (limited to 'fs/dlm/user.c')
0 files changed, 0 insertions, 0 deletions