[v3,1/4] test: fix typo in print statement

Message ID 1557349328-25783-1-git-send-email-drc@linux.vnet.ibm.com (mailing list archive)
State Accepted, archived
Delegated to: Thomas Monjalon
Headers
Series [v3,1/4] test: fix typo in print statement |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/intel-Performance-Testing success Performance Testing PASS
ci/mellanox-Performance-Testing success Performance Testing PASS
ci/Intel-compilation success Compilation OK

Commit Message

David Christensen May 8, 2019, 9:02 p.m. UTC
  Change "much" to "match" in a printf.

Signed-off-by: David Christensen <drc@linux.vnet.ibm.com>
---
v2:
* None
v3:
* None

 app/test/test_barrier.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Thomas Monjalon May 8, 2019, 9:24 p.m. UTC | #1
Hi David,

08/05/2019 23:02, David Christensen:
> v2:
> * None
> v3:
> * None

It seems your emails are not threaded at all.
It makes really hard to review them.
All patches of a patchset must be in the same thread,
ideally with a cover letter.
All versions of a patchset must be in the same thread,
ideally with the cover letter v1 being the common ancestor.
Please check http://core.dpdk.org/contribute/#send

You need also to report the acks you got in previous versions.

Thanks
  
David Christensen May 8, 2019, 10:50 p.m. UTC | #2
> It seems your emails are not threaded at all.
> It makes really hard to review them.
> All patches of a patchset must be in the same thread,
> ideally with a cover letter.
> All versions of a patchset must be in the same thread,
> ideally with the cover letter v1 being the common ancestor.
> Please check http://core.dpdk.org/contribute/#send
> 
> You need also to report the acks you got in previous versions.

I messed up with the first post in the patch series (no 0/4 message). In 
order to fix the long line problem, which post would you suggest I use 
as an anchor for my reply-to?  The original v1 1/4, the v3 1/4, or some 
other?

Dave
  
Thomas Monjalon May 9, 2019, 7:46 a.m. UTC | #3
09/05/2019 00:50, David Christensen:
> 
> > It seems your emails are not threaded at all.
> > It makes really hard to review them.
> > All patches of a patchset must be in the same thread,
> > ideally with a cover letter.
> > All versions of a patchset must be in the same thread,
> > ideally with the cover letter v1 being the common ancestor.
> > Please check http://core.dpdk.org/contribute/#send
> > 
> > You need also to report the acks you got in previous versions.
> 
> I messed up with the first post in the patch series (no 0/4 message). In 
> order to fix the long line problem, which post would you suggest I use 
> as an anchor for my reply-to?  The original v1 1/4, the v3 1/4, or some 
> other?

The patches of v3 are not threaded together,
so when doing a v4 you can skip --in-reply-to.
  
Thomas Monjalon May 9, 2019, 8:03 p.m. UTC | #4
09/05/2019 09:46, Thomas Monjalon:
> 09/05/2019 00:50, David Christensen:
> > > It seems your emails are not threaded at all.
> > > It makes really hard to review them.
> > > All patches of a patchset must be in the same thread,
> > > ideally with a cover letter.
> > > All versions of a patchset must be in the same thread,
> > > ideally with the cover letter v1 being the common ancestor.
> > > Please check http://core.dpdk.org/contribute/#send
> > > 
> > > You need also to report the acks you got in previous versions.
> > 
> > I messed up with the first post in the patch series (no 0/4 message). In 
> > order to fix the long line problem, which post would you suggest I use 
> > as an anchor for my reply-to?  The original v1 1/4, the v3 1/4, or some 
> > other?
> 
> The patches of v3 are not threaded together,
> so when doing a v4 you can skip --in-reply-to.

Series applied with added acks, fixes lines, etc, thanks.
  

Patch

diff --git a/app/test/test_barrier.c b/app/test/test_barrier.c
index 82b572c..c219d56 100644
--- a/app/test/test_barrier.c
+++ b/app/test/test_barrier.c
@@ -252,7 +252,7 @@  struct lcore_plock_test {
 
 		/* race condition occurred, lock doesn't work properly */
 		if (sum[i] != pt[i].val || 2 * iter != pt[i].iter) {
-			printf("error: local and shared sums don't much\n");
+			printf("error: local and shared sums don't match\n");
 			rc = -1;
 		}
 	}