Fix indentation in delegrecall_impl()

Signed-off-by: Malahal Naineni <malahal@us.ibm.com>
Acked-by: Frank S. Filz <ffilzlnx@mindspring.com>
Acked-by: Venkateswararao Jujjuri (JV) <jvrao@linux.vnet.ibm.com>
Acked-by: Matt Benjamin <matt@linuxbox.com>
Separate delegations from range lock lists

Delegations now use a new "struct deleg_data" instead of
state_lock_entry_t.  Since delegations always have an associated
state_t, struct deleg_data is just an extension of state_t, so they
don't exist in any list except "deleg_list" in cache inode entry to
just speed up list traversal looking for delegations.

Signed-off-by: Malahal Naineni <malahal@us.ibm.com>
Acked-by: Frank S. Filz <ffilzlnx@mindspring.com>
Acked-by: Venkateswararao Jujjuri (JV) <jvrao@linux.vnet.ibm.com>
Acked-by: Matt Benjamin <matt@linuxbox.com>
Remove unused sle_export_id field from state_lock_entry_t

Delegations use export_id, but now they use struct deleg_data rather
than state_lock_entry_t. So remove this unused field.

Signed-off-by: Malahal Naineni <malahal@us.ibm.com>
Acked-by: Frank S. Filz <ffilzlnx@mindspring.com>
Acked-by: Venkateswararao Jujjuri (JV) <jvrao@linux.vnet.ibm.com>
Acked-by: Matt Benjamin <matt@linuxbox.com>
Move delegation functions from state_lock.c to state_deleg.c

Moved remove_deleg_data, acquire_lease_lock, release_lease_lock
functions from state_lock.c to state_deleg.c. No functional change.

Signed-off-by: Malahal Naineni <malahal@us.ibm.com>
Acked-by: Frank S. Filz <ffilzlnx@mindspring.com>
Acked-by: Venkateswararao Jujjuri (JV) <jvrao@linux.vnet.ibm.com>
Acked-by: Matt Benjamin <matt@linuxbox.com>
Remove code that handles delegation state being there without its data object.

Delegation data object is just an extension of the delegation state, so
it should be allocated and freed along with its delegation state. One
should not exist without the other, so removed some obsolete code, and
now log LogWarn message if delegation state exists without its data
object!

Signed-off-by: Malahal Naineni <malahal@us.ibm.com>
Acked-by: Frank S. Filz <ffilzlnx@mindspring.com>
Acked-by: Venkateswararao Jujjuri (JV) <jvrao@linux.vnet.ibm.com>
Acked-by: Matt Benjamin <matt@linuxbox.com>
Remove state_lock_locked and state_unlock_locked

They are only called from state_lock and state_unlock now, so there is
no reason to keep them anywhere.

Signed-off-by: Malahal Naineni <malahal@us.ibm.com>
Acked-by: Matt Benjamin <matt@linuxbox.com>
Acked-by: Venkateswararao Jujjuri (JV) <jvrao@linux.vnet.ibm.com>