mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH 04/14] perf: fix typos in comments
@ 2026-09-07  6:54 Hemanth Selam
  2026-09-25 13:11 ` Arnaldo Carvalho de Melo
  0 siblings, 1 reply; 2+ messages in thread
From: Hemanth Selam @ 2026-09-07  6:54 UTC (permalink / raw)
  To: Peter Zijlstra, Ingo Molnar, Arnaldo Carvalho de Melo,
	Namhyung Kim, Mark Rutland, Alexander Shishkin, Jiri Olsa,
	Ian Rogers
  Cc: linux-perf-users, linux-kernel

Fix typos in comments, reported by scripts/checkpatch.pl using the
misspelling list in scripts/spelling.txt.  Only touches comments, no code
changes.

Assisted-by: Cursor:claude-opus-5
Signed-off-by: Hemanth Selam <hemanth.selam@gmail.com>
---
 tools/perf/builtin-trace.c                      | 2 +-
 tools/perf/trace/beauty/include/uapi/linux/fs.h | 2 +-
 tools/perf/trace/beauty/syscalltbl.sh           | 2 +-
 tools/perf/ui/hist.c                            | 2 +-
 tools/perf/util/annotate.h                      | 2 +-
 5 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/tools/perf/builtin-trace.c b/tools/perf/builtin-trace.c
index c3c7f1f85c53..de86901b178b 100644
--- a/tools/perf/builtin-trace.c
+++ b/tools/perf/builtin-trace.c
@@ -104,7 +104,7 @@
 /*
  * strtoul: Go from a string to a value, i.e. for msr: MSR_FS_BASE to 0xc0000100
  *
- * We have to explicitely mark the direction of the flow of data, if from the
+ * We have to explicitly mark the direction of the flow of data, if from the
  * kernel to user space or the other way around, since the BPF collector we
  * have so far copies only from user to kernel space, mark the arguments that
  * go that direction, so that we don´t end up collecting the previous contents
diff --git a/tools/perf/trace/beauty/include/uapi/linux/fs.h b/tools/perf/trace/beauty/include/uapi/linux/fs.h
index bd87262f2e34..ee2203d60643 100644
--- a/tools/perf/trace/beauty/include/uapi/linux/fs.h
+++ b/tools/perf/trace/beauty/include/uapi/linux/fs.h
@@ -88,7 +88,7 @@ struct fstrim_range {
  * We include a length field because some filesystems (vfat) have an identifier
  * that we do want to expose as a UUID, but doesn't have the standard length.
  *
- * We use a fixed size buffer beacuse this interface will, by fiat, never
+ * We use a fixed size buffer because this interface will, by fiat, never
  * support "UUIDs" longer than 16 bytes; we don't want to force all downstream
  * users to have to deal with that.
  */
diff --git a/tools/perf/trace/beauty/syscalltbl.sh b/tools/perf/trace/beauty/syscalltbl.sh
index 1199618dc178..e70a8c79e019 100755
--- a/tools/perf/trace/beauty/syscalltbl.sh
+++ b/tools/perf/trace/beauty/syscalltbl.sh
@@ -55,7 +55,7 @@ build_tables() {
 
 	echo "static const uint16_t syscall_sorted_names_${e_machine}[] = {" >> "$outfile"
 
-	# When sorting by name, add a suffix of 0s upto 20 characters so that
+	# When sorting by name, add a suffix of 0s up to 20 characters so that
 	# system calls that differ with a numerical suffix don't sort before
 	# those without. This default behavior of sort differs from that of
 	# strcmp used at runtime. Use sed to strip the trailing 0s suffix
diff --git a/tools/perf/ui/hist.c b/tools/perf/ui/hist.c
index e58327595d37..dcc535cc991c 100644
--- a/tools/perf/ui/hist.c
+++ b/tools/perf/ui/hist.c
@@ -431,7 +431,7 @@ static int hpp__header_mem_stat_fn(struct perf_hpp_fmt *fmt, struct perf_hpp *hp
 		int left, right;
 
 		len = 0;
-		/* update fmt->len for acutally used columns only */
+		/* update fmt->len for actually used columns only */
 		for (int i = 0; i < MEM_STAT_LEN; i++) {
 			if (hists->mem_stat_total[mem_stat_idx].entries[i])
 				len += MEM_STAT_PRINT_LEN;
diff --git a/tools/perf/util/annotate.h b/tools/perf/util/annotate.h
index fa08d09b80f7..c2a0a5cc4af9 100644
--- a/tools/perf/util/annotate.h
+++ b/tools/perf/util/annotate.h
@@ -275,7 +275,7 @@ struct cyc_hist {
 /**
  * struct annotated_source - symbols with hits have this attached as in annotation
  *
- * @source: List head for annotated_line (embeded in disasm_line).
+ * @source: List head for annotated_line (embedded in disasm_line).
  * @histograms: Array of symbol histograms per event to maintain the total number
  * 		of samples and period.
  * @nr_histograms: This may not be the same as evsel->evlist->core.nr_entries if

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

* Re: [PATCH 04/14] perf: fix typos in comments
  2026-09-07  6:54 [PATCH 04/14] perf: fix typos in comments Hemanth Selam
@ 2026-09-25 13:11 ` Arnaldo Carvalho de Melo
  0 siblings, 0 replies; 2+ messages in thread
