Fix indentation in delegrecall_impl()

Signed-off-by: Malahal Naineni <malahal@us.ibm.com>
Acked-by: Frank S. Filz <ffilzlnx@mindspring.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>
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>
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>
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>