On 7/15/2022 5:18 AM, Ilya Maximets wrote:
On 7/14/22 23:23, Greg Rose wrote:
It is time to remove support for the ovs kernel driver and push
towards use of the upstream linux openvswitch kernel driver
in it's place [1].

1.  https://mail.openvswitch.org/pipermail/ovs-dev/2022-april/393292.html

Changes in V2:
* Removed the Linux datapath and fixed up various Makefiles to adjust
* Moved openvswitch.h from datapath/linux/compat/include to include/linux
   and fixed up the Makefiles to adjust
* Removed further instances in utilities and auxiliary files where the
   --with-linux configuration command was still being used.
* Changed order of patches to (hopefully) fix up some robot auto-test
   errors

Changes in V3:
* Squashed the two patches removing rhel specs due to interdependence
   between the two that caused build errors between application of each
   of the patches
* Moved the changes to acinclude.m4 towards the end of the series

Changes in V4:
* Changed up the order of the patches again to fix robot errors
* Added back most of the tests in .github/workflows/build-and-test.yml
* Removed xenserver and related documentation, test scripts and makefile
   dependencies

Changes in V5:
* Squashed two patches from previous series into a single patch. There
   was no good reason to separate changes to auxiliary and utilities
   from the removal of the Linux datapath and the two patches had a
   tricky dependency between them.  Best to combine them.
* Various cleanups, title changes and amended commit messages as
   suggested by David Marchand in his reviews.
* Drops the xenserver removal patch - this patch causes 'make  check'
   failures and there is no reason why it can't  be submitted separately.

Changes in V6:
* Applied Ilya's suggested patch to fix up the robot error caused by
   missing all-distfiles.  We'll see if that fixup stops the robot from
   complaining.
* With these changes the github workflows build and test work without
   errors:
https://github.com/gvrose8192/ovs-experimental/runs/7347904715?check_suite_focus=true

Still to do:
* Submit the patch to remove xenserver separately once I figure out why
   the removal causes 'make check' errors for a few interface reconfiguration
   tests.

Yeh, would be nice to clean up that part as well in a future patch.

Working on it now.



Greg Rose (6):
   ci: Stop building OVS kernel module.
   rhel: Stop packaging OVS kernel module.
   tests: Remove support for check-kmod test.
   Make: Remove the Linux datapath.
   acinclude.m4: Remove support for building the OVS kernel module.
   Documentation: Remove kernel module documentation.

Thanks, Greg and David!  Applied.

Best regards, Ilya Maximets.

Awesome, thank you and much thanks to David as well!

- Greg
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to