diff mbox series

doc/net/l2tpv3: Update boolean fields' description to avoid short-form use

Message ID 20240708092630.2596945-1-zhao1.liu@intel.com
State New
Headers show
Series doc/net/l2tpv3: Update boolean fields' description to avoid short-form use | expand

Commit Message

Zhao Liu July 8, 2024, 9:26 a.m. UTC
The short-form boolean options has been deprecated since v6.0 (refer
to docs/about/deprecated.rst).

Update the description and example of boolean fields in l2tpv3 option to
avoid deprecation warning.

Cc: Jason Wang <jasowang@redhat.com>
Signed-off-by: Zhao Liu <zhao1.liu@intel.com>
---
 qemu-options.hx | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

Comments

Zhao Liu July 17, 2024, 6:31 a.m. UTC | #1
Hi Jason,

Just a kind ping. Does this update satisfy you?
Since the original example generates the warning.

Thanks,
Zhao

On Mon, Jul 08, 2024 at 05:26:30PM +0800, Zhao Liu wrote:
> Date: Mon, 8 Jul 2024 17:26:30 +0800
> From: Zhao Liu <zhao1.liu@intel.com>
> Subject: [PATCH] doc/net/l2tpv3: Update boolean fields' description to
>  avoid short-form use
> X-Mailer: git-send-email 2.34.1
> 
> The short-form boolean options has been deprecated since v6.0 (refer
> to docs/about/deprecated.rst).
> 
> Update the description and example of boolean fields in l2tpv3 option to
> avoid deprecation warning.
> 
> Cc: Jason Wang <jasowang@redhat.com>
> Signed-off-by: Zhao Liu <zhao1.liu@intel.com>
> ---
>  qemu-options.hx | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/qemu-options.hx b/qemu-options.hx
> index ad6521ef5e7e..edeaefe2c79a 100644
> --- a/qemu-options.hx
> +++ b/qemu-options.hx
> @@ -3353,7 +3353,7 @@ SRST
>                           -device e1000,netdev=n1,mac=52:54:00:12:34:56 \\
>                           -netdev socket,id=n1,mcast=239.192.168.1:1102,localaddr=1.2.3.4
>  
> -``-netdev l2tpv3,id=id,src=srcaddr,dst=dstaddr[,srcport=srcport][,dstport=dstport],txsession=txsession[,rxsession=rxsession][,ipv6=on|off][,udp=on|off][,cookie64][,counter][,pincounter][,txcookie=txcookie][,rxcookie=rxcookie][,offset=offset]``
> +``-netdev l2tpv3,id=id,src=srcaddr,dst=dstaddr[,srcport=srcport][,dstport=dstport],txsession=txsession[,rxsession=rxsession][,ipv6=on|off][,udp=on|off][,cookie64=on|off][,counter=on|off][,pincounter=on|off][,txcookie=txcookie][,rxcookie=rxcookie][,offset=offset]``
>      Configure a L2TPv3 pseudowire host network backend. L2TPv3 (RFC3931)
>      is a popular protocol to transport Ethernet (and other Layer 2) data
>      frames between two systems. It is present in routers, firewalls and
> @@ -3368,7 +3368,7 @@ SRST
>      ``dst=dstaddr``
>          destination address (mandatory)
>  
> -    ``udp``
> +    ``udp=on``
>          select udp encapsulation (default is ip).
>  
>      ``srcport=srcport``
> @@ -3377,7 +3377,7 @@ SRST
>      ``dstport=dstport``
>          destination udp port.
>  
> -    ``ipv6``
> +    ``ipv6=on``
>          force v6, otherwise defaults to v4.
>  
>      ``rxcookie=rxcookie``; \ ``txcookie=txcookie``
> @@ -3385,7 +3385,7 @@ SRST
>          Their function is mostly to prevent misconfiguration. By default
>          they are 32 bit.
>  
> -    ``cookie64``
> +    ``cookie64=on``
>          Set cookie size to 64 bit instead of the default 32
>  
>      ``counter=off``
> @@ -3419,7 +3419,7 @@ SRST
>          # launch QEMU instance - if your network has reorder or is very lossy add ,pincounter
>  
>          |qemu_system| linux.img -device e1000,netdev=n1 \\
> -            -netdev l2tpv3,id=n1,src=4.2.3.1,dst=1.2.3.4,udp,srcport=16384,dstport=16384,rxsession=0xffffffff,txsession=0xffffffff,counter
> +            -netdev l2tpv3,id=n1,src=4.2.3.1,dst=1.2.3.4,udp=on,srcport=16384,dstport=16384,rxsession=0xffffffff,txsession=0xffffffff,counter=on
>  
>  ``-netdev vde,id=id[,sock=socketpath][,port=n][,group=groupname][,mode=octalmode]``
>      Configure VDE backend to connect to PORT n of a vde switch running
> -- 
> 2.34.1
>
Jason Wang July 30, 2024, 2:54 a.m. UTC | #2
On Wed, Jul 17, 2024 at 2:15 PM Zhao Liu <zhao1.liu@intel.com> wrote:
>
> Hi Jason,
>
> Just a kind ping. Does this update satisfy you?
> Since the original example generates the warning.
>
> Thanks,
> Zhao

Queued.

