[dts] [PATCH V1] tests/vf_daemon: modify dts to adapt dpdk changes

Honnappa Nagarahalli Honnappa.Nagarahalli at arm.com
Wed Oct 20 18:41:05 CEST 2021


<snip>

> 
> On Tue, Oct 19, 2021 at 4:29 AM Yan Xia <yanx.xia at intel.com> wrote:
> >
> > dpdk commit-id: 04d43857
> > dpdk rename `s_addr` to `src_addr`
> >
> > Signed-off-by: Yan Xia <yanx.xia at intel.com>
> > ---
> >  tests/TestSuite_vf_daemon.py | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/tests/TestSuite_vf_daemon.py
> > b/tests/TestSuite_vf_daemon.py index a08ae7b5..2c23e760 100644
> > --- a/tests/TestSuite_vf_daemon.py
> > +++ b/tests/TestSuite_vf_daemon.py
> > @@ -528,7 +528,7 @@ class TestVfDaemon(TestCase):
> >              "\        struct rte_ether_addr fake_mac = {.addr_bytes = {0x00, 0x11,
> 0x22, 0x33, 0x44, 0x55},};'" +\
> >              " app/test-pmd/macswap_sse.h", "# ", 30)
> >          line_num = self.vm0_dut.send_expect("sed -n '/_mm_storeu_si128/='
> app/test-pmd/macswap_sse.h |sed -n 5p", "# ",30)
> > -        self.vm0_dut.send_expect("sed -i -e
> '%sa\\rte_ether_addr_copy(&fake_mac, &eth_hdr[0]->s_addr);'" %
> str(line_num)+\
> > +        self.vm0_dut.send_expect("sed -i -e
> > + '%sa\\rte_ether_addr_copy(&fake_mac, &eth_hdr[0]->src_addr);'" %
> > + str(line_num)+\
> 
> Please drop those hacks where DTS changes DPDK source code.
Totally agree. If you need any features in testpmd or any part of DPDK, please ask and we can talk about it.
Note that all the existing test cases that modify the DPDK code need to be reworked or dropped.

I also need to figure out a way to update the community of the DTS WG progress/recommendations. I will get back on this soon.

> 
> 
> >                      " app/test-pmd/macswap_sse.h", "# ", 30)
> >          time.sleep(3)
> >
> > --
> > 2.32.0
> >
> 
> 
> --
> David Marchand



More information about the dts mailing list