<div dir="ltr">I'll add that into a patch with any other suggestions I see. I also added that to my linter so it shouldn't happen again.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jul 16, 2020 at 3:49 PM Stephen Hemminger <<a href="mailto:stephen@networkplumber.org">stephen@networkplumber.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Thu, 16 Jul 2020 15:42:36 -0400<br>
Owen Hilyard <<a href="mailto:ohilyard@iol.unh.edu" target="_blank">ohilyard@iol.unh.edu</a>> wrote:<br>
<br>
> Sorry about sending this twice, I noticed after todays meeting that this was caught in the moderator queue and I want to get it out to the mailing list at large asap.<br>
> <br>
> add multicast filtering test case<br>
> add multicast test case test plan<br>
> <br>
> Signed-off-by: Owen Hilyard <<a href="mailto:ohilyard@iol.unh.edu" target="_blank">ohilyard@iol.unh.edu</a>><br>
> ---<br>
> test_plans/mac_filter_test_plan.rst | 27 ++++++++++++++++++++<br>
> tests/TestSuite_mac_filter.py | 39 ++++++++++++++++++++++++++---<br>
> 2 files changed, 62 insertions(+), 4 deletions(-)<br>
> <br>
> diff --git a/test_plans/mac_filter_test_plan.rst b/test_plans/mac_filter_test_plan.rst<br>
> index 1335e9f..e8a4f66 100644<br>
> --- a/test_plans/mac_filter_test_plan.rst<br>
> +++ b/test_plans/mac_filter_test_plan.rst<br>
> @@ -172,3 +172,30 @@ Add one more different address::<br>
> testpmd> mac_addr add 0 <A+n+1> <br>
> <br>
> Verify that the response is "No space left on device" (-ENOSPC)<br>
> +<br>
> +Test Case: Multicast Filter<br>
> +===========================<br>
> +<br>
> +Initialize first port without ``promiscuous mode``::<br>
> +<br>
> + testpmd> set promisc 0 off<br>
> +<br>
> +<br>
> +Add the multicast MAC address to the multicast filter::<br>
> +<br>
> + testpmd> mcast_addr add 0 01:00:5E:00:00:00<br>
> +<br>
> +Send a packet with multicast destination MAC address to port 0::<br>
> +<br>
> + port 0/queue 0: received 1 packets<br>
> + src=52:00:00:00:00:00 - dst=01:00:5E:00:00:00 - type=0x0800 - length=60 - nb_segs=1 - hw ptype: L2_ETHER L3_IPV4_EXT_UNKNOWN L4_UDP - sw ptype: L2_ETHER L3_IPV4 L4_UDP - l2_len=14 - l3_len=20 - l4_len=8 - Receive queue=0x0<br>
> + ol_flags: PKT_RX_L4_CKSUM_GOOD PKT_RX_IP_CKSUM_GOOD PKT_RX_OUTER_L4_CKSUM_UNKNOWN<br>
> +<br>
> +<br>
> +Remove the multicast MAC address from the multicast filter::<br>
> +<br>
> + testpmd> mcast_addr remove 0 01:00:5E:00:00:00<br>
> +<br>
> +Send a packet with multicast destination MAC address to port 0<br>
> +<br>
> +Verify that the packet was not received (Check for "received" in the output). There will be no output if the nic responds properly.<br>
> \ No newline at end of file<br>
<br>
Ok, but you must fix this.<br>
<br>
Please set your editor so it generates text files that always have a newline at the end.<br>
Although git supports files without ending newline, it is not something we want in this project.<br>
</blockquote></div>