qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] qemu-options.hx: Fix typo for netdev documentation
@ 2020-07-27  4:59 Tianjia Zhang
  2020-07-27  5:46 ` Thomas Huth
                   ` (2 more replies)
  0 siblings, 3 replies; 8+ messages in thread
From: Tianjia Zhang @ 2020-07-27  4:59 UTC (permalink / raw)
  To: peter.maydell, pbonzini, mst, kraxel, thuth, qemu-devel; +Cc: tianjia.zhang

This patch fixes the netdev document description typo in qemu-option.hx.

Signed-off-by: Tianjia Zhang <tianjia.zhang@linux.alibaba.com>
---
 qemu-options.hx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/qemu-options.hx b/qemu-options.hx
index 708583b4ce..92556ed96d 100644
--- a/qemu-options.hx
+++ b/qemu-options.hx
@@ -2684,7 +2684,7 @@ SRST
     disable script execution.
 
     If running QEMU as an unprivileged user, use the network helper
-    helper to configure the TAP interface and attach it to the bridge.
+    to configure the TAP interface and attach it to the bridge.
     The default network helper executable is
     ``/path/to/qemu-bridge-helper`` and the default bridge device is
     ``br0``.
-- 
2.17.1



^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [PATCH] qemu-options.hx: Fix typo for netdev documentation
  2020-07-27  4:59 [PATCH] qemu-options.hx: Fix typo for netdev documentation Tianjia Zhang
@ 2020-07-27  5:46 ` Thomas Huth
  2020-07-27  5:46 ` Li Qiang
  2020-07-27  9:44 ` Michael S. Tsirkin
  2 siblings, 0 replies; 8+ messages in thread
From: Thomas Huth @ 2020-07-27  5:46 UTC (permalink / raw)
  To: Tianjia Zhang, qemu-devel, Jason Wang, QEMU Trivial
  Cc: peter.maydell, mst, kraxel, pbonzini

On 27/07/2020 06.59, Tianjia Zhang wrote:
> This patch fixes the netdev document description typo in qemu-option.hx.
> 
> Signed-off-by: Tianjia Zhang <tianjia.zhang@linux.alibaba.com>
> ---
>  qemu-options.hx | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/qemu-options.hx b/qemu-options.hx
> index 708583b4ce..92556ed96d 100644
> --- a/qemu-options.hx
> +++ b/qemu-options.hx
> @@ -2684,7 +2684,7 @@ SRST
>      disable script execution.
>  
>      If running QEMU as an unprivileged user, use the network helper
> -    helper to configure the TAP interface and attach it to the bridge.
> +    to configure the TAP interface and attach it to the bridge.
>      The default network helper executable is
>      ``/path/to/qemu-bridge-helper`` and the default bridge device is
>      ``br0``.

Reviewed-by: Thomas Huth <thuth@redhat.com>



^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] qemu-options.hx: Fix typo for netdev documentation
  2020-07-27  4:59 [PATCH] qemu-options.hx: Fix typo for netdev documentation Tianjia Zhang
  2020-07-27  5:46 ` Thomas Huth
@ 2020-07-27  5:46 ` Li Qiang
  2020-07-27  9:44 ` Michael S. Tsirkin
  2 siblings, 0 replies; 8+ messages in thread
From: Li Qiang @ 2020-07-27  5:46 UTC (permalink / raw)
  To: Tianjia Zhang
  Cc: Peter Maydell, Thomas Huth, Michael S. Tsirkin, Qemu Developers,
	Gerd Hoffmann, Paolo Bonzini

Tianjia Zhang <tianjia.zhang@linux.alibaba.com> 于2020年7月27日周一 下午1:00写道:
>
> This patch fixes the netdev document description typo in qemu-option.hx.
>
> Signed-off-by: Tianjia Zhang <tianjia.zhang@linux.alibaba.com>

Reviewed-by: Li Qiang <liq3ea@gmail.com>

> ---
>  qemu-options.hx | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/qemu-options.hx b/qemu-options.hx
> index 708583b4ce..92556ed96d 100644
> --- a/qemu-options.hx
> +++ b/qemu-options.hx
> @@ -2684,7 +2684,7 @@ SRST
>      disable script execution.
>
>      If running QEMU as an unprivileged user, use the network helper
> -    helper to configure the TAP interface and attach it to the bridge.
> +    to configure the TAP interface and attach it to the bridge.
>      The default network helper executable is
>      ``/path/to/qemu-bridge-helper`` and the default bridge device is
>      ``br0``.
> --
> 2.17.1
>
>


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] qemu-options.hx: Fix typo for netdev documentation
  2020-07-27  4:59 [PATCH] qemu-options.hx: Fix typo for netdev documentation Tianjia Zhang
  2020-07-27  5:46 ` Thomas Huth
  2020-07-27  5:46 ` Li Qiang
