Re: [PATCH 1/2] bonding: fix rtnl locking merge error

2007-11-08 Thread Jeff Garzik
On Tue, Nov 06, 2007 at 01:33:28PM -0800, Jay Vosburgh wrote: > Looks like I incorrectly merged one of the rtnl lock changes, > so that one function, bonding_show_active_slave, held rtnl but didn't > release it, and another, bonding_store_active_slave, never held rtnl but > did release it. >

Re: [PATCH 1/2] bonding: fix rtnl locking merge error

2007-11-07 Thread Moni Shoua
Jay Vosburgh wrote: > Looks like I incorrectly merged one of the rtnl lock changes, > so that one function, bonding_show_active_slave, held rtnl but didn't > release it, and another, bonding_store_active_slave, never held rtnl but > did release it. > > Fixed so the first function doesn

[PATCH 1/2] bonding: fix rtnl locking merge error

2007-11-06 Thread Jay Vosburgh
Looks like I incorrectly merged one of the rtnl lock changes, so that one function, bonding_show_active_slave, held rtnl but didn't release it, and another, bonding_store_active_slave, never held rtnl but did release it. Fixed so the first function doesn't mess with rtnl, and the s