Hi Bernard: > -----Original Message----- > From: Iremonger, Bernard <[email protected]> > Sent: Thursday, January 9, 2020 8:17 PM > To: [email protected]; Xing, Beilei <[email protected]>; Zhang, Qi Z > <[email protected]>; Doherty, Declan <[email protected]> > Cc: Ananyev, Konstantin <[email protected]>; Byrne, Stephen1 > <[email protected]>; Zhang, Helin <[email protected]>; > Iremonger, Bernard <[email protected]> > Subject: [PATCH v3 6/9] net/i40e: display Flow Director packet > > call rte_hexdump in i40e_flow_fdir_construct_pkt() in i40e_fdir.c > > Signed-off-by: Bernard Iremonger <[email protected]> > --- > drivers/net/i40e/i40e_fdir.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/net/i40e/i40e_fdir.c b/drivers/net/i40e/i40e_fdir.c index > 3fa6297..78329d2 100644 > --- a/drivers/net/i40e/i40e_fdir.c > +++ b/drivers/net/i40e/i40e_fdir.c > @@ -21,6 +21,7 @@ > #include <rte_tcp.h> > #include <rte_sctp.h> > #include <rte_hash_crc.h> > +#include <rte_hexdump.h> > > #include "i40e_logs.h" > #include "base/i40e_type.h" > @@ -805,6 +806,7 @@ i40e_fdir_fill_eth_ip_head(const struct > rte_eth_fdir_input *fdir_input, > fdir_input->flow_type); > return -1; > } > + rte_hexdump(stdout, NULL, raw_pkt, len);
Why we need this? Does this just for debug? Regards Qi > return len; > } > > @@ -954,7 +956,7 @@ i40e_fdir_construct_pkt(struct i40e_pf *pf, > &fdir_input->flow_ext.flexbytes[dst], > size * sizeof(uint16_t)); > } > - > + rte_hexdump(stdout, NULL, raw_pkt, len); > return 0; > } > > -- > 2.7.4