@ 2020-07-27  9:44 ` Michael S. Tsirkin
  2020-07-28 12:28   ` Tianjia Zhang
  2020-08-27 13:00   ` Tianjia Zhang
  2 siblings, 2 replies; 8+ messages in thread
From: Michael S. Tsirkin @ 2020-07-27  9:44 UTC (permalink / raw)
  To: Tianjia Zhang
  Cc: peter.maydell, thuth, Jason Wang, qemu-devel, kraxel, pbonzini

On Mon, Jul 27, 2020 at 12:59:25PM +0800, Tianjia Zhang wrote:
> This patch fixes the netdev document description typo in qemu-option.hx.
> 
> Signed-off-by: Tianjia Zhang <tianjia.zhang@linux.alibaba.com>

Reviewed-by: Michael S. Tsirkin <mst@redhat.com>

Trivial tree? Jason's ?

> ---
>  qemu-options.hx | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/qemu-options.hx b/qemu-options.hx
> index 708583b4ce..92556ed96d 100644
> --- a/qemu-options.hx
> +++ b/qemu-options.hx
> @@ -2684,7 +2684,7 @@ SRST
>      disable script execution.
>  
>      If running QEMU as an unprivileged user, use the network helper
> -    helper to configure the TAP interface and attach it to the bridge.
> +    to configure the TAP interface and attach it to the bridge.
>      The default network helper executable is
>      ``/path/to/qemu-bridge-helper`` and the default bridge device is
>      ``br0``.
> -- 
> 2.17.1



^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] qemu-options.hx: Fix typo for netdev documentation
  2020-07-27  9:44 ` Michael S. Tsirkin
@ 2020-07-28 12:28   ` Tianjia Zhang
  2020-08-27 13:00   ` Tianjia Zhang
  1 sibling, 0 replies; 8+ messages in thread
From: Tianjia Zhang @ 2020-07-28 12:28 UTC (permalink / raw)
  To: Michael S. Tsirkin
  Cc: peter.maydell, thuth, Jason Wang, qemu-devel, kraxel, pbonzini



On 2020/7/27 17:44, Michael S. Tsirkin wrote:
> On Mon, Jul 27, 2020 at 12:59:25PM +0800, Tianjia Zhang wrote:
>> This patch fixes the netdev document description typo in qemu-option.hx.
>>
>> Signed-off-by: Tianjia Zhang <tianjia.zhang@linux.alibaba.com>
> 
> Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
> 
> Trivial tree? Jason's ?
> 

It's for mainline tree.

Thanks
>> ---
>>   qemu-options.hx | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/qemu-options.hx b/qemu-options.hx
>> index 708583b4ce..92556ed96d 100644
>> --- a/qemu-options.hx
>> +++ b/qemu-options.hx
>> @@ -2684,7 +2684,7 @@ SRST
>>       disable script execution.
>>   
>>       If running QEMU as an unprivileged user, use the network helper
>> -    helper to configure the TAP interface and attach it to the bridge.
>> +    to configure the TAP interface and attach it to the bridge.
>>       The default network helper executable is
>>       ``/path/to/qemu-bridge-helper`` and the default bridge device is
>>       ``br0``.
>> -- 
>> 2.17.1


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] qemu-options.hx: Fix typo for netdev documentation
  2020-07-27  9:44 ` Michael S. Tsirkin
  2020-07-28 12:28   ` Tianjia Zhang
@ 2020-08-27 13:00   ` Tianjia Zhang
  2020-08-27 13:03     ` Peter Maydell
  1 sibling, 1 reply; 8+ messages in thread
From: Tianjia Zhang @ 2020-08-27 13:00 UTC (permalink / raw)
  To: Michael S. Tsirkin
  Cc: peter.maydell, thuth, Jason Wang, qemu-devel, kraxel, pbonzini

Hi,

Other opinions needed?


Thanks,

Tianjia

