[PATCH] net/eth: Have eth_pad_short_frame() take void* arguments

Philippe Mathieu-Daudé posted 1 patch 10 months, 3 weeks ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20240108154123.53680-1-philmd@linaro.org
Maintainers: Dmitry Fleytman <dmitry.fleytman@gmail.com>, Akihiko Odaki <akihiko.odaki@daynix.com>, Jason Wang <jasowang@redhat.com>
include/net/eth.h | 2 +-
net/eth.c         | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
[PATCH] net/eth: Have eth_pad_short_frame() take void* arguments
Posted by Philippe Mathieu-Daudé 10 months, 3 weeks ago
Any kind of buffer can hold an Ethernet frame, no just
arrays of unsigned chars.

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
 include/net/eth.h | 2 +-
 net/eth.c         | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/net/eth.h b/include/net/eth.h
index 3b80b6e07f..d73737f07d 100644
--- a/include/net/eth.h
+++ b/include/net/eth.h
@@ -434,7 +434,7 @@ eth_parse_ipv6_hdr(const struct iovec *pkt, int pkt_frags,
  * @pkt_size: size of the original Ethernet frame
  * @return true if the frame is padded, otherwise false
  */
-bool eth_pad_short_frame(uint8_t *padded_pkt, size_t *padded_buflen,
+bool eth_pad_short_frame(void *padded_pkt, size_t *padded_buflen,
                          const void *pkt, size_t pkt_size);
 
 #endif
diff --git a/net/eth.c b/net/eth.c
index 3f680cc033..25c6eb9b20 100644
--- a/net/eth.c
+++ b/net/eth.c
@@ -520,7 +520,7 @@ bool eth_parse_ipv6_hdr(const struct iovec *pkt, int pkt_frags,
     return true;
 }
 
-bool eth_pad_short_frame(uint8_t *padded_pkt, size_t *padded_buflen,
+bool eth_pad_short_frame(void *padded_pkt, size_t *padded_buflen,
                          const void *pkt, size_t pkt_size)
 {
     assert(padded_buflen && *padded_buflen >= ETH_ZLEN);
-- 
2.41.0


Re: [PATCH] net/eth: Have eth_pad_short_frame() take void* arguments
Posted by Akihiko Odaki 10 months, 3 weeks ago
On 2024/01/09 0:41, Philippe Mathieu-Daudé wrote:
> Any kind of buffer can hold an Ethernet frame, no just
> arrays of unsigned chars.
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>

Reviewed-by: Akihiko Odaki <akihiko.odaki@daynix.com>