Staging: lustre: Use static inline functions instead of macros

PATCH Superseded Ksenija Stanojevic 2 participants 6 messages Oct 17, 2015 Staging

Share & tools

Ksenija Stanojevic

author

2015-10-17 07:56 UTC #0 lore raw

v2 cover Staging: lustre: Use static inline functions instead of macros

Convert macros class_export_lock_get and class_export_lock_put into static inline functions and also remove some unused macros.

Signed-off-by Ksenija Stanojevic<[email protected]>

Changes in v2:
- be more specific in commit message

Ksenija Stanojevic (4):
Staging: lustre: include: Move function prototypes
Staging: lustre: include: Convert macro class_export_lock_get into
static inline function
Staging: lustre: include: Convert macro class_export_lock_put into
static inline function
Staging: lustre: include : Remove unused macros

 drivers/staging/lustre/lustre/include/obd_class.h | 90 ++++++++++-------------
 1 file changed, 38 insertions(+), 52 deletions(-)

Signature · 1 line

1.9.1

Ksenija Stanojevic

author

2015-10-17 07:58 UTC #1 lore raw

PATCH v2 1/4 Staging: lustre: include: Move function prototypes

Functions:
struct obd_export *class_export_get(struct obd_export *exp); void class_export_put(struct obd_export *exp)
are being used in macros that are converted into static inline functions, therefore move function prototypes to avoid build error.

Signed-off-by Ksenija Stanojevic<[email protected]>

---
Changes in v2:
- be more specific in connit message

 drivers/staging/lustre/lustre/include/obd_class.h | 37 ++++++++++++-----------
 1 file changed, 19 insertions(+), 18 deletions(-)

drivers/staging/lustre/lustre/include/obd_class.h +19 ~~-18~~

index 82f1a26..4c2076e 100644
@@ -189,6 +189,25 @@ extern void (*class_export_dump_hook)(struct obd_export *);
 
 #endif
 
+/* genops.c */
+struct obd_export *class_export_get(struct obd_export *exp);
+void class_export_put(struct obd_export *exp);
+struct obd_export *class_new_export(struct obd_device *obddev,
+				struct obd_uuid *cluuid);
+void class_unlink_export(struct obd_export *exp);
+
+struct obd_import *class_import_get(struct obd_import *);
+void class_import_put(struct obd_import *);
+struct obd_import *class_new_import(struct obd_device *obd);
+void class_destroy_import(struct obd_import *exp);
+
+void class_put_type(struct obd_type *type);
+int class_connect(struct lustre_handle *conn, struct obd_device *obd,
+		struct obd_uuid *cluuid);
+int class_disconnect(struct obd_export *exp);
+void class_fail_export(struct obd_export *exp);
+int class_manual_cleanup(struct obd_device *obd);
+
 static inline void class_export_rpc_inc(struct obd_export *exp)
 {
 	atomic_inc(&(exp)->exp_rpc_count);

@@ -240,24 +259,6 @@ static inline void class_export_rpc_dec(struct obd_export *exp)
 	class_export_put(exp);					  \
 })
 
