Message ID | 20220714212331.24848-1-gvrose8192@gmail.com |
---|---|
Headers | show |
Series | Remove OVS kernel driver | expand |
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.
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