]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
staging: lustre: libcfs: discard KLASSERT()
authorNeilBrown <neilb@suse.com>
Wed, 13 Dec 2017 03:15:55 +0000 (14:15 +1100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 13 Dec 2017 11:32:13 +0000 (12:32 +0100)
This appears to be intended for assertions that only make
sense in the kernel, but as this code is now kernel-only,
it doesn't make sense any more.

Signed-off-by: NeilBrown <neilb@suse.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/lustre/include/linux/libcfs/libcfs_private.h
drivers/staging/lustre/lustre/obdclass/cl_page.c

index d32ce68e6c5804d8aa88eb194c3a7b7d1ccbc8cf..31403667be6b65ee8d488429aaaaa6db03a53331 100644 (file)
@@ -74,8 +74,6 @@ do {                                                                  \
 # define LINVRNT(exp) ((void)sizeof !!(exp))
 #endif
 
-#define KLASSERT(e) LASSERT(e)
-
 void __noreturn lbug_with_loc(struct libcfs_debug_msg_data *msg);
 
 #define LBUG()                                                   \
index 7f65439f9b95cfdf1dad176081b24d7d94bd14cb..d3b25667bc3a63bddaf2701a948db53790a2dc97 100644 (file)
@@ -202,7 +202,7 @@ struct cl_page *cl_page_find(const struct lu_env *env,
                 * vmpage lock is used to protect the child/parent
                 * relationship
                 */
-               KLASSERT(PageLocked(vmpage));
+               LASSERT(PageLocked(vmpage));
                /*
                 * cl_vmpage_page() can be called here without any locks as
                 *
@@ -340,7 +340,7 @@ struct cl_page *cl_vmpage_page(struct page *vmpage, struct cl_object *obj)
 {
        struct cl_page *page;
 
-       KLASSERT(PageLocked(vmpage));
+       LASSERT(PageLocked(vmpage));
 
        /*
         * NOTE: absence of races and liveness of data are guaranteed by page