Skip to contentSkip to conversation

perf: fix typos in comments

PATCH Applied Hemanth Selam 2 participants 2 messages Sep 7 – Sep 25 perf events

v1 of 6 Applied 1/14 Latest is v6 →

lore
Share & tools
lore
#0 lore raw

PATCH 4/14 perf: fix typos in comments

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
---
 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(-)
tools/perf/builtin-trace.c +1 -1
index c3c7f1f85c53..de86901b178b 100644
@@ -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
tools/perf/trace/beauty/include/uapi/linux/fs.h +1 -1
index bd87262f2e34..ee2203d60643 100644
@@ -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.
  */
tools/perf/trace/beauty/syscalltbl.sh +1 -1
index 1199618dc178..e70a8c79e019 100755
@@ -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
tools/perf/ui/hist.c +1 -1
index e58327595d37..dcc535cc991c 100644
@@ -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;
tools/perf/util/annotate.h +1 -1
index fa08d09b80f7..c2a0a5cc4af9 100644
@@ -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
maintainer Applied
#1 lore raw
Quoted 3 lines · Hemanth Selam

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

Quoted 75 lines
Assisted-by Cursor:claude-opus-5
---
 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(-)
tools/perf/builtin-trace.c +1 -1
index c3c7f1f85c53..de86901b178b 100644
@@ -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
tools/perf/trace/beauty/include/uapi/linux/fs.h +1 -1
index bd87262f2e34..ee2203d60643 100644
@@ -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.
  */
tools/perf/trace/beauty/syscalltbl.sh +1 -1
index 1199618dc178..e70a8c79e019 100755
@@ -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
tools/perf/ui/hist.c +1 -1
index e58327595d37..dcc535cc991c 100644
@@ -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;
tools/perf/util/annotate.h +1 -1
index fa08d09b80f7..c2a0a5cc4af9 100644
@@ -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