This is the mail archive of the
glibc-cvs@sourceware.org
mailing list for the glibc project.
GNU C Library master sources branch master updated. glibc-2.17-918-ge7074e4
- From: davem at sourceware dot org
- To: glibc-cvs at sourceware dot org
- Date: 23 Jul 2013 09:31:42 -0000
- Subject: GNU C Library master sources branch master updated. glibc-2.17-918-ge7074e4
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GNU C Library master sources".
The branch, master has been updated
via e7074e4c5edb0acaa979ea08e533736f906a9d68 (commit)
from 1fe2988f523ddbad93ca7abc98fea982f2ae0505 (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
http://sourceware.org/git/gitweb.cgi?p=glibc.git;a=commitdiff;h=e7074e4c5edb0acaa979ea08e533736f906a9d68
commit e7074e4c5edb0acaa979ea08e533736f906a9d68
Author: David S. Miller <davem@davemloft.net>
Date: Tue Jul 23 02:31:37 2013 -0700
Increase nptl test case buffer size so we really block on current Linux kernels.
* tst-cancel4.c (WRITE_BUFFER_SIZE): Increase to 16384.
diff --git a/nptl/ChangeLog b/nptl/ChangeLog
index fc2ef41..0e4747e 100644
--- a/nptl/ChangeLog
+++ b/nptl/ChangeLog
@@ -1,3 +1,7 @@
+2013-07-22 David S. Miller <davem@davemloft.net>
+
+ * tst-cancel4.c (WRITE_BUFFER_SIZE): Increase to 16384.
+
2013-07-19 Dominik Vogt <vogt@de.ibm.com>
* pthread_mutex_lock.c: Fix whitespace.
diff --git a/nptl/tst-cancel4.c b/nptl/tst-cancel4.c
index 9ffd5d1..10b7c6e 100644
--- a/nptl/tst-cancel4.c
+++ b/nptl/tst-cancel4.c
@@ -83,7 +83,30 @@ static pthread_barrier_t b2;
# define IPC_ADDVAL 0
#endif
-#define WRITE_BUFFER_SIZE 4096
+/* The WRITE_BUFFER_SIZE value needs to be choosen such that if we set
+ the socket send buffer size to '1', a write of this size on that
+ socket will block.
+
+ The Linux kernel imposes a minimum send socket buffer size
+ which has changed over the years. Currently the value is:
+
+ 2 * (2048 + SKB_DATA_ALIGN(sizeof(struct sk_buff)))
+
+ which is attempting to make sure that with standard MTUs,
+ TCP can always queue up at least 2 full sized packets.
+
+ Furthermore, there is logic in the socket send paths that
+ will allow one more packet (of any size) to be queued up as
+ long as some socket buffer space remains. Blocking only
+ occurs when we try to queue up a new packet and the send
+ buffer space has already been fully consumed.
+
+ Therefore we must set this value to the largest possible value of
+ the formula above (and since it depends upon the size of "struct
+ sk_buff", it is dependent upon machine word size etc.) plus some
+ slack space. */
+
+#define WRITE_BUFFER_SIZE 16384
/* Cleanup handling test. */
static int cl_called;
-----------------------------------------------------------------------
Summary of changes:
nptl/ChangeLog | 4 ++++
nptl/tst-cancel4.c | 25 ++++++++++++++++++++++++-
2 files changed, 28 insertions(+), 1 deletions(-)
hooks/post-receive
--
GNU C Library master sources