On Tue, Jan 30, 2018 at 03:13:41AM +0800, Zihan Yang wrote: > Currently, socket_listen does not allow caller to set custom > socket options, which is inconvenient when the caller wants > a non-blocking socket or wants to set TCP_NODELAY. Therefore, > two new structs are added and an extra parameter is provided > to socket_listen. Existing functions are unaffected by > providing a NULL pointer.
You've added all this extra functionality to pass arbitrary options, but then not used it in any of the later patches. We've been trying to remove complexity from this code, so I'm not in favour of adding new functionality that is not even used. I'm not seeing the point of adding the support for the O_NONBLOCK in the listener socket either - that can easily be turned on after you have the listener socket created. Regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
