The handling of the members of struct lookup_intent is obscure. There is a pointless union, void *it_data should be struct ptlrpc_request *it_request. The lock cookies are __u64 rather than struct lustre_handle. We do not use const qualifiers on function parameters where we should.