summaryrefslogtreecommitdiffstats
path: root/utils.h
diff options
context:
space:
mode:
authorMark Farrugia <mark.farrugia@fiberdyne.com.au>2018-10-25 15:57:34 +1100
committerMark Farrugia <mark.farrugia@fiberdyne.com.au>2018-10-26 17:27:49 +1100
commite734ca6ed2bb99343fd32c369a7eb3679447b115 (patch)
tree9e83db5e2b0feedad8870051650d879fda707bd6 /utils.h
parent18007ca500654b48011e8a8cf96c47b1a5aa3be7 (diff)
Optimize CHK_ macros
No need to be using kasprintf in a debug macro! Signed-off-by: Mark Farrugia <mark.farrugia@fiberdyne.com.au>
Diffstat (limited to 'utils.h')
-rw-r--r--utils.h23
1 files changed, 8 insertions, 15 deletions
diff --git a/utils.h b/utils.h
index 844ee55..0931e82 100644
--- a/utils.h
+++ b/utils.h
@@ -10,39 +10,32 @@
#ifndef __SOUND_AVIRT_UTILS_H
#define __SOUND_AVIRT_UTILS_H
-#include <linux/slab.h>
-
-#define PRINT_ERR(errno, errmsg) \
- pr_err("[%s]:[ERRNO:%d]: %s \n", __func__, errno, (errmsg));
-
#define CHK_ERR(errno) \
do { \
if ((errno) < 0) \
return (errno); \
} while (0)
-#define CHK_ERR_V(errno, errmsg, ...) \
+#define CHK_ERR_V(errno, fmt, args...) \
do { \
if ((errno) < 0) { \
- PRINT_ERR((errno), (errmsg), ##__VA_ARGS__) \
+ DERROR(D_LOGNAME, (fmt), ##args) \
return (errno); \
} \
} while (0)
-#define CHK_NULL(x, errno) \
+#define CHK_NULL(x) \
do { \
if (!(x)) \
- return errno; \
+ return -EFAULT; \
} while (0)
-#define CHK_NULL_V(x, errno, errmsg, ...) \
+#define CHK_NULL_V(x, fmt, args...) \
do { \
if (!(x)) { \
- char *errmsg_done = \
- kasprintf(GFP_KERNEL, errmsg, ##__VA_ARGS__); \
- PRINT_ERR(EFAULT, errmsg_done); \
- kfree(errmsg_done); \
- return errno; \
+ DERROR(D_LOGNAME, "[ERRNO: %d] " fmt, -EFAULT, \
+ ##args); \
+ return -EFAULT; \
} \
} while (0)