Date:   Tue, 8 Jun 2021 00:19:02 +0300
From:   Andrey Konovalov <andreyknvl@...il.com>
To:     Zhen Lei <thunder.leizhen@...wei.com>
Cc:     Alexei Starovoitov <ast@...nel.org>,
        Daniel Borkmann <daniel@...earbox.net>,
        Andrii Nakryiko <andrii@...nel.org>,
        Martin KaFai Lau <kafai@...com>,
        Song Liu <songliubraving@...com>, Yonghong Song <yhs@...com>,
        John Fastabend <john.fastabend@...il.com>,
        KP Singh <kpsingh@...nel.org>,
        Andrey Ryabinin <ryabinin.a.a@...il.com>,
        Alexander Potapenko <glider@...gle.com>,
        Dmitry Vyukov <dvyukov@...gle.com>,
        Luis Chamberlain <mcgrof@...nel.org>,
        Petr Mladek <pmladek@...e.com>,
        Steven Rostedt <rostedt@...dmis.org>,
        Sergey Senozhatsky <senozhatsky@...omium.org>,
        Andy Shevchenko <andriy.shevchenko@...ux.intel.com>,
        Rasmus Villemoes <linux@...musvillemoes.dk>,
        Andrew Morton <akpm@...ux-foundation.org>,
        netdev <netdev@...r.kernel.org>, bpf <bpf@...r.kernel.org>,
        kasan-dev <kasan-dev@...glegroups.com>,
        linux-kernel <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 1/1] lib/test: Fix spelling mistakes

On Mon, Jun 7, 2021 at 6:18 AM Zhen Lei <thunder.leizhen@...wei.com> wrote:
>
> Fix some spelling mistakes in comments:
> thats ==> that's
> unitialized ==> uninitialized
> panicing ==> panicking
> sucess ==> success
> possitive ==> positive
> intepreted ==> interpreted
>
> Signed-off-by: Zhen Lei <thunder.leizhen@...wei.com>
> ---
>  lib/test_bitops.c | 2 +-
>  lib/test_bpf.c    | 2 +-
>  lib/test_kasan.c  | 2 +-
>  lib/test_kmod.c   | 6 +++---
>  lib/test_scanf.c  | 2 +-
>  5 files changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/lib/test_bitops.c b/lib/test_bitops.c
> index 471141ddd691..3b7bcbee84db 100644
> --- a/lib/test_bitops.c
> +++ b/lib/test_bitops.c
> @@ -15,7 +15,7 @@
>   *   get_count_order/long
>   */
>
> -/* use an enum because thats the most common BITMAP usage */
> +/* use an enum because that's the most common BITMAP usage */
>  enum bitops_fun {
>         BITOPS_4 = 4,
>         BITOPS_7 = 7,
> diff --git a/lib/test_bpf.c b/lib/test_bpf.c
> index 4dc4dcbecd12..d500320778c7 100644
> --- a/lib/test_bpf.c
> +++ b/lib/test_bpf.c
> @@ -1095,7 +1095,7 @@ static struct bpf_test tests[] = {
>         {
>                 "RET_A",
>                 .u.insns = {
> -                       /* check that unitialized X and A contain zeros */
> +                       /* check that uninitialized X and A contain zeros */
>                         BPF_STMT(BPF_MISC | BPF_TXA, 0),
>                         BPF_STMT(BPF_RET | BPF_A, 0)
>                 },
> diff --git a/lib/test_kasan.c b/lib/test_kasan.c
> index cacbbbdef768..72b8e808c39c 100644
> --- a/lib/test_kasan.c
> +++ b/lib/test_kasan.c
> @@ -656,7 +656,7 @@ static void kasan_global_oob(struct kunit *test)
>  {
>         /*
>          * Deliberate out-of-bounds access. To prevent CONFIG_UBSAN_LOCAL_BOUNDS
> -        * from failing here and panicing the kernel, access the array via a
> +        * from failing here and panicking the kernel, access the array via a
>          * volatile pointer, which will prevent the compiler from being able to
>          * determine the array bounds.
>          *
> diff --git a/lib/test_kmod.c b/lib/test_kmod.c
> index 38c250fbace3..ce1589391413 100644
> --- a/lib/test_kmod.c
> +++ b/lib/test_kmod.c
> @@ -286,7 +286,7 @@ static int tally_work_test(struct kmod_test_device_info *info)
>   * If this ran it means *all* tasks were created fine and we
>   * are now just collecting results.
>   *
> - * Only propagate errors, do not override with a subsequent sucess case.
> + * Only propagate errors, do not override with a subsequent success case.
>   */
>  static void tally_up_work(struct kmod_test_device *test_dev)
>  {
> @@ -543,7 +543,7 @@ static int trigger_config_run(struct kmod_test_device *test_dev)
>          * wrong with the setup of the test. If the test setup went fine
>          * then userspace must just check the result of config->test_result.
>          * One issue with relying on the return from a call in the kernel
> -        * is if the kernel returns a possitive value using this trigger
> +        * is if the kernel returns a positive value using this trigger
>          * will not return the value to userspace, it would be lost.
>          *
>          * By not relying on capturing the return value of tests we are using
> @@ -585,7 +585,7 @@ trigger_config_store(struct device *dev,
>          * Note: any return > 0 will be treated as success
>          * and the error value will not be available to userspace.
>          * Do not rely on trying to send to userspace a test value
> -        * return value as possitive return errors will be lost.
> +        * return value as positive return errors will be lost.
>          */
>         if (WARN_ON(ret > 0))
>                 return -EINVAL;
> diff --git a/lib/test_scanf.c b/lib/test_scanf.c
> index 48ff5747a4da..84fe09eaf55e 100644
> --- a/lib/test_scanf.c
> +++ b/lib/test_scanf.c
> @@ -600,7 +600,7 @@ static void __init numbers_prefix_overflow(void)
>         /*
>          * 0x prefix in a field of width 2 using %i conversion: first field
>          * converts to 0. Next field scan starts at the character after "0x",
> -        * which will convert if can be intepreted as decimal but will fail
> +        * which will convert if can be interpreted as decimal but will fail
>          * if it contains any hex digits (since no 0x prefix).
>          */
>         test_number_prefix(long long,   "0x67", "%2lli%lli", 0, 67, 2, check_ll);
> --
> 2.25.1

For lib/test_kasan.c:

Acked-by: Andrey Konovalov <andreyknvl@...il.com>

Powered by blists - more mailing lists