All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2] typo fix in Documentation/networking/af_xdp.rst
@ 2018-10-04 17:01 Konrad Djimeli
  2018-10-04 17:38 ` Björn Töpel
  0 siblings, 1 reply; 4+ messages in thread
From: Konrad Djimeli @ 2018-10-04 17:01 UTC (permalink / raw)
  To: netdev; +Cc: Konrad Djimeli

Fix a simple typo: Completetion -> Completion

Signed-off-by: Konrad Djimeli <kdjimeli@igalia.com>
---
Changes in v2:
- Update line below to be same length as text above

 Documentation/networking/af_xdp.rst | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Documentation/networking/af_xdp.rst b/Documentation/networking/af_xdp.rst
index ff929cfab4f4..4ae4f9d8f8fe 100644
--- a/Documentation/networking/af_xdp.rst
+++ b/Documentation/networking/af_xdp.rst
@@ -159,8 +159,8 @@ log2(2048) LSB of the addr will be masked off, meaning that 2048, 2050
 and 3000 refers to the same chunk.
 
 
-UMEM Completetion Ring
-~~~~~~~~~~~~~~~~~~~~~~
+UMEM Completion Ring
+~~~~~~~~~~~~~~~~~~~~
 
 The Completion Ring is used transfer ownership of UMEM frames from
 kernel-space to user-space. Just like the Fill ring, UMEM indicies are
-- 
2.17.1

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

* Re: [PATCH v2] typo fix in Documentation/networking/af_xdp.rst
  2018-10-04 17:01 [PATCH v2] typo fix in Documentation/networking/af_xdp.rst Konrad Djimeli
@ 2018-10-04 17:38 ` Björn Töpel
  2018-10-04 22:14   ` Daniel Borkmann
  2018-10-05 21:43   ` Konrad Djimeli
  0 siblings, 2 replies; 4+ messages in thread
From: Björn Töpel @ 2018-10-04 17:38 UTC (permalink / raw)
  To: kdjimeli; +Cc: Netdev

Den tors 4 okt. 2018 kl 19:03 skrev Konrad Djimeli <kdjimeli@igalia.com>:
>
> Fix a simple typo: Completetion -> Completion
>
> Signed-off-by: Konrad Djimeli <kdjimeli@igalia.com>
> ---
> Changes in v2:
> - Update line below to be same length as text above
>
>  Documentation/networking/af_xdp.rst | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/networking/af_xdp.rst b/Documentation/networking/af_xdp.rst
> index ff929cfab4f4..4ae4f9d8f8fe 100644
> --- a/Documentation/networking/af_xdp.rst
> +++ b/Documentation/networking/af_xdp.rst
> @@ -159,8 +159,8 @@ log2(2048) LSB of the addr will be masked off, meaning that 2048, 2050
>  and 3000 refers to the same chunk.
>
>
> -UMEM Completetion Ring
> -~~~~~~~~~~~~~~~~~~~~~~
> +UMEM Completion Ring
> +~~~~~~~~~~~~~~~~~~~~
>
>  The Completion Ring is used transfer ownership of UMEM frames from
>  kernel-space to user-space. Just like the Fill ring, UMEM indicies are
> --
> 2.17.1
>

Thanks Konrad! For future patches, you should tag your patch with
'bpf' or 'bpf-next' as pointed out in
Documentation/bpf/bpf_devel_QA.rst. I guess this should go to 'bpf'.

Acked-by: Björn Töpel <bjorn.topel@intel.com>

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

