[Bug 181741] [kernel] [patch] Packet loss when 'control' messages are present with large data (sendmsg(2))

Sat Apr 25 05:32:38 UTC 2015

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=181741

--- Comment #9 from commit-hook at freebsd.org ---
A commit references this bug:

Author: ngie
Date: Sat Apr 25 05:32:03 UTC 2015
New revision: 281974
URL: https://svnweb.freebsd.org/changeset/base/281974

Log:
  MFC
r261550,r281354,r281355,r281356,r281358,r281359,r281360,r281361,r281362,r281391,r281392,r281393,r281394,r281395,r281397,r281398,r281399,r281400,r281401,r281402,r281403,r281404,r281407,r281408,r281409,r281410,r281411:

  r261550 (by glebius):

  Add test case for kern/181741. Right now test fails.

  PR:        181741
  Sponsored by:    Nginx, Inc.

  r281354:

  Fix warnings, fix a typo in a testcase description, bump WARNS to 3

  - Remove argc/argv (-Wunused)
  - Cast len in comparison to size_t (-Wsign-compare)

  Sponsored by: EMC / Isilon Storage Division

  r281355:

  Fix -Wunused warnings, bump WARNS to 6

  The testcase fails today on subtest # 9

  The output is still broken if prove -rv is run and the testcase aborts
  prematurely (the testcase doesn't really conform to TAP protocol properly,
  except when it completes fully)

  Sponsored by: EMC / Isilon Storage Division

  r281356:

  Fix -Wunused warnings, bump WARNS to 6

  The output is still broken if prove -rv is run and the testcase aborts
  prematurely with fail_assertion (the testcase doesn't really conform to TAP
  protocol properly, except when it completes fully)

  Sponsored by: EMC / Isilon Storage Division

  r281358:

  - Parameterize out the number of accept/connect attempts
  - Randomize the bind port to allow 2+ consecutive calls in < 10 minutes, and
    to also not fail if (for instance) there's a server already listening on
port
    8080
  - Don't leak the listening socket / fds into the child process
  - Fix warnings:
  -- Remove argc/argv (-Wunused)
  -- Mark sig __unused (-Wunused)
  -- Mark quit static (-Wmissing-variable-declarations)

  Sponsored by: EMC / Isilon Storage Division

  r281359:

  Remove argc/argv (-Wunused)

  Sponsored by: EMC / Isilon Storage Division

  r281360:

  Fix warnings

  - Remove argc/argv (-Wunused)
  - Mark some parameters to socket_listen_update __unused (-Wunused)

  Sponsored by: EMC / Isilon Storage Division

  r281361:

  Remove argc/argv (-Wunused)

  Sponsored by: EMC / Isilon Storage Division

  r281362:

  Use _exit, not exit in forked process

  Sponsored by: EMC / Isilon Storage Division

  r281391:

  - Use static buffers for temporary file paths instead of strdup of constant
strings
  - Don't use /tmp because it's outside ATF's prescribed sandbox
  - Use mkstemp instead of mktemp to eliminate warning

  Sponsored by: EMC / Isilon Storage Division

  r281392:

  - Garbage collect argc/argv (-Wunused)
  - Bump WARNS to 6

  Sponsored by: EMC / Isilon Storage Division

  r281393:

  Fix warnings and bump WARNS to 6
  - Garbage collect argc/argv (-Wunused)
  - sleep(3) will always return an unsigned int; don't check for return codes
<0
    (-Wsign-compare)

  Sponsored by: EMC / Isilon Storage Division

  r281394:

  - Don't use /tmp because it's outside ATF's prescribed sandbox
  - Replace a hardcoded PATH_MAX value with sizeof(path)
  - Use path like an array, not a pointer, and always try to unlink it in
