userspace/wrsw_rtud: fix condition of printing cleaning multicast entries

Fix condition of printing "cleaning multicast entries not supported yet..." Since the search was done over all entries, it doesn't mean entry is multicast if port_mask_dst doesn't equal (1 << dest_port) Signed-off-by: Adam Wujek <adam.wujek@cern.ch>

userspace/wrsw_rtud: fix condition of printing cleaning multicast entries
Fix condition of printing "cleaning multicast entries not supported yet..." Since the search was done over all entries, it doesn't mean entry is multicast if port_mask_dst doesn't equal (1 << dest_port) Signed-off-by: Adam Wujek <adam.wujek@cern.ch>
320cbf8e · Adam Wujek · 4151c36c · 320cbf8e
Commit 320cbf8e authored Feb 23, 2016 by Adam Wujek 💬
Hide whitespace changes
Inline Side-by-side

Showing with 10 additions and 5 deletions

rtu_fd.c userspace/wrsw_rtud/rtu_fd.c +10 -5

No files found.
--- a/userspace/wrsw_rtud/rtu_fd.c
+++ b/userspace/wrsw_rtud/rtu_fd.c
@@ -563,13 +563,18 @@ void rtu_fd_clear_entries_for_port(int dest_port)
 		for (j = RTU_BUCKETS; j-- > 0;) {
 			ent = &rtu_htab[i][j];
 			if (ent->valid && ent->dynamic) {
-				if (ent->port_mask_dst == (1 << dest_port))
+				if (ent->port_mask_dst == (1 << dest_port)) {
+					/* entry is _only_ for this port */
 					hw_request(HW_REMOVE_REQ, ent->addr,
 						   ent);
-				else {
-					pr_error(
-					      "cleaning multicast entries not supported yet...\n");
-
+				}
+				else if (ent->port_mask_dst & (1 << dest_port)) {
+					/* entry is for at least two ports 
+					 * including current */
+					pr_error("cleaning multicast entries "
+						 "not supported yet... "
+						 "mask=0x%x port=%d\n",
+						 ent->port_mask_dst, dest_port);
 				}
 			}
 		}