Re: [ovs-dev] [0/2] extend nlmon functionality to support TC flowers monitoring

2021-12-07 Thread Ilya Maximets
On 12/7/21 16:36, Mohammad Heib wrote: > Hi Ilya, > > Thank you for reviewing the patches. > > i fully agree that is a code duplication but I wasn't sure if I can > include those files in nlmon code since those are kernel-headers files. > so I sent an email to OVS-DISCUSS mailing list but didn't

Re: [ovs-dev] [0/2] extend nlmon functionality to support TC flowers monitoring

2021-12-07 Thread Mohammad Heib
Hi Ilya, Thank you for reviewing the patches. i fully agree that is a code duplication but I wasn't sure if I can include those files in nlmon code since those are kernel-headers files. so I sent an email to OVS-DISCUSS mailing list but didn't get a helpful answer: email link: https://www.mail-ar

Re: [ovs-dev] [0/2] extend nlmon functionality to support TC flowers monitoring

2021-12-07 Thread Ilya Maximets
On 11/22/21 00:39, Mohammad Heib wrote: > These two patches aim to make nlmon tool more generic > and extend its functionality to support TC flowers/actions > monitoring and parsing. > > This change will improve the visibility of the communication > between the OVS and the TC subsystem and can be

[ovs-dev] [0/2] extend nlmon functionality to support TC flowers monitoring

2021-11-21 Thread Mohammad Heib
These two patches aim to make nlmon tool more generic and extend its functionality to support TC flowers/actions monitoring and parsing. This change will improve the visibility of the communication between the OVS and the TC subsystem and can be used for debugging and testing OVS HW offload commun