vhost: fix code style
[dpdk.git] / lib / librte_vhost / vhost-net-cdev.c
index 91ff0d8..57c76cb 100644 (file)
 
 #include "vhost-net-cdev.h"
 
-#define FUSE_OPT_DUMMY         "\0\0"
-#define FUSE_OPT_FORE          "-f\0\0"
-#define FUSE_OPT_NOMULTI       "-s\0\0"
+#define FUSE_OPT_DUMMY "\0\0"
+#define FUSE_OPT_FORE  "-f\0\0"
+#define FUSE_OPT_NOMULTI "-s\0\0"
 
-static const uint32_t  default_major = 231;
-static const uint32_t  default_minor = 1;
-static const char      cuse_device_name[]      = "/dev/cuse";
-static const char      default_cdev[] = "vhost-net";
+static const uint32_t default_major = 231;
+static const uint32_t default_minor = 1;
+static const char cuse_device_name[] = "/dev/cuse";
+static const char default_cdev[] = "vhost-net";
 
-static struct fuse_session                     *session;
-static struct vhost_net_device_ops     const *ops;
+static struct fuse_session *session;
+static struct vhost_net_device_ops const *ops;
 
 /*
- * Returns vhost_device_ctx from given fuse_req_t. The index is populated later when
- * the device is added to the device linked list.
+ * Returns vhost_device_ctx from given fuse_req_t. The index is populated later
+ * when the device is added to the device linked list.
  */
 static struct vhost_device_ctx
 fuse_req_to_vhost_ctx(fuse_req_t req, struct fuse_file_info *fi)
@@ -75,7 +75,8 @@ fuse_req_to_vhost_ctx(fuse_req_t req, struct fuse_file_info *fi)
 }
 
 /*
- * When the device is created in QEMU it gets initialised here and added to the device linked list.
+ * When the device is created in QEMU it gets initialised here and
+ * added to the device linked list.
  */
 static void
 vhost_net_open(fuse_req_t req, struct fuse_file_info *fi)
@@ -91,7 +92,8 @@ vhost_net_open(fuse_req_t req, struct fuse_file_info *fi)
 
        fi->fh = err;
 
-       RTE_LOG(INFO, VHOST_CONFIG, "(%"PRIu64") Device configuration started\n", fi->fh);
+       RTE_LOG(INFO, VHOST_CONFIG,
+               "(%"PRIu64") Device configuration started\n", fi->fh);
        fuse_reply_open(req, fi);
 }
 
@@ -113,8 +115,8 @@ vhost_net_release(fuse_req_t req, struct fuse_file_info *fi)
  * Boilerplate code for CUSE IOCTL
  * Implicit arguments: ctx, req, result.
  */
-#define VHOST_IOCTL(func) do {                 \
-       result = (func)(ctx);                   \
+#define VHOST_IOCTL(func) do { \
+       result = (func)(ctx);   \
        fuse_reply_ioctl(req, result, NULL, 0); \
 } while (0)
 
@@ -122,57 +124,58 @@ vhost_net_release(fuse_req_t req, struct fuse_file_info *fi)
  * Boilerplate IOCTL RETRY
  * Implicit arguments: req.
  */
-#define VHOST_IOCTL_RETRY(size_r, size_w) do {         \
-       struct iovec iov_r = { arg, (size_r) };         \
-       struct iovec iov_w = { arg, (size_w) };         \
-       fuse_reply_ioctl_retry(req, &iov_r, (size_r)?1:0, &iov_w, (size_w)?1:0);        \
+#define VHOST_IOCTL_RETRY(size_r, size_w) do { \
+       struct iovec iov_r = { arg, (size_r) }; \
+       struct iovec iov_w = { arg, (size_w) }; \
+       fuse_reply_ioctl_retry(req, &iov_r,     \
+               (size_r) ? 1 : 0, &iov_w, (size_w) ? 1 : 0);\
 } while (0)
 
 /*
  * Boilerplate code for CUSE Read IOCTL
  * Implicit arguments: ctx, req, result, in_bufsz, in_buf.
  */
-#define VHOST_IOCTL_R(type, var, func) do {            \
-       if (!in_bufsz) {                                \
-               VHOST_IOCTL_RETRY(sizeof(type), 0);     \
-       } else {                                        \
-               (var) = *(const type*) in_buf;          \
-               result = func(ctx, &(var));             \
-               fuse_reply_ioctl(req, result, NULL, 0); \
-       }                                               \
+#define VHOST_IOCTL_R(type, var, func) do {    \
+       if (!in_bufsz) {        \
+               VHOST_IOCTL_RETRY(sizeof(type), 0);\
+       } else {        \
+               (var) = *(const type*)in_buf;   \
+               result = func(ctx, &(var));     \
+               fuse_reply_ioctl(req, result, NULL, 0);\
+       }       \
 } while (0)
 
 /*
- *     Boilerplate code for CUSE Write IOCTL
+ * Boilerplate code for CUSE Write IOCTL
  * Implicit arguments: ctx, req, result, out_bufsz.
  */
