Received-SPF: pass (google.com: domain of linux-kernel+bounces-108064-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Precedence: bulk
MIME-Version: 1.0
References: <20240319180005.246930-1-visitorckw@gmail.com> <20240319180005.246930-2-visitorckw@gmail.com>
In-Reply-To: <20240319180005.246930-2-visitorckw@gmail.com>
From: Ian Rogers <irogers@google.com>
Date: Tue, 19 Mar 2024 12:31:12 -0700
Message-ID: <CAP-5=fW=Mky3nkYerzwqgnrL3UJ1uONccyxuJ2aoD+ueY+kt-A@mail.gmail.com>
Subject: Re: [PATCH 01/13] perf/core: Fix several typos
To: Kuan-Wei Chiu <visitorckw@gmail.com>
Cc: colyli@suse.de, kent.overstreet@linux.dev, msakai@redhat.com, 
	peterz@infradead.org, mingo@redhat.com, acme@kernel.org, namhyung@kernel.org, 
	akpm@linux-foundation.org, bfoster@redhat.com, mark.rutland@arm.com, 
	alexander.shishkin@linux.intel.com, jolsa@kernel.org, adrian.hunter@intel.com, 
	jserv@ccns.ncku.edu.tw, linux-bcache@vger.kernel.org, 
	dm-devel@lists.linux.dev, linux-bcachefs@vger.kernel.org, 
	linux-perf-users@vger.kernel.org, linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Tue, Mar 19, 2024 at 11:00=E2=80=AFAM Kuan-Wei Chiu <visitorckw@gmail.co=
m> wrote:
>
> Replace 'artifically' with 'artificially'.
> Replace 'irrespecive' with 'irrespective'.
> Replace 'futher' with 'further'.
> Replace 'sufficent' with 'sufficient'.
>
> Signed-off-by: Kuan-Wei Chiu <visitorckw@gmail.com>

Reviewed-by: Ian Rogers <irogers@google.com>

Thanks,
Ian

> ---
>  kernel/events/core.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/kernel/events/core.c b/kernel/events/core.c
> index 724e6d7e128f..10ac2db83f14 100644
> --- a/kernel/events/core.c
> +++ b/kernel/events/core.c
> @@ -534,7 +534,7 @@ void perf_sample_event_took(u64 sample_len_ns)
>         __this_cpu_write(running_sample_length, running_len);
>
>         /*
> -        * Note: this will be biased artifically low until we have
> +        * Note: this will be biased artificially low until we have
>          * seen NR_ACCUMULATED_SAMPLES. Doing it this way keeps us
>          * from having to maintain a count.
>          */
> @@ -596,10 +596,10 @@ static inline u64 perf_event_clock(struct perf_even=
t *event)
>   *
>   * Event groups make things a little more complicated, but not terribly =
so. The
>   * rules for a group are that if the group leader is OFF the entire grou=
p is
> - * OFF, irrespecive of what the group member states are. This results in
> + * OFF, irrespective of what the group member states are. This results i=
n
>   * __perf_effective_state().
>   *
> - * A futher ramification is that when a group leader flips between OFF a=
nd
> + * A further ramification is that when a group leader flips between OFF =
and
>   * !OFF, we need to update all group member times.
>   *
>   *
> @@ -891,7 +891,7 @@ static int perf_cgroup_ensure_storage(struct perf_eve=
nt *event,
>         int cpu, heap_size, ret =3D 0;
>
>         /*
> -        * Allow storage to have sufficent space for an iterator for each
> +        * Allow storage to have sufficient space for an iterator for eac=
h
>          * possibly nested cgroup plus an iterator for events with no cgr=
oup.
>          */
>         for (heap_size =3D 1; css; css =3D css->parent)
> --
> 2.34.1
>