* Re: [PATCH v2] typo fix in Documentation/networking/af_xdp.rst
  2018-10-04 17:38 ` Björn Töpel
@ 2018-10-04 22:14   ` Daniel Borkmann
  2018-10-05 21:43   ` Konrad Djimeli
  1 sibling, 0 replies; 4+ messages in thread
From: Daniel Borkmann @ 2018-10-04 22:14 UTC (permalink / raw)
  To: Björn Töpel, kdjimeli; +Cc: Netdev

On 10/04/2018 07:38 PM, Björn Töpel wrote:
> Den tors 4 okt. 2018 kl 19:03 skrev Konrad Djimeli <kdjimeli@igalia.com>:
>>
>> Fix a simple typo: Completetion -> Completion
>>
>> Signed-off-by: Konrad Djimeli <kdjimeli@igalia.com>
>> ---
>> Changes in v2:
>> - Update line below to be same length as text above
>>
>>  Documentation/networking/af_xdp.rst | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/Documentation/networking/af_xdp.rst b/Documentation/networking/af_xdp.rst
>> index ff929cfab4f4..4ae4f9d8f8fe 100644
>> --- a/Documentation/networking/af_xdp.rst
>> +++ b/Documentation/networking/af_xdp.rst
>> @@ -159,8 +159,8 @@ log2(2048) LSB of the addr will be masked off, meaning that 2048, 2050
>>  and 3000 refers to the same chunk.
>>
>>
>> -UMEM Completetion Ring
>> -~~~~~~~~~~~~~~~~~~~~~~
>> +UMEM Completion Ring
>> +~~~~~~~~~~~~~~~~~~~~
>>
>>  The Completion Ring is used transfer ownership of UMEM frames from
>>  kernel-space to user-space. Just like the Fill ring, UMEM indicies are
>> --
>> 2.17.1
>>
> 
> Thanks Konrad! For future patches, you should tag your patch with
> 'bpf' or 'bpf-next' as pointed out in
> Documentation/bpf/bpf_devel_QA.rst. I guess this should go to 'bpf'.
> 
> Acked-by: Björn Töpel <bjorn.topel@intel.com>

I think it's so minor that bpf-next is totally fine, therefore applied
there, thanks Konrad!

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

* Re: [PATCH v2] typo fix in Documentation/networking/af_xdp.rst
  2018-10-04 17:38 ` Björn Töpel
  2018-10-04 22:14   ` Daniel Borkmann
@ 2018-10-05 21:43   ` Konrad Djimeli
  1 sibling, 0 replies; 4+ messages in thread
From: Konrad Djimeli @ 2018-10-05 21:43 UTC (permalink / raw)
  To: Björn Töpel; +Cc: Netdev

On 2018-10-04 19:38, Björn Töpel wrote:
> Den tors 4 okt. 2018 kl 19:03 skrev Konrad Djimeli <kdjimeli@igalia.com>:
>>
>> Fix a simple typo: Completetion -> Completion
>>
>> Signed-off-by: Konrad Djimeli <kdjimeli@igalia.com>
>> ---
>> Changes in v2:
>> - Update line below to be same length as text above
>>
>>  Documentation/networking/af_xdp.rst | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/Documentation/networking/af_xdp.rst b/Documentation/networking/af_xdp.rst
>> index ff929cfab4f4..4ae4f9d8f8fe 100644
>> --- a/Documentation/networking/af_xdp.rst
>> +++ b/Documentation/networking/af_xdp.rst
>> @@ -159,8 +159,8 @@ log2(2048) LSB of the addr will be masked off, meaning that 2048, 2050
>>  and 3000 refers to the same chunk.
>>
>>
>> -UMEM Completetion Ring
>> -~~~~~~~~~~~~~~~~~~~~~~
>> +UMEM Completion Ring
>> +~~~~~~~~~~~~~~~~~~~~
>>
>>  The Completion Ring is used transfer ownership of UMEM frames from
>>  kernel-space to user-space. Just like the Fill ring, UMEM indicies are
>> --
>> 2.17.1
>>
> 
> Thanks Konrad! For future patches, you should tag your patch with
> 'bpf' or 'bpf-next' as pointed out in
> Documentation/bpf/bpf_devel_QA.rst. I guess this should go to 'bpf'.
> 
> Acked-by: Björn Töpel <bjorn.topel@intel.com>

Thanks a lot, I would keep that in mind for future contributions.

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

end of thread, other threads:[~2018-10-06  4:44 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-10-04 17:01 [PATCH v2] typo fix in Documentation/networking/af_xdp.rst Konrad Djimeli
2018-10-04 17:38 ` Björn Töpel
2018-10-04 22:14   ` Daniel Borkmann
2018-10-05 21:43   ` Konrad Djimeli

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.