git: a60685ffbc74 - stable/13 - vnic: add TODO list

From: Ed Maste <emaste_at_FreeBSD.org>
Date: Sat, 12 Feb 2022 19:10:33 UTC
The branch stable/13 has been updated by emaste:

URL: https://cgit.FreeBSD.org/src/commit/?id=a60685ffbc743e417972390ce6674ce38918a115

commit a60685ffbc743e417972390ce6674ce38918a115
Author:     Ed Maste <emaste@FreeBSD.org>
AuthorDate: 2021-07-27 21:51:21 +0000
Commit:     Ed Maste <emaste@FreeBSD.org>
CommitDate: 2022-02-12 19:09:52 +0000

    vnic: add TODO list
    
    - non-promisc mode
    - multicast filter support
    
    Also drop ARM64TODO comments; this is an issue with this specific
    driver, not a general arm64 issue.
    
    PR:             223575
    PR:             223573
    
    (cherry picked from commit 941650aae9762293f4c63788ddb17e0114828813)
    (cherry picked from commit 29e9b487c1d427409a8bf1f893bc71a57c76f4b0)
---
 sys/dev/vnic/nic_main.c   | 12 ++++++++++++
 sys/dev/vnic/nicvf_main.c |  6 ++----
 2 files changed, 14 insertions(+), 4 deletions(-)

diff --git a/sys/dev/vnic/nic_main.c b/sys/dev/vnic/nic_main.c
index 8e6aa60f6ca8..08559442dbc7 100644
--- a/sys/dev/vnic/nic_main.c
+++ b/sys/dev/vnic/nic_main.c
@@ -27,6 +27,18 @@
  *
  */
 
+/*
+ * Marvell/Cavium ThunderX vnic/bgx network controller
+ *
+ * UNIMPLEMENTED FEATURES
+ * ----------------------
+ * A number of features supported by the hardware are not yet implemented in
+ * this driver:
+ *
+ * - PR223573 multicast rx filter
+ * - PR223575 non-promiscuous mode (driver currently forces promisc)
+ */
+
 #include <sys/cdefs.h>
 __FBSDID("$FreeBSD$");
 
diff --git a/sys/dev/vnic/nicvf_main.c b/sys/dev/vnic/nicvf_main.c
index c8b088dee1fa..73591a0b161a 100644
--- a/sys/dev/vnic/nicvf_main.c
+++ b/sys/dev/vnic/nicvf_main.c
@@ -485,16 +485,14 @@ nicvf_if_ioctl(struct ifnet *ifp, u_long cmd, caddr_t data)
 			if (if_getdrvflags(ifp) & IFF_DRV_RUNNING) {
 				if ((flags ^ nic->if_flags) & IFF_PROMISC) {
 					/* Change promiscous mode */
-#if 0
-					/* ARM64TODO */
+#if 0 /* XXX* /
 					nicvf_set_promiscous(nic);
 #endif
 				}
 
 				if ((flags ^ nic->if_flags) & IFF_ALLMULTI) {
 					/* Change multicasting settings */
-#if 0
-					/* ARM64TODO */
+#if 0 /* XXX */
 					nicvf_set_multicast(nic);
 #endif
 				}