On 7/27/20 5:44 PM, Michael S. Tsirkin wrote:
> On Mon, Jul 27, 2020 at 12:59:25PM +0800, Tianjia Zhang wrote:
>> This patch fixes the netdev document description typo in qemu-option.hx.
>>
>> Signed-off-by: Tianjia Zhang <tianjia.zhang@linux.alibaba.com>
> Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
>
> Trivial tree? Jason's ?
>
>> ---
>>   qemu-options.hx | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/qemu-options.hx b/qemu-options.hx
>> index 708583b4ce..92556ed96d 100644
>> --- a/qemu-options.hx
>> +++ b/qemu-options.hx
>> @@ -2684,7 +2684,7 @@ SRST
>>       disable script execution.
>>   
>>       If running QEMU as an unprivileged user, use the network helper
>> -    helper to configure the TAP interface and attach it to the bridge.
>> +    to configure the TAP interface and attach it to the bridge.
>>       The default network helper executable is
>>       ``/path/to/qemu-bridge-helper`` and the default bridge device is
>>       ``br0``.
>> -- 
>> 2.17.1


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] qemu-options.hx: Fix typo for netdev documentation
  2020-08-27 13:00   ` Tianjia Zhang
@ 2020-08-27 13:03     ` Peter Maydell
  2020-09-01  7:19       ` Laurent Vivier
  0 siblings, 1 reply; 8+ messages in thread
From: Peter Maydell @ 2020-08-27 13:03 UTC (permalink / raw)
  To: Tianjia Zhang
  Cc: Thomas Huth, Michael S. Tsirkin, Jason Wang, QEMU Developers,
	Gerd Hoffmann, Paolo Bonzini

On Thu, 27 Aug 2020 at 14:00, Tianjia Zhang
<tianjia.zhang@linux.alibaba.com> wrote:
> Other opinions needed?

No, the patch just got lost by accident; sorry.

Jason, could you take this via your net tree, please?

thanks
-- PMM

>
> On 7/27/20 5:44 PM, Michael S. Tsirkin wrote:
> > On Mon, Jul 27, 2020 at 12:59:25PM +0800, Tianjia Zhang wrote:
> >> This patch fixes the netdev document description typo in qemu-option.hx.
> >>
> >> Signed-off-by: Tianjia Zhang <tianjia.zhang@linux.alibaba.com>
> > Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
> >
> > Trivial tree? Jason's ?
> >
> >> ---
> >>   qemu-options.hx | 2 +-
> >>   1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/qemu-options.hx b/qemu-options.hx
> >> index 708583b4ce..92556ed96d 100644
> >> --- a/qemu-options.hx
> >> +++ b/qemu-options.hx
> >> @@ -2684,7 +2684,7 @@ SRST
> >>       disable script execution.
> >>
> >>       If running QEMU as an unprivileged user, use the network helper
> >> -    helper to configure the TAP interface and attach it to the bridge.
> >> +    to configure the TAP interface and attach it to the bridge.
> >>       The default network helper executable is
> >>       ``/path/to/qemu-bridge-helper`` and the default bridge device is
> >>       ``br0``.
> >> --
> >> 2.17.1


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] qemu-options.hx: Fix typo for netdev documentation
  2020-08-27 13:03     ` Peter Maydell
@ 2020-09-01  7:19       ` Laurent Vivier
  0 siblings, 0 replies; 8+ messages in thread
From: Laurent Vivier @ 2020-09-01  7:19 UTC (permalink / raw)
  To: Peter Maydell, Tianjia Zhang
  Cc: Thomas Huth, Michael S. Tsirkin, qemu-trivial@nongnu.org,
	Jason Wang, QEMU Developers, Gerd Hoffmann, Paolo Bonzini

Le 27/08/2020 à 15:03, Peter Maydell a écrit :
> On Thu, 27 Aug 2020 at 14:00, Tianjia Zhang
> <tianjia.zhang@linux.alibaba.com> wrote:
>> Other opinions needed?
> 
> No, the patch just got lost by accident; sorry.
> 
> Jason, could you take this via your net tree, please?
> 
> thanks
> -- PMM
> 
>>
>> On 7/27/20 5:44 PM, Michael S. Tsirkin wrote:
>>> On Mon, Jul 27, 2020 at 12:59:25PM +0800, Tianjia Zhang wrote:
>>>> This patch fixes the netdev document description typo in qemu-option.hx.
>>>>
>>>> Signed-off-by: Tianjia Zhang <tianjia.zhang@linux.alibaba.com>
>>> Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
>>>
>>> Trivial tree? Jason's ?

I'm taking it via the trivial tree.

Thanks,
Laurent


^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2020-09-01  7:19 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-07-27  4:59 [PATCH] qemu-options.hx: Fix typo for netdev documentation Tianjia Zhang
2020-07-27  5:46 ` Thomas Huth
2020-07-27  5:46 ` Li Qiang
2020-07-27  9:44 ` Michael S. Tsirkin
2020-07-28 12:28   ` Tianjia Zhang
2020-08-27 13:00   ` Tianjia Zhang
2020-08-27 13:03     ` Peter Maydell
2020-09-01  7:19       ` Laurent Vivier

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).