af_packet: use logging to unify the log at one place 90/42890/3
authorMohsin Kazmi <sykazmi@cisco.com>
Wed, 21 May 2025 19:20:23 +0000 (19:20 +0000)
committerFlorin Coras <florin.coras@gmail.com>
Thu, 22 May 2025 16:41:06 +0000 (16:41 +0000)
Type: fix

Signed-off-by: Mohsin Kazmi <sykazmi@cisco.com>
Change-Id: Iff5fe7e42ee10e2a0860dbd118083e9e64bb7398

src/plugins/af_packet/af_packet.c

index e3e67fb..e15d967 100644 (file)
@@ -175,6 +175,7 @@ af_packet_fd_read_ready (clib_file_t * uf)
 static clib_error_t *
 af_packet_fd_error (clib_file_t *uf)
 {
+  af_packet_main_t *apm = &af_packet_main;
   clib_error_t *err = 0;
   u64 u64;
 
@@ -183,19 +184,8 @@ af_packet_fd_error (clib_file_t *uf)
   if (ret < 0)
     {
       err = clib_error_return_unix (0, "");
-      ELOG_TYPE_DECLARE (e) = {
-       .format = "af-packet-msg: fd %u reason %s",
-       .format_args = "i4T4",
-      };
-      struct
-      {
-       u32 fd;
-       u32 reason;
-      } *ed;
-      ed = ELOG_DATA (&vlib_global_main.elog_main, e);
-      ed->fd = uf->file_descriptor;
-      ed->reason =
-       elog_string (vlib_get_elog_main (), "%U", format_clib_error, err);
+      vlib_log_err (apm->log_class, "fd %u reason %U", uf->file_descriptor,
+                   format_clib_error, err);
 
       clib_error_free (err);
     }