From: Arnaldo Carvalho de Melo @ 2026-09-25 13:11 UTC (permalink / raw)
  To: Hemanth Selam
  Cc: Peter Zijlstra, Ingo Molnar, Namhyung Kim, Mark Rutland,
	Alexander Shishkin, Jiri Olsa, Ian Rogers, linux-perf-users,
	linux-kernel

On Mon, Sep 07, 2026 at 12:24:21PM +0530, Hemanth Selam wrote:
> Fix typos in comments, reported by scripts/checkpatch.pl using the
> misspelling list in scripts/spelling.txt.  Only touches comments, no code
> changes.

Thanks, applied to perf-tools-next, for v7.4.

- Arnaldo
 
> Assisted-by: Cursor:claude-opus-5
> Signed-off-by: Hemanth Selam <hemanth.selam@gmail.com>
> ---
>  tools/perf/builtin-trace.c                      | 2 +-
>  tools/perf/trace/beauty/include/uapi/linux/fs.h | 2 +-
>  tools/perf/trace/beauty/syscalltbl.sh           | 2 +-
>  tools/perf/ui/hist.c                            | 2 +-
>  tools/perf/util/annotate.h                      | 2 +-
>  5 files changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/tools/perf/builtin-trace.c b/tools/perf/builtin-trace.c
> index c3c7f1f85c53..de86901b178b 100644
> --- a/tools/perf/builtin-trace.c
> +++ b/tools/perf/builtin-trace.c
> @@ -104,7 +104,7 @@
>  /*
>   * strtoul: Go from a string to a value, i.e. for msr: MSR_FS_BASE to 0xc0000100
>   *
> - * We have to explicitely mark the direction of the flow of data, if from the
> + * We have to explicitly mark the direction of the flow of data, if from the
>   * kernel to user space or the other way around, since the BPF collector we
>   * have so far copies only from user to kernel space, mark the arguments that
>   * go that direction, so that we don´t end up collecting the previous contents
> diff --git a/tools/perf/trace/beauty/include/uapi/linux/fs.h b/tools/perf/trace/beauty/include/uapi/linux/fs.h
> index bd87262f2e34..ee2203d60643 100644
> --- a/tools/perf/trace/beauty/include/uapi/linux/fs.h
> +++ b/tools/perf/trace/beauty/include/uapi/linux/fs.h
> @@ -88,7 +88,7 @@ struct fstrim_range {
>   * We include a length field because some filesystems (vfat) have an identifier
>   * that we do want to expose as a UUID, but doesn't have the standard length.
>   *
> - * We use a fixed size buffer beacuse this interface will, by fiat, never
> + * We use a fixed size buffer because this interface will, by fiat, never
>   * support "UUIDs" longer than 16 bytes; we don't want to force all downstream
>   * users to have to deal with that.
>   */
> diff --git a/tools/perf/trace/beauty/syscalltbl.sh b/tools/perf/trace/beauty/syscalltbl.sh
> index 1199618dc178..e70a8c79e019 100755
> --- a/tools/perf/trace/beauty/syscalltbl.sh
> +++ b/tools/perf/trace/beauty/syscalltbl.sh
> @@ -55,7 +55,7 @@ build_tables() {
>  
>  	echo "static const uint16_t syscall_sorted_names_${e_machine}[] = {" >> "$outfile"
>  
> -	# When sorting by name, add a suffix of 0s upto 20 characters so that
> +	# When sorting by name, add a suffix of 0s up to 20 characters so that
>  	# system calls that differ with a numerical suffix don't sort before
>  	# those without. This default behavior of sort differs from that of
>  	# strcmp used at runtime. Use sed to strip the trailing 0s suffix
> diff --git a/tools/perf/ui/hist.c b/tools/perf/ui/hist.c
> index e58327595d37..dcc535cc991c 100644
> --- a/tools/perf/ui/hist.c
> +++ b/tools/perf/ui/hist.c
> @@ -431,7 +431,7 @@ static int hpp__header_mem_stat_fn(struct perf_hpp_fmt *fmt, struct perf_hpp *hp
>  		int left, right;
>  
>  		len = 0;
> -		/* update fmt->len for acutally used columns only */
> +		/* update fmt->len for actually used columns only */
>  		for (int i = 0; i < MEM_STAT_LEN; i++) {
>  			if (hists->mem_stat_total[mem_stat_idx].entries[i])
>  				len += MEM_STAT_PRINT_LEN;
> diff --git a/tools/perf/util/annotate.h b/tools/perf/util/annotate.h
> index fa08d09b80f7..c2a0a5cc4af9 100644
> --- a/tools/perf/util/annotate.h
> +++ b/tools/perf/util/annotate.h
> @@ -275,7 +275,7 @@ struct cyc_hist {
>  /**
>   * struct annotated_source - symbols with hits have this attached as in annotation
>   *
> - * @source: List head for annotated_line (embeded in disasm_line).
> + * @source: List head for annotated_line (embedded in disasm_line).
>   * @histograms: Array of symbol histograms per event to maintain the total number
>   * 		of samples and period.
>   * @nr_histograms: This may not be the same as evsel->evlist->core.nr_entries if

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

end of thread, other threads:[~2026-09-25 13:11 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-09-07  6:54 [PATCH 04/14] perf: fix typos in comments Hemanth Selam
2026-09-25 13:11 ` Arnaldo Carvalho de Melo

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®