staging/lustre: Get rid of LIBLUSTRE_CLIENT and its users
authorOleg Drokin <green@linuxhacker.ru>
Thu, 3 Nov 2016 01:25:04 +0000 (21:25 -0400)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 7 Nov 2016 10:36:46 +0000 (11:36 +0100)
This define only made sense in a userspace library client, not in the kernel.

Signed-off-by: Oleg Drokin <green@linuxhacker.ru>
Reviewed-by: James Simmons <jsimmons@infradead.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/lustre/lustre/include/lustre_lib.h
drivers/staging/lustre/lustre/ldlm/ldlm_request.c

index 6b231913ba2ecb807ab8ec6a9ee0da1bbdd1e7e4..27f3148c43442e9918bd87094b9053fdc1846f1e 100644 (file)
@@ -350,8 +350,6 @@ do {                                                                           \
        l_wait_event_exclusive_head(wq, condition, &lwi);       \
 })
 
-#define LIBLUSTRE_CLIENT (0)
-
 /** @} lib */
 
 #endif /* _LUSTRE_LIB_H */
index c5d00d1a50fa2c108286da934d18b93ffdccfa1f..6a96f2c5dd5698e4f0711a2d1a8c805153cf8029 100644 (file)
@@ -475,12 +475,7 @@ int ldlm_cli_enqueue_fini(struct obd_export *exp, struct ptlrpc_request *req,
                                   "client-side enqueue, new policy data");
        }
 
-       if ((*flags) & LDLM_FL_AST_SENT ||
-           /* Cancel extent locks as soon as possible on a liblustre client,
-            * because it cannot handle asynchronous ASTs robustly (see
-            * bug 7311).
-            */
-           (LIBLUSTRE_CLIENT && type == LDLM_EXTENT)) {
+       if ((*flags) & LDLM_FL_AST_SENT) {
                lock_res_and_lock(lock);
                lock->l_flags |= LDLM_FL_CBPENDING |  LDLM_FL_BL_AST;
                unlock_res_and_lock(lock);
@@ -775,14 +770,6 @@ int ldlm_cli_enqueue(struct obd_export *exp, struct ptlrpc_request **reqp,
        body->lock_flags = ldlm_flags_to_wire(*flags);
        body->lock_handle[0] = *lockh;
 
-       /*
-        * Liblustre client doesn't get extent locks, except for O_APPEND case
-        * where [0, OBD_OBJECT_EOF] lock is taken, or truncate, where
-        * [i_size, OBD_OBJECT_EOF] lock is taken.
-        */
-       LASSERT(ergo(LIBLUSTRE_CLIENT, einfo->ei_type != LDLM_EXTENT ||
-                    policy->l_extent.end == OBD_OBJECT_EOF));
-
        if (async) {
                LASSERT(reqp);
                return 0;