cleanup

  Sponsored by: EMC / Isilon Storage Division

  r281395:

  Fix a -Wuninitialized warning by setting the socket to -1 and bump WARNS to 6

  Sponsored by: EMC / Isilon Storage Division

  r281397:

  Mark signum unused in signal_handler; bump WARNS to 6

  Sponsored by: EMC / Isilon Storage Division

  r281398:

  Garbage collect argc/argv and bump WARNS to 6

  Sponsored by: EMC / Isilon Storage Division

  r281399:

  Fix warnings and bump WARNS to 6
  - Staticize variables as needed
  - Garbage collect argc/argv
  - Fix -Wsign-compare warnings by casting small sizeof to (int)

  Sponsored by: EMC / Isilon Storage Division

  r281400:

  - Garbage collect argc/argv; bump WARNS to 6
  - Make the socket path random and move it out of /tmp as that's outside ATF's
    prescribed path

  Sponsored by: EMC / Isilon Storage Division

  r281401:

  - Garbage collect argc/argv
  - Use random paths instead of one in /tmp

  Sponsored by: EMC / Isilon Storage Division

  r281402:

  Garbage collect argc/argv and bump WARNS to 6

  Sponsored by: EMC / Isilon Storage Division

  r281403:

  Garbage collect argc/argv and bump WARNS to 6

  Sponsored by: EMC / Isilon Storage Division

  r281404:

  Generate temporary files with mkstemp instead of mktemp

  Sponsored by: EMC / Isilon Storage Division

  r281407:

  Fix the knob twiddling to work properly per src.opts.mk

  Sponsored by: EMC / Isilon Storage Division

  r281408:

  - Remove the .t wrapper and put the "magic" of determining the number of
    testcases into the .c file
  - Require root for now because it fails with SOCK_RAW without root privileges
  - Increment the test count properly on socket create failure

  Sponsored by: EMC / Isilon Storage Division

  r281409:

  Fix warnings, bump WARNS to 6, and use a temporary socket instead of one in
/tmp

  Sponsored by: EMC / Isilon Storage Division

  r281410:

  Fix more warnings I didn't catch in the first go-around

  Sponsored by: EMC / Isilon Storage Division

  r281411:

  Fix even more warnings..

  Sponsored by: EMC / Isilon Storage Division

Changes:
_U  stable/10/
  stable/10/tools/regression/sockets/accept_fd_leak/Makefile
  stable/10/tools/regression/sockets/accept_fd_leak/accept_fd_leak.c
  stable/10/tools/regression/sockets/accf_data_attach/Makefile
  stable/10/tools/regression/sockets/accf_data_attach/accf_data_attach.c
  stable/10/tools/regression/sockets/fstat/Makefile
  stable/10/tools/regression/sockets/fstat/fstat.c
  stable/10/tools/regression/sockets/kqueue/Makefile
  stable/10/tools/regression/sockets/kqueue/kqueue.c
  stable/10/tools/regression/sockets/listen_backlog/Makefile
  stable/10/tools/regression/sockets/listen_backlog/listen_backlog.c
  stable/10/tools/regression/sockets/listenclose/Makefile
  stable/10/tools/regression/sockets/listenclose/listenclose.c
  stable/10/tools/regression/sockets/pr_atomic/Makefile
  stable/10/tools/regression/sockets/pr_atomic/pr_atomic.c
  stable/10/tools/regression/sockets/reconnect/Makefile
  stable/10/tools/regression/sockets/reconnect/reconnect.c
  stable/10/tools/regression/sockets/rtsocket/Makefile
  stable/10/tools/regression/sockets/rtsocket/rtsocket.c
  stable/10/tools/regression/sockets/sblock/Makefile
  stable/10/tools/regression/sockets/sblock/sblock.c
  stable/10/tools/regression/sockets/sendfile/sendfile.c
  stable/10/tools/regression/sockets/shutdown/Makefile
  stable/10/tools/regression/sockets/shutdown/shutdown.c
  stable/10/tools/regression/sockets/sigpipe/Makefile
  stable/10/tools/regression/sockets/sigpipe/sigpipe.c
  stable/10/tools/regression/sockets/so_setfib/Makefile
  stable/10/tools/regression/sockets/so_setfib/so_setfib.c
  stable/10/tools/regression/sockets/so_setfib/so_setfib.t
  stable/10/tools/regression/sockets/socketpair/Makefile
  stable/10/tools/regression/sockets/socketpair/socketpair.c
  stable/10/tools/regression/sockets/unix_bindconnect/Makefile
  stable/10/tools/regression/sockets/unix_bindconnect/unix_bindconnect.c
  stable/10/tools/regression/sockets/unix_close_race/Makefile
  stable/10/tools/regression/sockets/unix_close_race/unix_close_race.c
  stable/10/tools/regression/sockets/unix_passfd/Makefile
  stable/10/tools/regression/sockets/unix_passfd/unix_passfd.c
  stable/10/tools/regression/sockets/unix_sendtorace/Makefile
  stable/10/tools/regression/sockets/unix_sendtorace/unix_sendtorace.c
  stable/10/tools/regression/sockets/unix_socket/Makefile
  stable/10/tools/regression/sockets/unix_socket/unix_socket.c
  stable/10/tools/regression/sockets/unix_sorflush/Makefile
  stable/10/tools/regression/sockets/unix_sorflush/unix_sorflush.c
  stable/10/tools/regression/sockets/zerosend/zerosend.c

-- 
You are receiving this mail because:
You are the assignee for the bug.