Thanks
Zhao Liu July 30, 2024, 3:45 a.m. UTC | #3
On Tue, Jul 30, 2024 at 10:54:07AM +0800, Jason Wang wrote:
> Date: Tue, 30 Jul 2024 10:54:07 +0800
> From: Jason Wang <jasowang@redhat.com>
> Subject: Re: [PATCH] doc/net/l2tpv3: Update boolean fields' description to
>  avoid short-form use
> 
> On Wed, Jul 17, 2024 at 2:15 PM Zhao Liu <zhao1.liu@intel.com> wrote:
> >
> > Hi Jason,
> >
> > Just a kind ping. Does this update satisfy you?
> > Since the original example generates the warning.
> >
> > Thanks,
> > Zhao
> 
> Queued.
> 
> Thanks
>

Hi Jason, thank you! I noticed Michael has already helped me merge this
(commit cb8de74ac6df "doc/net/l2tpv3: Update boolean fields' description
to avoid short-form use"). 

Best Regards,
Zhao
Jason Wang July 30, 2024, 3:45 a.m. UTC | #4
On Tue, Jul 30, 2024 at 11:29 AM Zhao Liu <zhao1.liu@intel.com> wrote:
>
> On Tue, Jul 30, 2024 at 10:54:07AM +0800, Jason Wang wrote:
> > Date: Tue, 30 Jul 2024 10:54:07 +0800
> > From: Jason Wang <jasowang@redhat.com>
> > Subject: Re: [PATCH] doc/net/l2tpv3: Update boolean fields' description to
> >  avoid short-form use
> >
> > On Wed, Jul 17, 2024 at 2:15 PM Zhao Liu <zhao1.liu@intel.com> wrote:
> > >
> > > Hi Jason,
> > >
> > > Just a kind ping. Does this update satisfy you?
> > > Since the original example generates the warning.
> > >
> > > Thanks,
> > > Zhao
> >
> > Queued.
> >
> > Thanks
> >
>
> Hi Jason, thank you! I noticed Michael has already helped me merge this
> (commit cb8de74ac6df "doc/net/l2tpv3: Update boolean fields' description
> to avoid short-form use").

Great, so I dropped it.

Thanks

>
> Best Regards,
> Zhao
>
diff mbox series

Patch

diff --git a/qemu-options.hx b/qemu-options.hx
index ad6521ef5e7e..edeaefe2c79a 100644
--- a/qemu-options.hx
+++ b/qemu-options.hx
@@ -3353,7 +3353,7 @@  SRST
                          -device e1000,netdev=n1,mac=52:54:00:12:34:56 \\
                          -netdev socket,id=n1,mcast=239.192.168.1:1102,localaddr=1.2.3.4
 
-``-netdev l2tpv3,id=id,src=srcaddr,dst=dstaddr[,srcport=srcport][,dstport=dstport],txsession=txsession[,rxsession=rxsession][,ipv6=on|off][,udp=on|off][,cookie64][,counter][,pincounter][,txcookie=txcookie][,rxcookie=rxcookie][,offset=offset]``
+``-netdev l2tpv3,id=id,src=srcaddr,dst=dstaddr[,srcport=srcport][,dstport=dstport],txsession=txsession[,rxsession=rxsession][,ipv6=on|off][,udp=on|off][,cookie64=on|off][,counter=on|off][,pincounter=on|off][,txcookie=txcookie][,rxcookie=rxcookie][,offset=offset]``
     Configure a L2TPv3 pseudowire host network backend. L2TPv3 (RFC3931)
     is a popular protocol to transport Ethernet (and other Layer 2) data
     frames between two systems. It is present in routers, firewalls and
@@ -3368,7 +3368,7 @@  SRST
     ``dst=dstaddr``
         destination address (mandatory)
 
-    ``udp``
+    ``udp=on``
         select udp encapsulation (default is ip).
 
     ``srcport=srcport``
@@ -3377,7 +3377,7 @@  SRST
     ``dstport=dstport``
         destination udp port.
 
-    ``ipv6``
+    ``ipv6=on``
         force v6, otherwise defaults to v4.
 
     ``rxcookie=rxcookie``; \ ``txcookie=txcookie``
@@ -3385,7 +3385,7 @@  SRST
         Their function is mostly to prevent misconfiguration. By default
         they are 32 bit.
 
-    ``cookie64``
+    ``cookie64=on``
         Set cookie size to 64 bit instead of the default 32
 
     ``counter=off``
@@ -3419,7 +3419,7 @@  SRST
         # launch QEMU instance - if your network has reorder or is very lossy add ,pincounter
 
         |qemu_system| linux.img -device e1000,netdev=n1 \\
-            -netdev l2tpv3,id=n1,src=4.2.3.1,dst=1.2.3.4,udp,srcport=16384,dstport=16384,rxsession=0xffffffff,txsession=0xffffffff,counter
+            -netdev l2tpv3,id=n1,src=4.2.3.1,dst=1.2.3.4,udp=on,srcport=16384,dstport=16384,rxsession=0xffffffff,txsession=0xffffffff,counter=on
 
 ``-netdev vde,id=id[,sock=socketpath][,port=n][,group=groupname][,mode=octalmode]``
     Configure VDE backend to connect to PORT n of a vde switch running