]> git.proxmox.com Git - mirror_ubuntu-zesty-kernel.git/commit - drivers/net/ethernet/rocker/rocker.c
rocker: support prepare-commit transaction model
authorScott Feldman <sfeldma@gmail.com>
Sun, 10 May 2015 16:47:50 +0000 (09:47 -0700)
committerDavid S. Miller <davem@davemloft.net>
Tue, 12 May 2015 22:43:53 +0000 (18:43 -0400)
commitc4f20321d9680760a291991d77bc5b6d0eb2ed78
treec17f8e6954937ce520d343ad4a24adb5ea5f8e3f
parentf8e20a9f87d33865cc1d67f13da0db8d457fc3c9
rocker: support prepare-commit transaction model

For rocker, support prepare-commit transaction model for setting attributes
(and for adding objects).  This requires rocker to preallocate memory
needed for the commit up front in the prepare phase.  Since rtnl_lock is
held between prepare-commit, store the allocated memory on a queue hanging
off of the rocker_port.  Also, in prepare phase, do everything right up to
calling into HW.  The same code paths are tranversed in the driver for both
prepare and commit phases.  In some cases, any state modified in the
prepare phase must be reverted before returning so the commit phase makes
the same decisions.

As a consequence of holding rtnl_lock in process context for all attr sets
(and obj adds), all memory is GFP_KERNEL allocated and we don't need to
busy spin waiting for the device to complete the command.  So the bulk of
this patch is simplifying the memory allocations to only use GFP_KERNEL and
to remove the nowait flag and busy spin loop.

Signed-off-by: Scott Feldman <sfeldma@gmail.com>
Acked-by: Jiri Pirko <jiri@resnulli.us>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/rocker/rocker.c