Re: [PATCH v2] [trivial] perf: Spelling s/EACCESS/EACCES/, s/privilidge/privilege/

From: Mark Rutland
Date: Thu Oct 24 2019 - 08:31:27 EST


On Thu, Oct 24, 2019 at 02:29:04PM +0200, Geert Uytterhoeven wrote:
> As per POSIX, the correct spelling of the error code is EACCES:
>
> include/uapi/asm-generic/errno-base.h:#define EACCES 13 /* Permission denied */
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>

Looks sane to me; brings the documentation into line with reality given
EACCESS doesn't exist in tree.

Acked-by: Mark Rutland <mark.rutland@xxxxxxx>

Mark.

> ---
> v2:
> - Add POSIX reference,
> - Also correct privilidges in the same line.
> ---
> 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 61448c19a132c29c..68ccc5b1913b485b 100644
> --- a/include/linux/perf_event.h
> +++ b/include/linux/perf_event.h
> @@ -292,7 +292,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 privileges
> *
> * 0 -- @event is for this PMU and valid
> *
> --
> 2.17.1
>