mirror of https://github.com/xemu-project/xemu.git
igb: Fix Rx packet type encoding
igb's advanced descriptor uses a packet type encoding different from
one used in e1000e's extended descriptor. Fix the logic to encode
Rx packet type accordingly.
Fixes: 3a977deebe
("Intrdocue igb device emulation")
Signed-off-by: Akihiko Odaki <akihiko.odaki@daynix.com>
Reviewed-by: Sriram Yagnaraman <sriram.yagnaraman@est.tech>
Signed-off-by: Jason Wang <jasowang@redhat.com>
This commit is contained in:
parent
f3f9b726af
commit
ed447c60b3
|
@ -1227,7 +1227,6 @@ igb_build_rx_metadata(IGBCore *core,
|
|||
struct virtio_net_hdr *vhdr;
|
||||
bool hasip4, hasip6;
|
||||
EthL4HdrProto l4hdr_proto;
|
||||
uint32_t pkt_type;
|
||||
|
||||
*status_flags = E1000_RXD_STAT_DD;
|
||||
|
||||
|
@ -1266,28 +1265,29 @@ igb_build_rx_metadata(IGBCore *core,
|
|||
trace_e1000e_rx_metadata_ack();
|
||||
}
|
||||
|
||||
if (hasip6 && (core->mac[RFCTL] & E1000_RFCTL_IPV6_DIS)) {
|
||||
trace_e1000e_rx_metadata_ipv6_filtering_disabled();
|
||||
pkt_type = E1000_RXD_PKT_MAC;
|
||||
} else if (l4hdr_proto == ETH_L4_HDR_PROTO_TCP ||
|
||||
l4hdr_proto == ETH_L4_HDR_PROTO_UDP) {
|
||||
pkt_type = hasip4 ? E1000_RXD_PKT_IP4_XDP : E1000_RXD_PKT_IP6_XDP;
|
||||
} else if (hasip4 || hasip6) {
|
||||
pkt_type = hasip4 ? E1000_RXD_PKT_IP4 : E1000_RXD_PKT_IP6;
|
||||
} else {
|
||||
pkt_type = E1000_RXD_PKT_MAC;
|
||||
}
|
||||
|
||||
trace_e1000e_rx_metadata_pkt_type(pkt_type);
|
||||
|
||||
if (pkt_info) {
|
||||
if (rss_info->enabled) {
|
||||
*pkt_info = rss_info->type;
|
||||
*pkt_info = rss_info->enabled ? rss_info->type : 0;
|
||||
|
||||
if (hasip4) {
|
||||
*pkt_info |= E1000_ADVRXD_PKT_IP4;
|
||||
}
|
||||
|
||||
*pkt_info |= (pkt_type << 4);
|
||||
} else {
|
||||
*status_flags |= E1000_RXD_PKT_TYPE(pkt_type);
|
||||
if (hasip6) {
|
||||
*pkt_info |= E1000_ADVRXD_PKT_IP6;
|
||||
}
|
||||
|
||||
switch (l4hdr_proto) {
|
||||
case ETH_L4_HDR_PROTO_TCP:
|
||||
*pkt_info |= E1000_ADVRXD_PKT_TCP;
|
||||
break;
|
||||
|
||||
case ETH_L4_HDR_PROTO_UDP:
|
||||
*pkt_info |= E1000_ADVRXD_PKT_UDP;
|
||||
break;
|
||||
|
||||
default:
|
||||
break;
|
||||
}
|
||||
}
|
||||
|
||||
if (hdr_info) {
|
||||
|
|
|
@ -641,6 +641,11 @@ union e1000_adv_rx_desc {
|
|||
|
||||
#define E1000_STATUS_NUM_VFS_SHIFT 14
|
||||
|
||||
#define E1000_ADVRXD_PKT_IP4 BIT(4)
|
||||
#define E1000_ADVRXD_PKT_IP6 BIT(6)
|
||||
#define E1000_ADVRXD_PKT_TCP BIT(8)
|
||||
#define E1000_ADVRXD_PKT_UDP BIT(9)
|
||||
|
||||
static inline uint8_t igb_ivar_entry_rx(uint8_t i)
|
||||
{
|
||||
return i < 8 ? i * 4 : (i - 8) * 4 + 2;
|
||||
|
|
Loading…
Reference in New Issue