Since it is a function pointer,
the expression *dev->extern_ops.pre_msg_handle and
dev->extern_ops.pre_msg_handle are the same.

Signed-off-by: Stephen Hemminger <step...@networkplumber.org>
---
 lib/vhost/vhost_user.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/vhost/vhost_user.c b/lib/vhost/vhost_user.c
index 26dc0bde97..45f2b71e36 100644
--- a/lib/vhost/vhost_user.c
+++ b/lib/vhost/vhost_user.c
@@ -3186,7 +3186,7 @@ vhost_user_msg_handler(int vid, int fd)
        handled = false;
        if (dev->extern_ops.pre_msg_handle) {
                RTE_BUILD_BUG_ON(offsetof(struct vhu_msg_context, msg) != 0);
-               msg_result = (*dev->extern_ops.pre_msg_handle)(dev->vid, &ctx);
+               msg_result = dev->extern_ops.pre_msg_handle(dev->vid, &ctx);
                switch (msg_result) {
                case RTE_VHOST_MSG_RESULT_REPLY:
                        send_vhost_reply(dev, fd, &ctx);
@@ -3238,7 +3238,7 @@ vhost_user_msg_handler(int vid, int fd)
        if (msg_result != RTE_VHOST_MSG_RESULT_ERR &&
                        dev->extern_ops.post_msg_handle) {
                RTE_BUILD_BUG_ON(offsetof(struct vhu_msg_context, msg) != 0);
-               msg_result = (*dev->extern_ops.post_msg_handle)(dev->vid, &ctx);
+               msg_result = dev->extern_ops.post_msg_handle(dev->vid, &ctx);
                switch (msg_result) {
                case RTE_VHOST_MSG_RESULT_REPLY:
                        send_vhost_reply(dev, fd, &ctx);
-- 
2.47.2

Reply via email to