Re: [ovs-dev] [PATCH V6 0/6] Remove OVS kernel driver
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
Re: [ovs-dev] [PATCH V6 0/6] Remove OVS kernel driver
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. > > 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. ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev