From ab85eaf41a67e71bd50a0c4445544f2f977cb401 Mon Sep 17 00:00:00 2001 From: Jan Klemkow Date: Tue, 21 May 2024 19:49:07 +0000 Subject: Refactor vmx(4) receive offloading. - avoid combination of M_TCP_CSUM_IN_OK and M_UDP_CSUM_IN_OK - remove useless letoh32() calls - simplify the IPv4 checksum flag check ok bluhm@ --- sys/dev/pci/if_vmx.c | 51 +++++++++++++++++++++++++++++++-------------------- 1 file changed, 31 insertions(+), 20 deletions(-) (limited to 'sys') diff --git a/sys/dev/pci/if_vmx.c b/sys/dev/pci/if_vmx.c index 28853b5471b..0c5883a1bca 100644 --- a/sys/dev/pci/if_vmx.c +++ b/sys/dev/pci/if_vmx.c @@ -1,4 +1,4 @@ -/* $OpenBSD: if_vmx.c,v 1.85 2024/05/13 01:15:51 jsg Exp $ */ +/* $OpenBSD: if_vmx.c,v 1.86 2024/05/21 19:49:06 jan Exp $ */ /* * Copyright (c) 2013 Tsubai Masanari @@ -203,7 +203,7 @@ void vmxnet3_rxintr(struct vmxnet3_softc *, struct vmxnet3_rxqueue *); void vmxnet3_rxfill_tick(void *); void vmxnet3_rxfill(struct vmxnet3_rxring *); void vmxnet3_iff(struct vmxnet3_softc *); -void vmxnet3_rx_csum(struct vmxnet3_rxcompdesc *, struct mbuf *); +void vmxnet3_rx_offload(struct vmxnet3_rxcompdesc *, struct mbuf *); void vmxnet3_stop(struct ifnet *); void vmxnet3_reset(struct vmxnet3_softc *); int vmxnet3_init(struct vmxnet3_softc *); @@ -1127,14 +1127,8 @@ vmxnet3_rxintr(struct vmxnet3_softc *sc, struct vmxnet3_rxqueue *rq) } m->m_pkthdr.len = m->m_len = len; - vmxnet3_rx_csum(rxcd, m); -#if NVLAN > 0 - if (letoh32(rxcd->rxc_word2 & VMXNET3_RXC_VLAN)) { - m->m_flags |= M_VLANTAG; - m->m_pkthdr.ether_vtag = letoh32((rxcd->rxc_word2 >> - VMXNET3_RXC_VLANTAG_S) & VMXNET3_RXC_VLANTAG_M); - } -#endif + vmxnet3_rx_offload(rxcd, m); + if (((letoh32(rxcd->rxc_word0) >> VMXNET3_RXC_RSSTYPE_S) & VMXNET3_RXC_RSSTYPE_M) != VMXNET3_RXC_RSSTYPE_NONE) { m->m_pkthdr.ph_flowid = letoh32(rxcd->rxc_word1); @@ -1215,22 +1209,39 @@ vmxnet3_iff(struct vmxnet3_softc *sc) void -vmxnet3_rx_csum(struct vmxnet3_rxcompdesc *rxcd, struct mbuf *m) +vmxnet3_rx_offload(struct vmxnet3_rxcompdesc *rxcd, struct mbuf *m) { - if (letoh32(rxcd->rxc_word0 & VMXNET3_RXC_NOCSUM)) + /* + * VLAN Offload + */ + +#if NVLAN > 0 + if (ISSET(rxcd->rxc_word2, VMXNET3_RXC_VLAN)) { + SET(m->m_flags, M_VLANTAG); + m->m_pkthdr.ether_vtag = letoh32((rxcd->rxc_word2 >> + VMXNET3_RXC_VLANTAG_S) & VMXNET3_RXC_VLANTAG_M); + } +#endif + + /* + * Checksum Offload + */ + + if (ISSET(rxcd->rxc_word0, VMXNET3_RXC_NOCSUM)) return; - if ((rxcd->rxc_word3 & (VMXNET3_RXC_IPV4 | VMXNET3_RXC_IPSUM_OK)) == - (VMXNET3_RXC_IPV4 | VMXNET3_RXC_IPSUM_OK)) - m->m_pkthdr.csum_flags |= M_IPV4_CSUM_IN_OK; + if (ISSET(rxcd->rxc_word3, VMXNET3_RXC_IPV4) && + ISSET(rxcd->rxc_word3, VMXNET3_RXC_IPSUM_OK)) + SET(m->m_pkthdr.csum_flags, M_IPV4_CSUM_IN_OK); - if (rxcd->rxc_word3 & VMXNET3_RXC_FRAGMENT) + if (ISSET(rxcd->rxc_word3, VMXNET3_RXC_FRAGMENT)) return; - if (rxcd->rxc_word3 & (VMXNET3_RXC_TCP | VMXNET3_RXC_UDP)) { - if (rxcd->rxc_word3 & VMXNET3_RXC_CSUM_OK) - m->m_pkthdr.csum_flags |= - M_TCP_CSUM_IN_OK | M_UDP_CSUM_IN_OK; + if (ISSET(rxcd->rxc_word3, VMXNET3_RXC_CSUM_OK)) { + if (ISSET(rxcd->rxc_word3, VMXNET3_RXC_TCP)) + SET(m->m_pkthdr.csum_flags, M_TCP_CSUM_IN_OK); + else if (ISSET(rxcd->rxc_word3, VMXNET3_RXC_UDP)) + SET(m->m_pkthdr.csum_flags, M_UDP_CSUM_IN_OK); } } -- cgit v1.2.3