-#define        VHOST_IOCTL_W(type, var, func) do {             \
-       if (!out_bufsz) {                               \
-               VHOST_IOCTL_RETRY(0, sizeof(type));     \
-       } else {                                        \
-               result = (func)(ctx, &(var));           \
-               fuse_reply_ioctl(req, result, &(var), sizeof(type));    \
-       }                                                               \
+#define VHOST_IOCTL_W(type, var, func) do {    \
+       if (!out_bufsz) {       \
+               VHOST_IOCTL_RETRY(0, sizeof(type));\
+       } else {        \
+               result = (func)(ctx, &(var));\
+               fuse_reply_ioctl(req, result, &(var), sizeof(type));\
+       } \
 } while (0)
 
 /*
  * Boilerplate code for CUSE Read/Write IOCTL
  * Implicit arguments: ctx, req, result, in_bufsz, in_buf.
  */
-#define VHOST_IOCTL_RW(type1, var1, type2, var2, func) do {            \
-       if (!in_bufsz) {                                                \
-               VHOST_IOCTL_RETRY(sizeof(type1), sizeof(type2));        \
-       } else {                                                        \
-               (var1) = *(const type1*) (in_buf);                      \
-               result = (func)(ctx, (var1), &(var2));                  \
-               fuse_reply_ioctl(req, result, &(var2), sizeof(type2));  \
-       }                                                               \
+#define VHOST_IOCTL_RW(type1, var1, type2, var2, func) do {    \
+       if (!in_bufsz) {        \
+               VHOST_IOCTL_RETRY(sizeof(type1), sizeof(type2));\
+       } else {        \
+               (var1) = *(const type1*) (in_buf);      \
+               result = (func)(ctx, (var1), &(var2));  \
+               fuse_reply_ioctl(req, result, &(var2), sizeof(type2));\
+       }       \
 } while (0)
 
 /*
- * The IOCTLs are handled using CUSE/FUSE in userspace. Depending on
- * the type of IOCTL a buffer is requested to read or to write. This
- * request is handled by FUSE and the buffer is then given to CUSE.
+ * The IOCTLs are handled using CUSE/FUSE in userspace. Depending on the type
+ * of IOCTL a buffer is requested to read or to write. This request is handled
+ * by FUSE and the buffer is then given to CUSE.
  */
 static void
 vhost_net_ioctl(fuse_req_t req, int cmd, void *arg,
@@ -189,33 +192,39 @@ vhost_net_ioctl(fuse_req_t req, int cmd, void *arg,
 
        switch (cmd) {
        case VHOST_NET_SET_BACKEND:
-               LOG_DEBUG(VHOST_CONFIG, "(%"PRIu64") IOCTL: VHOST_NET_SET_BACKEND\n", ctx.fh);
+               LOG_DEBUG(VHOST_CONFIG,
+                       "(%"PRIu64") IOCTL: VHOST_NET_SET_BACKEND\n", ctx.fh);
                VHOST_IOCTL_R(struct vhost_vring_file, file, ops->set_backend);
                break;
 
        case VHOST_GET_FEATURES:
-               LOG_DEBUG(VHOST_CONFIG, "(%"PRIu64") IOCTL: VHOST_GET_FEATURES\n", ctx.fh);
+               LOG_DEBUG(VHOST_CONFIG,
+                       "(%"PRIu64") IOCTL: VHOST_GET_FEATURES\n", ctx.fh);
                VHOST_IOCTL_W(uint64_t, features, ops->get_features);
                break;
 
        case VHOST_SET_FEATURES:
-               LOG_DEBUG(VHOST_CONFIG, "(%"PRIu64") IOCTL: VHOST_SET_FEATURES\n", ctx.fh);
+               LOG_DEBUG(VHOST_CONFIG,
+                       "(%"PRIu64") IOCTL: VHOST_SET_FEATURES\n", ctx.fh);
                VHOST_IOCTL_R(uint64_t, features, ops->set_features);
                break;
 
        case VHOST_RESET_OWNER:
-               LOG_DEBUG(VHOST_CONFIG, "(%"PRIu64") IOCTL: VHOST_RESET_OWNER\n", ctx.fh);
+               LOG_DEBUG(VHOST_CONFIG,
+                       "(%"PRIu64") IOCTL: VHOST_RESET_OWNER\n", ctx.fh);
                VHOST_IOCTL(ops->reset_owner);
                break;
 
        case VHOST_SET_OWNER:
-               LOG_DEBUG(VHOST_CONFIG, "(%"PRIu64") IOCTL: VHOST_SET_OWNER\n", ctx.fh);
+               LOG_DEBUG(VHOST_CONFIG,
+                       "(%"PRIu64") IOCTL: VHOST_SET_OWNER\n", ctx.fh);
                VHOST_IOCTL(ops->set_owner);
                break;
 
        case VHOST_SET_MEM_TABLE:
                /*TODO fix race condition.*/
-               LOG_DEBUG(VHOST_CONFIG, "(%"PRIu64") IOCTL: VHOST_SET_MEM_TABLE\n", ctx.fh);
+               LOG_DEBUG(VHOST_CONFIG,
+                       "(%"PRIu64") IOCTL: VHOST_SET_MEM_TABLE\n", ctx.fh);
                static struct vhost_memory mem_temp;
 
                switch (in_bufsz) {
@@ -227,7 +236,9 @@ vhost_net_ioctl(fuse_req_t req, int cmd, void *arg,
                        mem_temp = *(const struct vhost_memory *) in_buf;
 
                        if (mem_temp.nregions > 0) {
-                               VHOST_IOCTL_RETRY(sizeof(struct vhost_memory) + (sizeof(struct vhost_memory_region) * mem_temp.nregions), 0);
+                               VHOST_IOCTL_RETRY(sizeof(struct vhost_memory) +
+                                       (sizeof(struct vhost_memory_region) *
+                                               mem_temp.nregions), 0);
                        } else {
                                result = -1;
                                fuse_reply_ioctl(req, result, NULL, 0);
@@ -235,56 +246,70 @@ vhost_net_ioctl(fuse_req_t req, int cmd, void *arg,
                        break;
 
                default:
-                       result = ops->set_mem_table(ctx, in_buf, mem_temp.nregions);
+                       result = ops->set_mem_table(ctx,
+                                       in_buf, mem_temp.nregions);
                        if (result)
                                fuse_reply_err(req, EINVAL);
                        else
                                fuse_reply_ioctl(req, result, NULL, 0);
-
                }
-
                break;
 
        case VHOST_SET_VRING_NUM:
-               LOG_DEBUG(VHOST_CONFIG, "(%"PRIu64") IOCTL: VHOST_SET_VRING_NUM\n", ctx.fh);
-               VHOST_IOCTL_R(struct vhost_vring_state, state, ops->set_vring_num);
+               LOG_DEBUG(VHOST_CONFIG,
+                       "(%"PRIu64") IOCTL: VHOST_SET_VRING_NUM\n", ctx.fh);
+               VHOST_IOCTL_R(struct vhost_vring_state, state,
+                       ops->set_vring_num);
                break;
 
        case VHOST_SET_VRING_BASE:
-               LOG_DEBUG(VHOST_CONFIG, "(%"PRIu64") IOCTL: VHOST_SET_VRING_BASE\n", ctx.fh);
-               VHOST_IOCTL_R(struct vhost_vring_state, state, ops->set_vring_base);
+               LOG_DEBUG(VHOST_CONFIG,
+                       "(%"PRIu64") IOCTL: VHOST_SET_VRING_BASE\n", ctx.fh);
+               VHOST_IOCTL_R(struct vhost_vring_state, state,
+                       ops->set_vring_base);
                break;
 
        case VHOST_GET_VRING_BASE:
-               LOG_DEBUG(VHOST_CONFIG, "(%"PRIu64") IOCTL: VHOST_GET_VRING_BASE\n", ctx.fh);
-               VHOST_IOCTL_RW(uint32_t, index, struct vhost_vring_state, state, ops->get_vring_base);
+               LOG_DEBUG(VHOST_CONFIG,
+                       "(%"PRIu64") IOCTL: VHOST_GET_VRING_BASE\n", ctx.fh);
+               VHOST_IOCTL_RW(uint32_t, index,
+                       struct vhost_vring_state, state, ops->get_vring_base);
                break;
 
        case VHOST_SET_VRING_ADDR:
-               LOG_DEBUG(VHOST_CONFIG, "(%"PRIu64") IOCTL: VHOST_SET_VRING_ADDR\n", ctx.fh);
-               VHOST_IOCTL_R(struct vhost_vring_addr, addr, ops->set_vring_addr);
+               LOG_DEBUG(VHOST_CONFIG,
+                       "(%"PRIu64") IOCTL: VHOST_SET_VRING_ADDR\n", ctx.fh);
+               VHOST_IOCTL_R(struct vhost_vring_addr, addr,
+                       ops->set_vring_addr);
                break;
 
        case VHOST_SET_VRING_KICK:
-               LOG_DEBUG(VHOST_CONFIG, "(%"PRIu64") IOCTL: VHOST_SET_VRING_KICK\n", ctx.fh);
-               VHOST_IOCTL_R(struct vhost_vring_file, file, ops->set_vring_kick);
+               LOG_DEBUG(VHOST_CONFIG,
+                       "(%"PRIu64") IOCTL: VHOST_SET_VRING_KICK\n", ctx.fh);
+               VHOST_IOCTL_R(struct vhost_vring_file, file,
+                       ops->set_vring_kick);
                break;
 
        case VHOST_SET_VRING_CALL:
-               LOG_DEBUG(VHOST_CONFIG, "(%"PRIu64") IOCTL: VHOST_SET_VRING_CALL\n", ctx.fh);
-               VHOST_IOCTL_R(struct vhost_vring_file, file, ops->set_vring_call);
+               LOG_DEBUG(VHOST_CONFIG,
+                       "(%"PRIu64") IOCTL: VHOST_SET_VRING_CALL\n", ctx.fh);
+               VHOST_IOCTL_R(struct vhost_vring_file, file,
+                       ops->set_vring_call);
                break;
 
        default:
-               RTE_LOG(ERR, VHOST_CONFIG, "(%"PRIu64") IOCTL: DOESN NOT EXIST\n", ctx.fh);
+               RTE_LOG(ERR, VHOST_CONFIG,
+                       "(%"PRIu64") IOCTL: DOESN NOT EXIST\n", ctx.fh);
                result = -1;
                fuse_reply_ioctl(req, result, NULL, 0);
        }
 
        if (result < 0)
-               LOG_DEBUG(VHOST_CONFIG, "(%"PRIu64") IOCTL: FAIL\n", ctx.fh);
+               LOG_DEBUG(VHOST_CONFIG,
+                       "(%"PRIu64") IOCTL: FAIL\n", ctx.fh);
        else
-               LOG_DEBUG(VHOST_CONFIG, "(%"PRIu64") IOCTL: SUCCESS\n", ctx.fh);
+               LOG_DEBUG(VHOST_CONFIG,
+                       "(%"PRIu64") IOCTL: SUCCESS\n", ctx.fh);
 }
 
 /*
@@ -297,8 +322,8 @@ static const struct cuse_lowlevel_ops vhost_net_ops = {
 };
 
 /*
- * cuse_info is populated and used to register the cuse device. vhost_net_device_ops are
- * also passed when the device is registered in main.c.
+ * cuse_info is populated and used to register the cuse device.
+ * vhost_net_device_ops are also passed when the device is registered in app.
  */
 int
 rte_vhost_driver_register(const char *dev_name)
@@ -314,20 +339,23 @@ rte_vhost_driver_register(const char *dev_name)
        char *fuse_argv[] = {fuse_opt_dummy, fuse_opt_fore, fuse_opt_nomulti};
 
        if (access(cuse_device_name, R_OK | W_OK) < 0) {
-               RTE_LOG(ERR, VHOST_CONFIG, "Character device %s can't be accessed, maybe not exist\n", cuse_device_name);
+               RTE_LOG(ERR, VHOST_CONFIG,
+                       "char device %s can't be accessed, maybe not exist\n",
+                       cuse_device_name);
                return -1;
        }
 
        /*
-        * The device name is created. This is passed to QEMU so that it can register
-        * the device with our application.
+        * The device name is created. This is passed to QEMU so that it can
+        * register the device with our application.
         */
        snprintf(device_name, PATH_MAX, "DEVNAME=%s", dev_name);
        snprintf(char_device_name, PATH_MAX, "/dev/%s", dev_name);
 
        /* Check if device already exists. */
        if (access(char_device_name, F_OK) != -1) {
-               RTE_LOG(ERR, VHOST_CONFIG, "Character device %s already exists\n", char_device_name);
+               RTE_LOG(ERR, VHOST_CONFIG,
+                       "char device %s already exists\n", char_device_name);
                return -1;
        }
 
@@ -341,7 +369,7 @@ rte_vhost_driver_register(const char *dev_name)
        ops = get_virtio_net_callbacks();
 
        session = cuse_lowlevel_setup(3, fuse_argv,
-                               &cuse_info, &vhost_net_ops, 0, NULL);
+                       &cuse_info, &vhost_net_ops, 0, NULL);
        if (session == NULL)
                return -1;
 
@@ -349,7 +377,8 @@ rte_vhost_driver_register(const char *dev_name)
 }
 
 /**
- * The CUSE session is launched allowing the application to receive open, release and ioctl calls.
+ * The CUSE session is launched allowing the application to receive open,
+ * release and ioctl calls.
  */
 int
 rte_vhost_driver_session_start(void)