Skip to contentSkip to conversation

sock_valbool_flag is required by VMware

DISCUSSION Jiri Slaby 2 participants 2 messages Nov 30, 2007 – Dec 2, 2007

Subsystems (2)
lore
Share & tools
lore

Jiri Slaby

author
#0 lore raw

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,

Signature · 2 lines
Jiri Slaby ([email protected])
Faculty of Informatics, Masaryk University
#1 lore raw
Quoted 12 lines · 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.

Quoted 7 lines

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.