[Open-FCoE] [RFC PATCH V4 00/16] prepwork and bugfixes before enabling NPIV

Joe Eykholt jeykholt at cisco.com
Fri Jul 17 18:53:59 UTC 2009

Chris Leech wrote:
> Repost of my NPIV prepwork series, with corrected locking and some additional
> bug fixes.  This includes locking changes to synchronize multiple create /
> destroy / load / unload events, as well as fixes to handle netdev
> unregistration cleanly.
> I've reposted Joe's initialization cleanup here to make sure it didn't get
> lost.  It was posted with other patches that conflicted with / were merged
> into this series.

I did a brief scan through the patches and didn't see any problems.
I'll try running them with my rport patches when I get a chance.

Thanks for doing all this work, Chris.
It's great to get rid of the races and deadlocks.


> ---
> Chris Leech (15):
>       fcoe: use rtnl mutex in place of hostlist lock
>       fcoe: Fix module ref count bug by adding NETDEV UNREGISTER handling
>       fcoe: move the host-list add/remove to keep out VN_Ports
>       fcoe: add mutex to protect create and destroy
>       fcoe: split out per interface setup
>       fcoe: fcoe_interface create, destroy and refcounting
>       fcoe: remove fcoe_interface->priv pointer
>       fcoe: move offload exchange manager pointer from fcoe_port to fcoe_interface
>       fcoe: move FIP controller from fcoe_port to fcoe_interface
>       fcoe: move packet handlers from fcoe_port to fcoe_interface
>       fcoe: move real_dev and phys_dev to fcoe_interface
>       fcoe: Introduce and allocate fcoe_interface structure, 1:1 with net_device
>       fcoe: interface changes to fcoe_if_create and fcoe_if_destroy
>       fcoe: fix missing error check in call to fcoe_if_init
>       libfcoe: fcoe_ctlr_destroy use cancel_work_sync instead of flush_work
> Joe Eykholt (1):
>       fcoe: remove unnecessary list and lock initializations.
>  drivers/scsi/fcoe/fcoe.c    |  875 +++++++++++++++++++++++--------------------
>  drivers/scsi/fcoe/fcoe.h    |   25 +
>  drivers/scsi/fcoe/libfcoe.c |    8 
>  3 files changed, 493 insertions(+), 415 deletions(-)
> _______________________________________________
> devel mailing list
> devel at open-fcoe.org
> http://www.open-fcoe.org/mailman/listinfo/devel

More information about the devel mailing list