dlm: remove extra blocking callback check
authorDavid Teigland <teigland@redhat.com>
Mon, 8 Dec 2008 23:14:10 +0000 (17:14 -0600)
committerDavid Teigland <teigland@redhat.com>
Tue, 23 Dec 2008 16:16:32 +0000 (10:16 -0600)
commit03339696314fffb95dafb349b84243358e945ce6
tree4fcdacf43dbe0adaeba79374d54395b5b4f0c035
parentd61e9aac96317a43c192f1faabfa95d4d675b7ce
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>
fs/dlm/ast.c