Re: sock_valbool_flag is required by VMware

2007-12-02 Thread Christoph Hellwig
On Fri, Nov 30, 2007 at 10:52:22PM +0100, Jiri Slaby wrote:
 Hi,
 
 this commit:
 [NET]: Move sock_valbool_flag to socket.c
 
 The sock_valbool_flag() helper is used in setsockopt to
 set or reset some flag on the sock. This helper is required
 in the net/socket.c only, so move it there.
 
 Besides, patch two places in sys_setsockopt() that repeat
 this helper functionality manually.
 
 Since this is not a bugfix, but a trivial cleanup, I
 prepared this patch against net-2.6.25, but it also
 applies (with a single offset) to the latest net-2.6.
 
 breaks vmware module compilation, since it uses sock_valbool_flag(). Is this
 their business (and they should use sock_set_flag/sock_reset_flag) or should
 this be reverted?

It's their business, they should never have used it.

--
To unsubscribe from this list: send the line unsubscribe netdev in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


sock_valbool_flag is required by VMware

2007-11-30 Thread Jiri Slaby
Hi,

this commit:
[NET]: Move sock_valbool_flag to socket.c

The sock_valbool_flag() helper is used in setsockopt to
set or reset some flag on the sock. This helper is required
in the net/socket.c only, so move it there.

Besides, patch two places in sys_setsockopt() that repeat
this helper functionality manually.

Since this is not a bugfix, but a trivial cleanup, I
prepared this patch against net-2.6.25, but it also
applies (with a single offset) to the latest net-2.6.

breaks vmware module compilation, since it uses sock_valbool_flag(). Is this
their business (and they should use sock_set_flag/sock_reset_flag) or should
this be reverted?

thanks,
-- 
Jiri Slaby ([EMAIL PROTECTED])
Faculty of Informatics, Masaryk University
-
To unsubscribe from this list: send the line unsubscribe netdev in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html