diff mbox series

perf tests: Fix a typo

Message ID 20191023083324.12093-1-leo.yan@linaro.org
State Accepted
Commit b7dc21f546d854aea75245efd88d455f39f5d005
Headers show
Series perf tests: Fix a typo | expand

Commit Message

Leo Yan Oct. 23, 2019, 8:33 a.m. UTC
Correct typo in comment: s/suck/stuck.

Signed-off-by: Leo Yan <leo.yan@linaro.org>

---
 tools/perf/tests/bp_signal.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.17.1

Comments

Will Deacon Oct. 23, 2019, 9:14 a.m. UTC | #1
On Wed, Oct 23, 2019 at 04:33:24PM +0800, Leo Yan wrote:
> Correct typo in comment: s/suck/stuck.

> 

> Signed-off-by: Leo Yan <leo.yan@linaro.org>

> ---

>  tools/perf/tests/bp_signal.c | 2 +-

>  1 file changed, 1 insertion(+), 1 deletion(-)

> 

> diff --git a/tools/perf/tests/bp_signal.c b/tools/perf/tests/bp_signal.c

> index 166f411568a5..415903b48578 100644

> --- a/tools/perf/tests/bp_signal.c

> +++ b/tools/perf/tests/bp_signal.c

> @@ -295,7 +295,7 @@ bool test__bp_signal_is_supported(void)

>  	 * breakpointed instruction.

>  	 *

>  	 * Since arm64 has the same issue with arm for the single-step

> -	 * handling, this case also gets suck on the breakpointed

> +	 * handling, this case also gets stuck on the breakpointed

>  	 * instruction.

>  	 *

>  	 * Just disable the test for these architectures until these


Thanks, and sorry for only spotting this after the offending patch was
merged.

Will
Leo Yan Oct. 23, 2019, 9:28 a.m. UTC | #2
On Wed, Oct 23, 2019 at 10:14:17AM +0100, Will Deacon wrote:
> On Wed, Oct 23, 2019 at 04:33:24PM +0800, Leo Yan wrote:

> > Correct typo in comment: s/suck/stuck.

> > 

> > Signed-off-by: Leo Yan <leo.yan@linaro.org>

> > ---

> >  tools/perf/tests/bp_signal.c | 2 +-

> >  1 file changed, 1 insertion(+), 1 deletion(-)

> > 

> > diff --git a/tools/perf/tests/bp_signal.c b/tools/perf/tests/bp_signal.c

> > index 166f411568a5..415903b48578 100644

> > --- a/tools/perf/tests/bp_signal.c

> > +++ b/tools/perf/tests/bp_signal.c

> > @@ -295,7 +295,7 @@ bool test__bp_signal_is_supported(void)

> >  	 * breakpointed instruction.

> >  	 *

> >  	 * Since arm64 has the same issue with arm for the single-step

> > -	 * handling, this case also gets suck on the breakpointed

> > +	 * handling, this case also gets stuck on the breakpointed

> >  	 * instruction.

> >  	 *

> >  	 * Just disable the test for these architectures until these

> 

> Thanks, and sorry for only spotting this after the offending patch was

> merged.


No worries and thanks for review.
diff mbox series

Patch

diff --git a/tools/perf/tests/bp_signal.c b/tools/perf/tests/bp_signal.c
index 166f411568a5..415903b48578 100644
--- a/tools/perf/tests/bp_signal.c
+++ b/tools/perf/tests/bp_signal.c
@@ -295,7 +295,7 @@  bool test__bp_signal_is_supported(void)
 	 * breakpointed instruction.
 	 *
 	 * Since arm64 has the same issue with arm for the single-step
-	 * handling, this case also gets suck on the breakpointed
+	 * handling, this case also gets stuck on the breakpointed
 	 * instruction.
 	 *
 	 * Just disable the test for these architectures until these