|To:||Amir Noam <amir.noam@xxxxxxxxx>|
|Subject:||Re: [bonding] Add basic support for dynamic configuration of bond interfaces|
|From:||Jeff Garzik <jgarzik@xxxxxxxxx>|
|Date:||Sun, 11 Jan 2004 14:39:19 -0500|
|Cc:||bonding-devel@xxxxxxxxxxxxxxxxxxxxx, netdev@xxxxxxxxxxx, hadi@xxxxxxxxxx, Ben Greear <greearb@xxxxxxxxxxxxxxx>|
|User-agent:||Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.4) Gecko/20030703|
Amir Noam wrote:
The new SIOCBONDING ioctl handles commands aimed for the bonding module (and not to any specific bonding interface). This is done via an ioctl hook that captures these commands at the socket level (before dev_ioctl()) and passes it to the bonding code. Such commands do not require opening a bonding interface at all, just a valid socket to send the ioctl.
Right. And this last requirement is spurious. The operation is essentially "open a socket unrelated to X, to perform actions on X".
bonding (and it sounds like VLAN too) needs a specific, well-known control point, for controlling module-level data such as the creation and deletion of interfaces.
One such well-known control point would be a /dev node. Jamal's suggestion is also an excellent one: let userspace use netlink to communicate with a well known "address" inside the kernel, which would work as the central (and thus bonding-module-wide) bonding control interface.
I was trying to follow the example of several other network modules (e.g. vlan, dlci, bridge) that use a socket level ioctl hook to handle such deviceless commands.
If vlan/dlci/bridge has similar requirements to described above ("open socket unrelated to X, to perform actions on X") then they need to be fixed up too.
Opening a random socket to use an ioctl(2) to produce some magic behavior is just ugly, and we need to avoid that. Maybe bgreear would be open to updating VLAN's interface to netlink at the same time, for example? That would allow us (us==netdev) to understand all angles and implications of a good interface design. VLAN AFAICS has a similar requirement: need an interface _unrelated_ to specific VLAN interfaces, to control VLAN interfaces. And netlink certainly works as a control protocol :)
|<Prev in Thread]||Current Thread||[Next in Thread>|
|Previous by Date:||forcedeth 0.20 eth0: received interrupt report, Stanislav Karchebny|
|Next by Date:||Re: [bonding] Add basic support for dynamic configuration of bond interfaces, Ben Greear|
|Previous by Thread:||Re: [bonding] Add basic support for dynamic configuration of bond interfaces, Amir Noam|
|Next by Thread:||Re: [bonding] Add basic support for dynamic configuration of bond interfaces, Ben Greear|
|Indexes:||[Date] [Thread] [Top] [All Lists]|