-/* genops.c */
-struct obd_export *class_export_get(struct obd_export *exp);
-void class_export_put(struct obd_export *exp);
-struct obd_export *class_new_export(struct obd_device *obddev,
-				    struct obd_uuid *cluuid);
-void class_unlink_export(struct obd_export *exp);
-
-struct obd_import *class_import_get(struct obd_import *);
-void class_import_put(struct obd_import *);
-struct obd_import *class_new_import(struct obd_device *obd);
-void class_destroy_import(struct obd_import *exp);
-
-void class_put_type(struct obd_type *type);
-int class_connect(struct lustre_handle *conn, struct obd_device *obd,
-		  struct obd_uuid *cluuid);
-int class_disconnect(struct obd_export *exp);
-void class_fail_export(struct obd_export *exp);
-int class_manual_cleanup(struct obd_device *obd);
 static inline enum obd_option exp_flags_from_obd(struct obd_device *obd)
 {
 	return ((obd->obd_fail ? OBD_OPT_FAILOVER : 0) |

Signature · 1 line

1.9.1

Greg KH

maintainer

2015-10-17 16:05 UTC #2 lore raw

Re: [Outreachy kernel] [PATCH v2 1/4] Staging: lustre: include: Move function prototypes

Quoted 4 lines · Ksenija Stanojevic

Functions:
struct obd_export *class_export_get(struct obd_export *exp); void class_export_put(struct obd_export *exp)
are being used in macros that are converted into static inline

Trailing whitespace :(

Quoted 2 lines

functions, therefore move function prototypes to avoid build error.

The build error will happen in later patches, please be specific about this, as written it sounds like there is a build error now.

Quoted 1 line

Signed-off-by Ksenija Stanojevic<[email protected]>

We need a space between the '<' character and your your name :(

Can you redo this series again please?

thanks,

greg k-h

Ksenija Stanojevic

author

2015-10-17 07:59 UTC #3 lore raw

PATCH v2 2/4 Staging: lustre: include: Convert macro class_export_lock_get into static inline function

In reply to Ksenija Stanojevic

Static inline functions are preferred over macros. This change is safe because the types of arguments at all the call sites are same.

Signed-off-by Ksenija Stanojevic<[email protected]>

---
Changes in v2:
- nothing

 drivers/staging/lustre/lustre/include/obd_class.h | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

drivers/staging/lustre/lustre/include/obd_class.h +9 -8

index 4c2076e..5a57876 100644
@@ -223,14 +223,15 @@ static inline void class_export_rpc_dec(struct obd_export *exp)
 	       (exp), atomic_read(&(exp)->exp_rpc_count));
 }
 
-#define class_export_lock_get(exp, lock)				\
-({								      \
-	atomic_inc(&(exp)->exp_locks_count);			\
-	__class_export_add_lock_ref(exp, lock);			 \
-	CDEBUG(D_INFO, "lock GETting export %p : new locks_count %d\n", \
-	       (exp), atomic_read(&(exp)->exp_locks_count));	\
-	class_export_get(exp);					  \
-})
+static inline struct obd_export *class_export_lock_get(struct obd_export *exp,
+						       struct ldlm_lock *lock)
+{
+	atomic_inc(&(exp)->exp_locks_count);
+	__class_export_add_lock_ref(exp, lock);
+	CDEBUG(D_INFO, "lock GETting export %p : new locks_count %d\n",
+	       (exp), atomic_read(&(exp)->exp_locks_count));
+	return class_export_get(exp);
+}
 
 #define class_export_lock_put(exp, lock)				\
 ({								      \

Signature · 1 line

1.9.1

Ksenija Stanojevic

author

2015-10-17 08:00 UTC #4 lore raw

PATCH v2 3/4 Staging: lustre: include: Convert macro class_export_lock_put into static inline function

In reply to Ksenija Stanojevic

Static inline functions are preferred over macros. This change is safe because the types of arguments at all the call sites are same.

Signed-off-by Ksenija Stanojevic<[email protected]>

---
Changes in v2:
- nothing

 drivers/staging/lustre/lustre/include/obd_class.h | 19 ++++++++++---------
 1 file changed, 10 insertions(+), 9 deletions(-)

drivers/staging/lustre/lustre/include/obd_class.h +10 -9

index 5a57876..4871b6a 100644
@@ -233,15 +233,16 @@ static inline struct obd_export *class_export_lock_get(struct obd_export *exp,
 	return class_export_get(exp);
 }
 
-#define class_export_lock_put(exp, lock)				\
-({								      \
-	LASSERT_ATOMIC_POS(&exp->exp_locks_count);		      \
-	atomic_dec(&(exp)->exp_locks_count);			\
-	__class_export_del_lock_ref(exp, lock);			 \
-	CDEBUG(D_INFO, "lock PUTting export %p : new locks_count %d\n", \
-	       (exp), atomic_read(&(exp)->exp_locks_count));	\
-	class_export_put(exp);					  \
-})
+static inline void class_export_lock_put(struct obd_export *exp,
+					 struct ldlm_lock *lock)
+{
+	LASSERT_ATOMIC_POS(&exp->exp_locks_count);
+	atomic_dec(&(exp)->exp_locks_count);
+	__class_export_del_lock_ref(exp, lock);
+	CDEBUG(D_INFO, "lock PUTting export %p : new locks_count %d\n",
+	       (exp), atomic_read(&(exp)->exp_locks_count));
+	class_export_put(exp);
+}
 
 #define class_export_cb_get(exp)					\
 ({								      \

Signature · 1 line

1.9.1

Ksenija Stanojevic

author

2015-10-17 08:01 UTC #5 lore raw

PATCH v2 4/4 Staging: lustre: include : Remove unused macros

In reply to Ksenija Stanojevic

These macro are not used anymore, therefore remove them.

Signed-off-by Ksenija Stanojevic<[email protected]>

---
Changes in v2:
- nothing

 drivers/staging/lustre/lustre/include/obd_class.h | 17 -----------------
 1 file changed, 17 deletions(-)

drivers/staging/lustre/lustre/include/obd_class.h +0 ~~-17~~

index 4871b6a..53c255c 100644
@@ -244,23 +244,6 @@ static inline void class_export_lock_put(struct obd_export *exp,
 	class_export_put(exp);
 }
 
-#define class_export_cb_get(exp)					\
-({								      \
-	atomic_inc(&(exp)->exp_cb_count);			   \
-	CDEBUG(D_INFO, "callback GETting export %p : new cb_count %d\n",\
-	       (exp), atomic_read(&(exp)->exp_cb_count));	   \
-	class_export_get(exp);					  \
-})
-
-#define class_export_cb_put(exp)					\
-({								      \
-	LASSERT_ATOMIC_POS(&exp->exp_cb_count);			 \
-	atomic_dec(&(exp)->exp_cb_count);			   \
-	CDEBUG(D_INFO, "callback PUTting export %p : new cb_count %d\n",\
-	       (exp), atomic_read(&(exp)->exp_cb_count));	   \
-	class_export_put(exp);					  \
-})
-
 static inline enum obd_option exp_flags_from_obd(struct obd_device *obd)
 {
 	return ((obd->obd_fail ? OBD_OPT_FAILOVER : 0) |

Signature · 1 line

1.9.1