Re: [ovs-dev] [PATCH V3 7/7] Documentation: Remove kernel module documentation

2022-07-11 Thread Gregory Rose




On 7/9/2022 12:31 AM, David Marchand wrote:

On Fri, Jul 8, 2022 at 10:09 PM Greg Rose  wrote:

diff --git a/Documentation/internals/contributing/backporting-patches.rst 
b/Documentation/internals/contributing/backporting-patches.rst
index 162e9d209..8370c954d 100644
--- a/Documentation/internals/contributing/backporting-patches.rst
+++ b/Documentation/internals/contributing/backporting-patches.rst
@@ -119,6 +119,13 @@ userspace changes.
  How to backport kernel patches
  ~~

+These instructions only apply to Open vSwitch releases 2.16 and older.
+As of Open vSwitch branch 2.17 the Open vSwitch kernel module is no
+longer supported and only the Linux openvswitch kernel module is used.
+In the case of Open vSwitch releases 2.16 and older kernel backports
+may be required for bux fixes and feature implementation so these
+instructions are preserved for that reason.
+
  First, the patch should be submitted upstream to `netdev`. When the patch has
  been applied to `net-next`, it is ready to be backported. Starting from the
  Linux tree, use ``git format-patch`` to format each patch that should be


Adding to Ilya comments.

Here, should those versions here be bumped by one too?
Leaving it as proposed in this patch, this saves us from potential
backporting effort to 2.17 too :-).




I think they're supposed to be bumped.  I'll take care of  it.

Thanks,

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


Re: [ovs-dev] [PATCH V3 7/7] Documentation: Remove kernel module documentation

2022-07-09 Thread David Marchand
On Fri, Jul 8, 2022 at 10:09 PM Greg Rose  wrote:
> diff --git a/Documentation/internals/contributing/backporting-patches.rst 
> b/Documentation/internals/contributing/backporting-patches.rst
> index 162e9d209..8370c954d 100644
> --- a/Documentation/internals/contributing/backporting-patches.rst
> +++ b/Documentation/internals/contributing/backporting-patches.rst
> @@ -119,6 +119,13 @@ userspace changes.
>  How to backport kernel patches
>  ~~
>
> +These instructions only apply to Open vSwitch releases 2.16 and older.
> +As of Open vSwitch branch 2.17 the Open vSwitch kernel module is no
> +longer supported and only the Linux openvswitch kernel module is used.
> +In the case of Open vSwitch releases 2.16 and older kernel backports
> +may be required for bux fixes and feature implementation so these
> +instructions are preserved for that reason.
> +
>  First, the patch should be submitted upstream to `netdev`. When the patch has
>  been applied to `net-next`, it is ready to be backported. Starting from the
>  Linux tree, use ``git format-patch`` to format each patch that should be

Adding to Ilya comments.

Here, should those versions here be bumped by one too?
Leaving it as proposed in this patch, this saves us from potential
backporting effort to 2.17 too :-).


-- 
David Marchand

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


Re: [ovs-dev] [PATCH V3 7/7] Documentation: Remove kernel module documentation

2022-07-08 Thread Ilya Maximets
On 7/8/22 22:03, Greg Rose wrote:
> As of Open vSwitch release 2.18 the OVS kernel module is no longer
> supported.  Pull the documentation references.
> 
> Signed-off-by: Greg Rose 
> ---
>  Documentation/faq/releases.rst|  5 +-
>  .../contributing/backporting-patches.rst  |  7 +++
>  Documentation/intro/install/fedora.rst| 24 ---
>  Documentation/intro/install/general.rst   | 63 ---
>  4 files changed, 10 insertions(+), 89 deletions(-)
> 
> diff --git a/Documentation/faq/releases.rst b/Documentation/faq/releases.rst
> index 8cfe2d392..ab2925a62 100644
> --- a/Documentation/faq/releases.rst
> +++ b/Documentation/faq/releases.rst
> @@ -74,7 +74,7 @@ Q: What Linux kernel versions does each Open vSwitch 
> release work with?
>  2.14.x   3.16 to 5.5
>  2.15.x   3.16 to 5.8
>  2.16.x   3.16 to 5.8
> -2.17.x   3.16 to 5.8
> +2.17.x   N/A

We should leave the 2.17 entry here add something like:

   2.18+N/A   

>   ==
>  
>  Open vSwitch userspace should also work with the Linux kernel module 
> built
> @@ -110,7 +110,8 @@ Q: Are all features available with all datapaths?
>  Linux OVS tree
>The datapath implemented by the Linux kernel module distributed with
>the OVS source tree. This datapath is deprecated starting with OVS
> -  2.15.x and support capped at Linux kernel version 5.8.
> +  2.15.x and support capped at Linux kernel version 5.8. As of OVS 2.17.x
> +  the Linux OVS tree is no longer supported.

It should be 2.18 as well.

Best regards, Ilya Maximets.
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


Re: [ovs-dev] [PATCH V3 7/7] Documentation: Remove kernel module documentation

2022-07-08 Thread 0-day Robot
Bleep bloop.  Greetings Gregory Rose, I am a robot and I have tried out your 
patch.
Thanks for your contribution.

I encountered some error that I wasn't expecting.  See the details below.


Patch skipped due to previous failure.

Please check this out.  If you feel there has been an error, please email 
acon...@redhat.com

Thanks,
0-day Robot
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev