Thread (10 messages) 10 messages, 4 authors, 2010-10-01

Re: [PATCH v2 1/2] Phonet: Implement Pipe Controller to support Nokia Slim Modems

From: Rémi Denis-Courmont <hidden>
Date: 2010-10-01 09:20:57

   Hello,

On Friday 01 October 2010, Kumar SANGHVI wrote:
Hi,

On Fri, Oct 01, 2010 at 10:42:44 +0200, Rémi Denis-Courmont wrote:
quoted
quoted
I have not introduced any new ioctl()'s as part of Pipe controller
implementation.
Sure. What you did is basically worse than ioctl()'s. You've implemented
them as socket options. Socket options are meant to configure parameters
with setsockopt and read paramters with getsockopt. They are not meant
for 'doing' things - that's what ioctl()'s are for.
Isn't the existing phonet stack 'doing' something as part of
PNPIPE_ENCAP rather than simply configuring some socket option or flag ?
It sets (or gets) the delivery path for incoming data.

-- 
Rémi Denis-Courmont
http://www.remlab.net/
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help