On Wed, Nov 25, 2020 at 4:05 PM Ilya Maximets <[email protected]> wrote:
>
> On 11/25/20 3:57 PM, Ilya Maximets wrote:
> > Version 2:
> >   - Added fix for dpdk-dir cache.  We should not use 'native' machine.

Interesting.
This explains a crash I got once on GHA dpdk side but did not
reproduce: testpmd test failing with an Illegal instruction with a x86
ccache in place...
dpdk-latest will need to force -Dmachine=default.


I tested your series on dpdk-latest to check against 20.11-rc5, lgtm.
For 20.11 support, we will need some small adjustments:
- add -Dmachine=default in .ci/linux-build.sh,
- install ninja, python3-setuptools packages
- install the python3-wheel package, I suppose it is implicitly
installed in Travis CI envs but not in GHA envs
- with meson pip-installed only for user, PATH must be appended a
$HOME/.local/bin


As far as my own DPDK patch is concerned, your key generation is
interesting in that a change in configuration triggers a cache
invalidation.
OTOH we have a ccache whose benefits could be seen with ovsrobot.
In Travis CI, ovsrobot/dpdk has a weekly job to maintain a fresh
ccache in its main branch.
The "secrets" trick you mentioned offlist could be used for it for
both use cases.


--
David Marchand

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to