Phonet: cleanup pipe enable socket option
authorRémi Denis-Courmont <remi.denis-courmont@nokia.com>
Fri, 8 Oct 2010 04:02:02 +0000 (04:02 +0000)
committerDavid S. Miller <davem@davemloft.net>
Fri, 8 Oct 2010 21:09:10 +0000 (14:09 -0700)
commit03789f26722a15ccfe6f191e9fb3d356f2f18a1e
treea55a493a1b4804d0acafad5fd1a4a3b1100f3ff4
parent6d8e74ed377dd4cbad7ccc69300f734090e15c05
Phonet: cleanup pipe enable socket option

The current code works like this:

  int garbage, status;
  socklen_t len = sizeof(status);

  /* enable pipe */
  setsockopt(fd, SOL_PNPIPE, PNPIPE_ENABLE, &garbage, sizeof(garbage));
  /* disable pipe */
  setsockopt(fd, SOL_PNPIPE, PNPIPE_DISABLE, &garbage, sizeof(garbage));
  /* get status */
  getsockopt(fd, SOL_PNPIPE, PNPIPE_INQ, &status, &len);

...which does not follow the usual socket option pattern. This patch
merges all three "options" into a single gettable&settable option,
before Linux 2.6.37 gets out:

  int status;
  socklen_t len = sizeof(status);

  /* enable pipe */
  status = 1;
  setsockopt(fd, SOL_PNPIPE, PNPIPE_ENABLE, &status, sizeof(status));
  /* disable pipe */
  status = 0;
  setsockopt(fd, SOL_PNPIPE, PNPIPE_ENABLE, &status, sizeof(status));
  /* get status */
  getsockopt(fd, SOL_PNPIPE, PNPIPE_ENABLE, &status, &len);

This also fixes the error code from EFAULT to ENOTCONN.

Signed-off-by: Rémi Denis-Courmont <remi.denis-courmont@nokia.com>
Cc: Kumar Sanghvi <kumar.sanghvi@stericsson.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Documentation/networking/phonet.txt
include/linux/phonet.h
net/phonet/pep.c