Re: [PATCH] [trivial] perf: Spelling s/EACCESS/EACCES/

From: Ingo Molnar
Date: Mon May 27 2019 - 10:15:21 EST



* Geert Uytterhoeven <geert+renesas@xxxxxxxxx> wrote:

> The correct spelling is EACCES:
>
> include/uapi/asm-generic/errno-base.h:#define EACCES 13 /* Permission denied */
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
> ---
> include/linux/perf_event.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/linux/perf_event.h b/include/linux/perf_event.h
> index 0ab99c7b652d41b1..10569e25b5a9b656 100644
> --- a/include/linux/perf_event.h
> +++ b/include/linux/perf_event.h
> @@ -289,7 +289,7 @@ struct pmu {
> * -EBUSY -- @event is for this PMU but PMU temporarily unavailable
> * -EINVAL -- @event is for this PMU but @event is not valid
> * -EOPNOTSUPP -- @event is for this PMU, @event is valid, but not supported
> - * -EACCESS -- @event is for this PMU, @event is valid, but no privilidges
> + * -EACCES -- @event is for this PMU, @event is valid, but no privilidges
> *
> * 0 -- @event is for this PMU and valid
> *


Actually, -EACCES got typoed itself and survived due to historic reasons.
I think we can tolerate the 'typo' fixed in documentation, can we?

Also, the *far* bigger typo is, in the same line:

s/privilidges
/privileges

:-)

Thanks,

Ingo