)]}' { "log": [ { "commit": "370c3bd05cf02afabea9cd3f2de66202d6b516dc", "tree": "832cf6b3091db117f119aa8692d050f43fe0a9cb", "parents": [ "68a1750b46ad5177f7703081b5fe85624f1aa62b" ], "author": { "name": "Joe Eykholt", "email": "jeykholt@cisco.com", "time": "Tue Aug 25 14:03:47 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@suse.de", "time": "Thu Sep 10 12:08:02 2009 -0500" }, "message": "[SCSI] libfc: use ADISC to verify rport login state\n\nWhen rport_login is called on an rport that is already thought\nto be logged in, use ADISC. If that fails, redo PLOGI.\nThis is less disruptive after fabric changes that don\u0027t affect\nthe state of the target.\n\nImplement the sending of ADISC via fc_els_fill.\n\nAdd ADISC state to the rport state machine. This is entered from READY\nand returns to READY after successful completion. If it fails, the rport\nis either logged off and deleted or re-does PLOGI.\n\nSigned-off-by: Joe Eykholt \u003cjeykholt@cisco.com\u003e\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@suse.de\u003e\n" }, { "commit": "f657d299cf05883e23e12a69e86842da1df378ad", "tree": "3593d2054bd361f28593eea12c83868709013b0c", "parents": [ "25b37b981e706c6df72c28c94f7787c3ea0cd343" ], "author": { "name": "Joe Eykholt", "email": "jeykholt@cisco.com", "time": "Tue Aug 25 14:03:21 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@suse.de", "time": "Thu Sep 10 12:07:59 2009 -0500" }, "message": "[SCSI] libfc: improve debug messages for ELS response handlers\n\nImprove lport and rport debug messages to indicate whether\nthe response is LS_ACC, LS_RJT, closed, or timeout.\n\nSigned-off-by: Joe Eykholt \u003cjeykholt@cisco.com\u003e\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@suse.de\u003e\n" }, { "commit": "131203a1ef53f3a4deb3260031bc53c7e4db4a24", "tree": "0f85ee5bf5db4612a7a62aa8835f33a1acc5a9ac", "parents": [ "6bd054cbf3f7da3442f30a7d4eb7da4dd1c44f21" ], "author": { "name": "Joe Eykholt", "email": "jeykholt@cisco.com", "time": "Tue Aug 25 14:03:10 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@suse.de", "time": "Thu Sep 10 12:07:58 2009 -0500" }, "message": "[SCSI] libfc: move remote port lookup for ELS requests into fc_rport.c.\n\nThis moves the remote port lookup for incoming ELS requests into\nfc_rport.c, in preparation for handing PLOGI and LOGO from\nunknown rports.\n\nThis changes the arg to rport_recv_req from an rdata to an lport.\n\nSigned-off-by: Joe Eykholt \u003cjeykholt@cisco.com\u003e\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@suse.de\u003e\n" }, { "commit": "9737e6a7b5b8af48f983cd565df93493597c565b", "tree": "2bb3e50171af1fbc18d0f739d760b6218031fb31", "parents": [ "935d0fce44b906268b8a29de4e72ebb57a3a06d8" ], "author": { "name": "Robert Love", "email": "robert.w.love@intel.com", "time": "Tue Aug 25 14:02:59 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@suse.de", "time": "Thu Sep 10 12:07:57 2009 -0500" }, "message": "[SCSI] libfc: Initialize fc_rport_identifiers inside fc_rport_create\n\nCurrently these values are initialized by the callers. This was exposed\nby a later patch that adds PLOGI request support. The patch failed to\ninitialize the new remote port\u0027s roles and it caused problems. This patch\nhas the rport_create routine initialize the identifiers and then the\ncallers can override them with real values.\n\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@suse.de\u003e\n" }, { "commit": "0f6c6149870e03c722af6eae406758b28cb71320", "tree": "7dca629255539d73bd44fea4fb50a9c0a9ddd079", "parents": [ "8025b5db7e10cd90cadec940cc766be3bbda65e8" ], "author": { "name": "Joe Eykholt", "email": "jeykholt@cisco.com", "time": "Tue Aug 25 14:02:11 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@suse.de", "time": "Thu Sep 10 12:07:48 2009 -0500" }, "message": "[SCSI] libfc: do not log off rports before or after discovery\n\nWhen receiving an RSCN, do not log off all rports. This is\nextremely disruptive. If, after the GPN_FT response, some\nrports haven\u0027t been listed, delete them.\n\nAdd field disc_id to structs fc_rport_priv and fc_disc.\ndisc_id is an arbitrary serial number used to identify the\nrports found by the latest discovery. This eliminates the need\nto go through the rport list when restarting discovery.\n\nSigned-off-by: Joe Eykholt \u003cjeykholt@cisco.com\u003e\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@suse.de\u003e\n" }, { "commit": "b84c7962653e4d04065d2603f0e1424ee0f455ae", "tree": "7f5086f5a97a36dd081cb18c17a3a1a7b4cd8759", "parents": [ "786681b96fc1a5b94d187160b7bf80bf6b4681ed" ], "author": { "name": "Joe Eykholt", "email": "jeykholt@cisco.com", "time": "Tue Aug 25 14:01:34 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@suse.de", "time": "Thu Sep 10 12:07:45 2009 -0500" }, "message": "[SCSI] libfc: remove unused disc-\u003edelay element\n\nDelete unused disc-\u003edelay element.\n\nSigned-off-by: Joe Eykholt \u003cjeykholt@cisco.com\u003e\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@suse.de\u003e\n" }, { "commit": "786681b96fc1a5b94d187160b7bf80bf6b4681ed", "tree": "f15c89228d48057957d86ef468118a2c76dcce42", "parents": [ "00fea930d404b9a9039291d5a61975e6c2ea974e" ], "author": { "name": "Joe Eykholt", "email": "jeykholt@cisco.com", "time": "Tue Aug 25 14:01:29 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@suse.de", "time": "Thu Sep 10 12:07:44 2009 -0500" }, "message": "[SCSI] libfc: eliminate disc-\u003eevent\n\nThere was no need to have the discovery status stored in struct fc_disc.\n\nChange fc_disc_done() to take the discovery status as an argument\nand just pass it on to the discovery callback.\n\nSigned-off-by: Joe Eykholt \u003cjeykholt@cisco.com\u003e\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@suse.de\u003e\n" }, { "commit": "9e9d0452fe12115b1c1883c0d4d2ee509079791b", "tree": "7cf8bb38e28e562274dcd7eeb2adfdf394876d14", "parents": [ "4c0f62b5679321b2e5572cf541ffb9f7b344d47c" ], "author": { "name": "Joe Eykholt", "email": "jeykholt@cisco.com", "time": "Tue Aug 25 14:01:18 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@suse.de", "time": "Thu Sep 10 12:07:43 2009 -0500" }, "message": "[SCSI] libfc: don\u0027t create dummy (rogue) remote ports\n\nDon\u0027t create a \"dummy\" remote port to go with fc_rport_priv.\n\nMake the rport truly optional by allocating fc_rport_priv separately\nand not requiring a dummy rport to be there if we haven\u0027t yet done\nfc_remote_port_add().\n\nThe fc_rport_libfc_priv remains as a structure attached to the\nrport for I/O purposes.\n\nBe sure to hold references on rdata when the lock is dropped in\nfc_rport_work().\n\nSigned-off-by: Joe Eykholt \u003cjeykholt@cisco.com\u003e\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@suse.de\u003e\n" }, { "commit": "4c0f62b5679321b2e5572cf541ffb9f7b344d47c", "tree": "0c4fba12e9098c63cfdc37dac15313f313887b1a", "parents": [ "629f44279d169f29b084d406e9f1c33314f220fa" ], "author": { "name": "Joe Eykholt", "email": "jeykholt@cisco.com", "time": "Tue Aug 25 14:01:12 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@suse.de", "time": "Thu Sep 10 12:07:43 2009 -0500" }, "message": "[SCSI] libfc: rename rport event CREATED to READY\n\nRemote ports will become READY more than once after\nADISC is implemented in a later patch.\n\nThe event callback that has been called \"CREATED\" will mean \"READY\".\nRename it now in preparation for those changes.\n\nSigned-off-by: Joe Eykholt \u003cjeykholt@cisco.com\u003e\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@suse.de\u003e\n" }, { "commit": "f211fa514a07326c0f9364c0e6ed17e38860172f", "tree": "9c2c54fee556816f36211185b6d6df0812b9acec", "parents": [ "a46f327aa5caf2cce138e98ddd863b6cca0e71e2" ], "author": { "name": "Joe Eykholt", "email": "jeykholt@cisco.com", "time": "Tue Aug 25 14:01:01 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@suse.de", "time": "Thu Sep 10 12:07:42 2009 -0500" }, "message": "[SCSI] libfc: make rport structure optional\n\nAllow a struct fc_rport_priv to have no fc_rport associated with it.\nThis sets up to remove the need for \"rogue\" rports.\n\nAdd a few fields to fc_rport_priv that are needed before the fc_rport\nis created. These are the ids, maxframe_size, classes, and rport pointer.\n\nRemove the macro PRIV_TO_RPORT(). Just use rdata-\u003erport where appropriate.\n\nTo take the place of the get_device()/put_device ops that were used to\nhold both the rport and rdata, add a reference count to rdata structures\nusing kref. When kref_get decrements the refcount to zero, a new template\nfunction releasing the rdata should be called. This will take care of\nfreeing the rdata and releasing the hold on the rport (for now). After\nsubsequent patches make the rport truly optional, this release function\nwill simply free the rdata.\n\nRemove the simple inline function fc_rport_set_name(), which becomes\nsemanticly ambiguous otherwise. The caller will set the port_name and\nnode_name in the rdata-\u003eIds, which will later be copied to the rport\nwhen it its created.\n\nSigned-off-by: Joe Eykholt \u003cjeykholt@cisco.com\u003e\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@suse.de\u003e\n" }, { "commit": "a46f327aa5caf2cce138e98ddd863b6cca0e71e2", "tree": "dbb230e8d4e024085e126445e6d819d5088e0406", "parents": [ "9fb9d32831fd687e427ec5b147bb690f468b99a0" ], "author": { "name": "Joe Eykholt", "email": "jeykholt@cisco.com", "time": "Tue Aug 25 14:00:55 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@suse.de", "time": "Thu Sep 10 12:07:41 2009 -0500" }, "message": "[SCSI] libfc: change elsct to use FC_ID instead of rdata\n\ntt.elsct_send is used by both FCP and by the rport state machine.\nAfter further patches, these two modules will use different\nstructures for the remote port.\n\nSo, change elsct_send to use the FC_ID instead of the fc_rport_priv\nas its argument. It currently only uses the FC_ID anyway.\n\nFor CT requests the destination FC_ID is still implicitly 0xfffffc.\nAfter further patches the did arg on CT requests will be used to\nspecify the FC_ID being inquired about for GPN_ID or other queries.\n\nSigned-off-by: Joe Eykholt \u003cjeykholt@cisco.com\u003e\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@suse.de\u003e\n" }, { "commit": "9fb9d32831fd687e427ec5b147bb690f468b99a0", "tree": "c3b6c29cb94040718ea2fe00daac05abf10db714", "parents": [ "922aa210bcad4b34a7bb98ec9d318b7e59e7a5ca" ], "author": { "name": "Joe Eykholt", "email": "jeykholt@cisco.com", "time": "Tue Aug 25 14:00:50 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@suse.de", "time": "Thu Sep 10 12:07:41 2009 -0500" }, "message": "[SCSI] libfc: make fc_rport_priv the primary rport interface.\n\nThe rport and discovery modules deal with remote ports\nbefore fc_remote_port_add() can be done, because the\nfull set of rport identifiers is not known at early stages.\n\nIn preparation for splitting the fc_rport/fc_rport_priv allocation,\nmake fc_rport_priv the primary interface for the remote port and\ndiscovery engines.\n\nThe FCP / SCSI layers still deal with fc_rport and\nfc_rport_libfc_priv, however.\n\nSigned-off-by: Joe Eykholt \u003cjeykholt@cisco.com\u003e\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@suse.de\u003e\n" }, { "commit": "922aa210bcad4b34a7bb98ec9d318b7e59e7a5ca", "tree": "480d1304252f34cdb8286ec104850648b852e598", "parents": [ "795d86f55ec3bf6280dda368f208943f1fb7d366" ], "author": { "name": "Joe Eykholt", "email": "jeykholt@cisco.com", "time": "Tue Aug 25 14:00:45 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@suse.de", "time": "Thu Sep 10 12:07:40 2009 -0500" }, "message": "[SCSI] libfc: fix RPORT_TO_PRIV and PRIV_TO_RPORT() macros.\n\nThese macros introduce extra undesirable semicolons that keep\nthem from being used in expressions, and they don\u0027t protect\nagainst being passed an expression.\n\nAdd parens and remove the semicolons.\n\nSigned-off-by: Joe Eykholt \u003cjeykholt@cisco.com\u003e\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@suse.de\u003e\n" }, { "commit": "795d86f55ec3bf6280dda368f208943f1fb7d366", "tree": "8d123acf1e7d9d07495d702b88ef2d75dfe696b1", "parents": [ "ab28f1fd3b0d14c1bd693e640decd711d5e6642a" ], "author": { "name": "Joe Eykholt", "email": "jeykholt@cisco.com", "time": "Tue Aug 25 14:00:39 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@suse.de", "time": "Thu Sep 10 12:07:40 2009 -0500" }, "message": "[SCSI] libfc: change interface for rport_create\n\nThe interface for lport-\u003ett.rport_create() takes a fc_disc_port arg,\nwhich is unnatural for most calls. The only reason for this was\nto avoid passing in the local port as an argument, but otherwise\nadded to complexity.\n\nSimplify by just using lport and fc_rport_identifiers.\n\nSigned-off-by: Joe Eykholt \u003cjeykholt@cisco.com\u003e\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@suse.de\u003e\n" }, { "commit": "ab28f1fd3b0d14c1bd693e640decd711d5e6642a", "tree": "3d7b051d7ab8b6d7fb7d936f95e336bf25f26832", "parents": [ "090eb6c41aa74273d3f0721637cff738cfd80669" ], "author": { "name": "Joe Eykholt", "email": "jeykholt@cisco.com", "time": "Tue Aug 25 14:00:34 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@suse.de", "time": "Thu Sep 10 12:07:39 2009 -0500" }, "message": "[SCSI] libfc: prepare to split off struct fc_rport_priv from fc_rport_libfc_priv\n\nWhile the I/O and LLD interfaces use fc_rport_libfc_priv, the\ndisc and rport interfaces will use fc_rport_priv, which will\nbe separately allocated.\n\nChange the disc and rport usage of fc_rport_libfc_priv to fc_rport_priv.\n\nUse #define temporarily to make both names equivalent until a\nsubsequent patch splits them.\n\nSigned-off-by: Joe Eykholt \u003cjeykholt@cisco.com\u003e\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@suse.de\u003e\n" }, { "commit": "b2f0091fbf8b475fa09b5e1712e0ab84cb3e1ca4", "tree": "98d3427aaae0b3d20d5fd077023b4ea23f5b575c", "parents": [ "e4bc50bedf0dd6c63f20a7bc0a2b46667664fba1" ], "author": { "name": "Vasu Dev", "email": "vasu.dev@intel.com", "time": "Tue Aug 25 13:58:53 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@suse.de", "time": "Sat Sep 05 09:47:37 2009 -0500" }, "message": "[SCSI] fcoe, libfc: fully makes use of per cpu exch pool and then removes em_lock\n\n1. Updates fcoe_rcv() to queue incoming frames to the fcoe per\n cpu thread on which this frame\u0027s exch was originated and simply\n use current cpu for request exch not originated by initiator.\n It is redundant to add this code under CONFIG_SMP, so removes\n CONFIG_SMP uses around this code.\n\n2. Updates fc_exch_em_alloc, fc_exch_delete, fc_exch_find to use\n per cpu exch pools, here fc_exch_delete is rename of older\n fc_exch_mgr_delete_ep since ep/exch are now deleted in pools\n of EM and so brief new name is sufficient and better name.\n\n Updates these functions to map exch id to their index into exch\n pool using fc_cpu_mask, fc_cpu_order and EM min_xid.\n This mapping is as per detailed explanation about this in\n last patch and basically this is just as lower fc_cpu_mask\n bits of exch id as cpu number and upper bit sum of EM min_xid\n and exch index in pool.\n\n Uses pool next_index to keep track of exch allocation from\n pool along with pool_max_index as upper bound of exches array\n in pool.\n\n3. Adds exch pool ptr to fc_exch to free exch to its pool in\n fc_exch_delete.\n\n4. Updates fc_exch_mgr_reset to reset all exch pools of an EM,\n this required adding fc_exch_pool_reset func to reset exches\n in pool and then have fc_exch_mgr_reset call fc_exch_pool_reset\n for each pool within each EM for a lport.\n\n5. Removes no longer needed exches array, em_lock, next_xid, and\n total_exches from struct fc_exch_mgr, these are not needed after\n use of per cpu exch pool, also removes not used max_read,\n last_read from struct fc_exch_mgr.\n\n6. Updates locking notes for exch pool lock with fc_exch lock and\n uses pool lock in exch allocation, lookup and reset.\n\nSigned-off-by: Vasu Dev \u003cvasu.dev@intel.com\u003e\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@suse.de\u003e\n" }, { "commit": "e4bc50bedf0dd6c63f20a7bc0a2b46667664fba1", "tree": "43f4e7a1d1fb5c042330c07cd0209f1b0eb92fb5", "parents": [ "a69b06bc5e7b153043db8984564b731f99e014fc" ], "author": { "name": "Vasu Dev", "email": "vasu.dev@intel.com", "time": "Tue Aug 25 13:58:47 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@suse.de", "time": "Sat Sep 05 09:47:36 2009 -0500" }, "message": "[SCSI] fcoe, libfc: adds per cpu exch pool within exchange manager(EM)\n\nAdds per cpu exch pool for these reasons:-\n\n 1. Currently an EM instance is shared across all cpus to manage\n all exches for all cpus. This required em_lock across all\n cpus for an exch alloc, free, lookup and reset each frame\n and that made em_lock expensive, so instead having per cpu\n exch pool with their own per cpu pool lock will likely reduce\n locking contention in fast path for an exch alloc, free and\n lookup.\n\n 2. Per cpu exch pool will likely improve cache hit ratio since\n all frames of an exch will be processed on the same cpu on\n which exch originated.\n\nThis patch is only prep work to help in keeping complexity of next\npatch low, so this patch only sets up per cpu exch pool and related\nhelper funcs to be used by next patch. The next patch fully makes\nuse of per cpu exch pool in all code paths ie. tx, rx and reset.\n\nDivides per EM exch id range equally across all cpus to setup per\ncpu exch pool. This division is such that lower bits of exch id\ncarries cpu number info on which exch originated, later a simple\nbitwise AND operation on exch id of incoming frame with fc_cpu_mask\nretrieves cpu number info to direct all frames to same cpu on which\nexch originated. This required a global fc_cpu_mask and fc_cpu_order\ninitialized to max possible cpus number nr_cpu_ids rounded up to 2\u0027s\npower, this will be used in mapping exch id and exch ptr array\nindex in pool during exch allocation, find or reset code paths.\n\nAdds a check in fc_exch_mgr_alloc() to ensure specified min_xid\nlower bits are zero since these bits are used to carry cpu info.\n\nAdds and initializes struct fc_exch_pool with all required fields\nto manage exches in pool.\n\nAllocates per cpu struct fc_exch_pool with memory for exches array\nfor range of exches per pool. The exches array memory is followed\nby struct fc_exch_pool.\n\nAdds fc_exch_ptr_get/set() helper functions to get/set exch ptr in\npool exches array at specified array index.\n\nIncreases default FCOE_MAX_XID to 0x0FFF from 0x07EF, so that more\nexches are available per cpu after above described exch id range\ndivision across all cpus to each pool.\n\nSigned-off-by: Vasu Dev \u003cvasu.dev@intel.com\u003e\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@suse.de\u003e\n" }, { "commit": "52ff878c912215210f53c0a080552dd6ba3055a2", "tree": "efaf0c6a6585aedc02af5faa3b47f4f4685fc783", "parents": [ "d459b7ea1b4c7aa3dacfeee174d02b2f7a95850d" ], "author": { "name": "Vasu Dev", "email": "vasu.dev@intel.com", "time": "Wed Jul 29 17:05:10 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@suse.de", "time": "Sat Aug 22 17:52:08 2009 -0500" }, "message": "[SCSI] fcoe, fnic, libfc: modifies current code paths to use EM anchor list\n\nModifies current code to use EM anchor list in EM allocation, EM free,\nEM reset, exch allocation and exch lookup code paths.\n\n 1. Modifies fc_exch_mgr_alloc to accept EM match function and then\n have allocated EM added to the lport using fc_exch_mgr_add API\n while also updating EM kref for newly added EM.\n\n 2. Updates fc_exch_mgr_free API to accept only lport pointer instead\n EM and then have this API free all EMs of the lport from EM anchor\n list.\n\n 3. Removes single lport pointer link from the EM, which was used in\n associating lport pointer in newly allocated exchange. Instead have\n lport pointer passed along new exchange allocation call path and\n then store passed lport pointer in newly allocated exchange, this\n will allow a single EM instance to be used across more than one\n lport and used in EM reset to reset only lport specific exchanges.\n\n 4. Modifies fc_exch_mgr_reset to reset all EMs from the EM anchor list\n of the lport, adds additional exch lport pointer (ep-\u003elp) check for\n shared EM case to reset exchange specific to a lport requested reset.\n\n 5. Updates exch allocation API fc_exch_alloc to use EM anchor list and\n its anchor match func pointer. The fc_exch_alloc will walk the list\n of EMs until it finds a match, a match will be either null match\n func pointer or call to match function returning true value.\n\n 6. Updates fc_exch_recv to accept incoming frame on local port using\n only lport pointer and frame pointer without specifying EM instance\n of incoming frame. Instead modified fc_exch_recv to locate EM for the\n incoming frame by matching xid of incoming frame against a EM xid range.\n This change was required to use EM list in libfc Rx path and after this\n change the lport fc_exch_mgr pointer emp is not needed anymore, so\n removed emp pointer.\n\n 7. Updates fnic for removed lport emp pointer and above modified libfc APIs\n fc_exch_recv, fc_exch_mgr_alloc and fc_exch_mgr_free.\n\n 8. Removes exch_get and exch_put from libfc_function_template as these\n are no longer needed with EM anchor list and its match function use.\n Also removes its default function fc_exch_get.\n\nA defect this patch introduced regarding the libfc initialization order in\nthe fnic driver was fixed by Joe Eykholt \u003cjeykholt@cisco.com\u003e.\n\nSigned-off-by: Vasu Dev \u003cvasu.dev@intel.com\u003e\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@suse.de\u003e\n" }, { "commit": "d459b7ea1b4c7aa3dacfeee174d02b2f7a95850d", "tree": "65fafcc2af6d44713fd5e7f113fa7f180d4f4675", "parents": [ "96316099ac3cb259eac2d6891f3c75b38b29d26e" ], "author": { "name": "Robert Love", "email": "robert.w.love@intel.com", "time": "Wed Jul 29 17:05:05 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@suse.de", "time": "Sat Aug 22 17:52:08 2009 -0500" }, "message": "[SCSI] libfc: Remove the FC_EM_DBG macro\n\nCurrently there is a 1:1 relationship between the lport\nand exchange manager. This macro takes an EM as an argument\nand determines the lport from it. However, later patches\nwill use an EM list per lport, so we will no longer have\nthis 1:1 relationship- this macro must change.\n\nThe FC_EM_DBG macro is rarely used. There are four callers,\ntwo can use FC_LPORT_DBG instead and two can be removed\nsince they\u0027re not necessary. This patch makes those changes\nand removes the macro.\n\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@suse.de\u003e\n" }, { "commit": "96316099ac3cb259eac2d6891f3c75b38b29d26e", "tree": "f4ec7001462bd00340de19b3045063872652c5a6", "parents": [ "201e5795b7b9582accb6d83316e30f24d7d40fd3" ], "author": { "name": "Vasu Dev", "email": "vasu.dev@intel.com", "time": "Wed Jul 29 17:05:00 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@suse.de", "time": "Sat Aug 22 17:52:07 2009 -0500" }, "message": "[SCSI] fcoe, libfc: adds exchange manager(EM) anchor list per lport and related APIs\n\nAdds EM list using a anchor struct fc_exch_mgr_anchor, anchor is used\nto allow same EM instance sharing across more than one lport on a eth\ndevice, this implementation is per discussed design posted at\nhttp://www.open-fcoe.org/pipermail/devel/2009-June/002566.html.\n\nThe shared EM is required for multiple lports on eth device when\nusing multiple VLANs or NPIV.\n\nAdds fc_exch_mgr_add API to add a EM to the lport and fc_exch_mgr_del\nAPI to delete previously added EM.\n\nAlso adds function fc_exch_mgr_destroy() to destroy allocated EM.\nThe kref is added to the EM to keep track of EM usage count, the EM is\ndestroyed when no longer in use upon kref reaching to zero.\n\nThe caller can specify match function to fc_exch_mgr_add, this\nwill be used in determining exchange allocation from its EM or not.\n\nMoved calling of fcoe_em_config below fcoe_libfc_config calling,\nso that list head lp-\u003eema_list is initialized before configuring\nEM.\n\nSigned-off-by: Vasu Dev \u003cvasu.dev@intel.com\u003e\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@suse.de\u003e\n" }, { "commit": "141940548c6919c22bf0573c68fd59d961e22475", "tree": "64ef27e9f0e11df5e64bd5616ed194a02b0eea5c", "parents": [ "84b05445b9f0b1ac2192f32260c916426d902d79" ], "author": { "name": "Joe Eykholt", "email": "jeykholt@cisco.com", "time": "Wed Jul 29 17:04:43 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@suse.de", "time": "Sat Aug 22 17:52:06 2009 -0500" }, "message": "[SCSI] libfc: rename rport state \"NONE\" to \"DELETE\".\n\nState RPORT_ST_NONE was intented to be an invalid state (0), never used.\nThis was a misguided attempt to be sure it was always initialized.\nHaving an extra state meaning nothing requires switch statements to\nhave a case covering that state.\n\nState NONE has been used instead to mean the remote port is being deleted.\nChanging the name to RPORT_ST_DELETE.\n\nSigned-off-by: Joe Eykholt \u003cjeykholt@cisco.com\u003e\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@suse.de\u003e\n" }, { "commit": "b1d9fd5574763abe5c763e32e3547a4adee9bd88", "tree": "fbf80ee698c4f9c9ffa8dc054d5c84df2459076f", "parents": [ "f161fb72104c7addac3d404a1ff543b2491c1426" ], "author": { "name": "Joe Eykholt", "email": "jeykholt@cisco.com", "time": "Wed Jul 29 17:04:22 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@suse.de", "time": "Sat Aug 22 17:52:04 2009 -0500" }, "message": "[SCSI] libfc: rename lport NONE state to DISABLED\n\nThe state NONE was meant to be invalid, but has been used as\nthe initial state. Rename it to be DISABLED, as more descriptive.\nFurther patches will make it the like the RESET state, except\nit won\u0027t transition to FLOGI until fc_lport_fabric_login() is called.\n\nSigned-off-by: Joe Eykholt \u003cjeykholt@cisco.com\u003e\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@suse.de\u003e\n" }, { "commit": "7f74549ff630ad444b0b6bbcabf426f781910906", "tree": "37e0a61a3cb43cd55799bb3b5bbb98cba215eee3", "parents": [ "beb29a6d421f6dbd41d68d0621c1b28ad1d4a9f4" ], "author": { "name": "Joe Eykholt", "email": "jeykholt@cisco.com", "time": "Wed Jul 29 17:04:12 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@suse.de", "time": "Sat Aug 22 17:52:03 2009 -0500" }, "message": "[SCSI] libfc: change debug messages to give host number.\n\nlibfc debug messages currently show \u0027lport: \u003cfc-id\u003e:\u0027\nwher \u003cfc-id\u003e is the hex assigned port-id. When the lport\nis logged off, that will be zero, so its hard to distinguish\nwhich instance is involved. The FC-ID can change\nif the port is re-patched or changes VSANs.\n\nTwo lports may even have the same FC-ID if connected to isolated SANs.\n\nChange the debug messages to print the SCSI host number \"hostN:\",\nwhich will not change for the life of the lport.\nStill show the FC_ID on lport messages.\n\nAlso, add a macro to FC_RPORT_ID_DBG for rport debugging where there\u0027s\nno rdata structure involved. It takes the lport and port_id as parameters.\nUse this in fc_rport_recv_plogi_req() and fc_rport_recv_logo_req().\n\nSigned-off-by: Joe Eykholt \u003cjeykholt@cisco.com\u003e\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@suse.de\u003e\n" }, { "commit": "beb29a6d421f6dbd41d68d0621c1b28ad1d4a9f4", "tree": "a899439f436fe913f8ee505bd367387395516c76", "parents": [ "2f718d64ecc7010463d36e6ec4ae37778d03fc0b" ], "author": { "name": "Joe Eykholt", "email": "jeykholt@cisco.com", "time": "Wed Jul 29 17:04:06 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@suse.de", "time": "Sat Aug 22 17:52:03 2009 -0500" }, "message": "[SCSI] libfc: remove extra semicolons from debug macros\n\nThis is unlikely to cause any problems, but the libfc debug macros\nintroduce extra undesirable semicolons.\n\nSigned-off-by: Joe Eykholt \u003cjeykholt@cisco.com\u003e\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@suse.de\u003e\n" }, { "commit": "7414705ea4aef9ce438e547f3138a680d2d1096c", "tree": "c8c14a80e31dbecbd6a9d083369f3e5e11105756", "parents": [ "650bd12b9e31ec51d7ad0df3c4f94d863b827976" ], "author": { "name": "Robert Love", "email": "robert.w.love@intel.com", "time": "Wed Jun 10 15:31:10 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@HansenPartnership.com", "time": "Sun Jun 21 11:07:08 2009 -0500" }, "message": "libfc: Add runtime debugging with debug_logging module parameter\n\nThis patch adds the /sys/module/libfc/parameters/debug_logging\nfile to sysfs as a module parameter. It accepts an integer\nbitmask for logging. Currently it supports:\n\n bit\nLSB 0 \u003d general libfc debugging\n 1 \u003d lport debugging\n 2 \u003d disc debugging\n 3 \u003d rport debugging\n 4 \u003d fcp debugging\n 5 \u003d EM debugging\n 6 \u003d exch/seq debugging\n 7 \u003d scsi logging (mostly error handling)\n\nthe other bits are not used at this time.\n\nThe patch converts all of the libfc source files to use\nthese new macros and removes the old FC_DBG macro.\n\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\n" }, { "commit": "a366695592ebc9151dd5a248681270f0925d8324", "tree": "cbf40a330cdd04733a34fb8f684cfe1d7a979c31", "parents": [ "d17bf602fbfe4a5ab134a90f6c082fa161d398ee" ], "author": { "name": "Abhijeet Joglekar", "email": "abjoglek@cisco.com", "time": "Fri May 01 10:01:26 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@HansenPartnership.com", "time": "Sat May 23 15:44:18 2009 -0500" }, "message": "[SCSI] libfc,fcoe,fnic: Separate rport and lport max retry counts\n\nThis allows fnic to configure number of retries for lport and rport\nseparately.\n\nSigned-off-by: Abhijeet Joglekar \u003cabjoglek@cisco.com\u003e\nAcked-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\n" }, { "commit": "b4c6f54632ad664a3d9e7f05e4ea0f1803e32755", "tree": "6d419c4a2cdf1e58ae8044d006c53df9cb856f07", "parents": [ "76f6804e7e7bb836cbdf4a73fe6c5485e4cc04c2" ], "author": { "name": "Abhijeet Joglekar", "email": "abjoglek@cisco.com", "time": "Tue Apr 21 16:27:04 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@HansenPartnership.com", "time": "Mon Apr 27 10:18:57 2009 -0500" }, "message": "[SCSI] libfc: Track rogue remote ports\n\nRogue ports are currently not tracked on any list. The only reference\nto them is through any outstanding exchanges pending on the rogue ports.\nIf the module is removed while a retry is set on a rogue port\n(say a Plogi retry for instance), this retry is not cancelled because there\nis no reference to the rogue port in the discovery rports list. Thus the\nlocal port can clean itself up, delete the exchange pool, and then the\nrogue port timeout can fire and try to start up another exchange.\n\nThis patch tracks the rogue ports in a new list disc-\u003erogue_rports. Creating\na new list instead of using the disc-\u003erports list keeps remote port code\nchange to a minimum.\n\n1) Whenever a rogue port is created, it is immediately added to the\ndisc-\u003erogue_rports list.\n\n2) When the rogues port goes to ready, it is removed from the rogue list\nand the real remote port is added to the disc-\u003erports list\n\n3) The removal of the rogue from the disc-\u003erogue_rports list is done in\nthe context of the fc_rport_work() workQ thread in discovery callback.\n\n4) Real rports are removed from the disc-\u003erports list like before. Lookup\nis done only in the real rports list. This avoids making large changes\nto the remote port code.\n\n5) In fc_disc_stop_rports, the rogues list is traversed in addition to the\nreal list to stop the rogue ports and issue logoffs on them. This way, rogue\nports get cleaned up when the local port goes away.\n\n6) rogue remote ports are not removed from the list right away, but\nremoved late in fc_rport_work() context, multiple threads can find the same\nremote port in the list and call rport_logoff(). Rport_logoff() only\ncontinues with the logoff if port is not in NONE state, thus preventing\nmultiple logoffs and multiple list deletions.\n\n7) Since the rport is removed from the disc list at a later stage\n(in the disc callback), incoming frames can find the rport even if\nrport_logoff() has been called on the rport. When rport_logoff() is called,\nthe rport state is set to NONE, and we are trying to cancel all exchanges\nand retries on that port. While in this state, if an incoming\nPlogi/Prli/Logo or other frames match the rport, we should not reply\nbecause the rport is in the NONE state. Just drop the frame, since the\nrport will be deleted soon in the disc callback (fc_rport_work)\n\n8) In fc_disc_single(), remove rport lookup and call to fc_disc_del_target.\nfc_disc_single() is called from recv_rscn_req() where rport lookup\nand rport_logoff is already done.\n\nSigned-off-by: Abhijeet Joglekar \u003cabjoglek@cisco.com\u003e\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\n" }, { "commit": "a0a25da2a495a889664dc946942b8666665cd1e1", "tree": "f3ac1dd6adb76490afa828298e71992ba5045d8f", "parents": [ "fdd78027fd472351783fb6110a72d991c1a07402" ], "author": { "name": "Vasu Dev", "email": "vasu.dev@intel.com", "time": "Tue Mar 17 11:42:29 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@HansenPartnership.com", "time": "Fri Apr 03 09:23:07 2009 -0500" }, "message": "[SCSI] fcoe, libfc: fix double fcoe_softc memory alloc\n\nThe foce_softc mem was reserved by libfc_host_alloc as well as\nby fcoe_host_alloc.\n\nRemoves one liner fcoe_host_alloc completely, instead directly calls\nlibfc_host_alloc to alloc scsi_host with libfc for just one fcoe_softc\nas fcoe private data.\n\nMoves libfc_host_alloc to libfc.h since it is a libfc API, placed\nlport_priv API adjacent to libfc_host_alloc since this is related\nto scsi_host priv data.\n\nSigned-off-by: Vasu Dev \u003cvasu.dev@intel.com\u003e\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\n" }, { "commit": "582b45bc577f78b5bfff3db874594ce2d962b846", "tree": "b0cfe95b235aeca9b967cba8446db58b8a102533", "parents": [ "5e5e92df49d4dfbef9ba981297c7f76d189376ac" ], "author": { "name": "Robert Love", "email": "robert.w.love@intel.com", "time": "Tue Mar 31 15:51:50 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@HansenPartnership.com", "time": "Fri Apr 03 09:22:58 2009 -0500" }, "message": "[SCSI] fcoe: Use per-CPU kernel function for dev_stats instead of an array\n\nRemove the hotplug creation of dev_stats, we allocate for all possible CPUs\nnow when we allocate the lport.\n\nv2: Durring the 2.6.30 merge window, before these patches were comitted,\n\u0027percpu_ptr\u0027 was renamed \u0027per_cpu_ptr\u0027. This latest update updates this\npatch for the name change.\n\nSigned-off-by: Yi Zou \u003cyi.zou@intel.com\u003e\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\n" }, { "commit": "b277d2aa9a4d969002c4157bf77b76b9ad9ca04a", "tree": "1054ff4ab2d03eaaa527919380ed6334233f632b", "parents": [ "39ca9a065a5a0a6f2f0cd648090a979ba3f4f018" ], "author": { "name": "Yi Zou", "email": "yi.zou@intel.com", "time": "Fri Feb 27 14:07:21 2009 -0800" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@HansenPartnership.com", "time": "Fri Mar 13 15:15:15 2009 -0500" }, "message": "[SCSI] libfc: add support of large receive offload by ddp in fc_fcp\n\nWhen LLD supports direct data placement (ddp) for large receive of an scsi\ni/o coming into fc_fcp, we call into libfc_function_template\u0027s ddp_setup()\nto prepare for a ddp of large receive for this read I/O. When I/O is complete,\nwe call the corresponding ddp_done() to get the length of data ddped as well\nas to let LLD do clean up.\n\nfc_fcp_ddp_setup()/fc_fcp_ddp_done() are added to setup and complete a ddped\nread I/O described by the given fc_fcp_pkt. They would call into corresponding\nddp_setup/ddp_done implemented by the fcoe layer. Eventually, fcoe layer calls\ninto LLD\u0027s ddp_setup/ddp_done provided through net_device\n\nSigned-off-by: Yi Zou \u003cyi.zou@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\n" }, { "commit": "ea1e9a9df5e1fde7ad8878c85b4a097cad0ddcea", "tree": "5594c002619125f8976c7d3dacdd6791db497d2e", "parents": [ "4d288d5767f853bfca25adc7b6030dc95518cb2e" ], "author": { "name": "Yi Zou", "email": "yi.zou@intel.com", "time": "Fri Feb 27 14:07:04 2009 -0800" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@HansenPartnership.com", "time": "Fri Mar 13 15:13:48 2009 -0500" }, "message": "[SCSI] fcoe, libfc: check offload features from LLD through netdev\n\nThis checks if net_devices supports FCoE offload ops in netdev_ops and it\nif it does, then sets up the corresponding flags in the associated fc_lport.\n\nFor large send offload, the maximum length supported in one large send is now\ndescribed by the added lso_max in fc_lport, which is setup initially from\nnetdev-\u003egso_max_size.\n\nSigned-off-by: Yi Zou \u003cyi.zou@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\n" }, { "commit": "34f42a070fc98f5dc07e9fa2338b7b8d1dc347eb", "tree": "7e08edd537159be8a4fa341e7022cafd6f13b9e4", "parents": [ "0ae4d4ae47d2ccbcad813b0d6d8fe12590c7d648" ], "author": { "name": "Robert Love", "email": "robert.w.love@intel.com", "time": "Fri Feb 27 10:55:45 2009 -0800" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@HansenPartnership.com", "time": "Tue Mar 10 09:04:40 2009 -0500" }, "message": "[SCSI] libfc, fcoe: Fix kerneldoc comments\n\n1) Added \u0027()\u0027 for function names in kerneldoc comments\n\n2) Changed comment bookends from \u0027**/\u0027 to \u0027*/\u0027. The comment on the the\n mailing list was that \u0027**/\u0027 \"is consistently unconventional. Not\n wrong, just odd.\" The Documentation/kernel-doc-nano-HOWTO.txt\n states that kerneldoc comment blocks should end with \u0027**/\u0027 but most\n (if not all) instance I found under drivers/scsi/ were only using\n the \u0027*/\u0027 so I converted to that style.\n\n3) Removed incorrect linebreaks in kerneldoc comments where found\n\n4) Removed a few unnecessary blank comment lines in kerneldoc comment\n blocks\n\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\n" }, { "commit": "0ae4d4ae47d2ccbcad813b0d6d8fe12590c7d648", "tree": "a58434d723ef74a7282c29835dec8496326a14a9", "parents": [ "efaf5c085dd2d31757b0ff7886970dfddd8d1808" ], "author": { "name": "Robert Love", "email": "robert.w.love@intel.com", "time": "Fri Feb 27 10:55:39 2009 -0800" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@HansenPartnership.com", "time": "Fri Mar 06 15:45:32 2009 -0600" }, "message": "[SCSI] libfc: Cleanup libfc_function_template comments\n\nMade the comments more like the comments for struct scsi_host_template.\n\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\n" }, { "commit": "5101ff99f59aefb72e0c96e82aa32048ac9f8425", "tree": "83fba57c7cf542eed4466265619632e7d804eb88", "parents": [ "f7db2c150cf5082cf74555f30a1305938041de80" ], "author": { "name": "Robert Love", "email": "robert.w.love@intel.com", "time": "Fri Feb 27 10:55:18 2009 -0800" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@HansenPartnership.com", "time": "Fri Mar 06 15:41:01 2009 -0600" }, "message": "[SCSI] libfc: Don\u0027t violate transport template for rogue port creation\n\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\n" }, { "commit": "bc0e17f691085315ae9303eb5b0883fe16dfe6b1", "tree": "53138e80aec810604d4eca4626e4f8df65376ccc", "parents": [ "a7e84f2b83f17f8f11da34ccef3ba5a862dc0182" ], "author": { "name": "Vasu Dev", "email": "vasu.dev@intel.com", "time": "Fri Feb 27 10:54:57 2009 -0800" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@HansenPartnership.com", "time": "Fri Mar 06 15:37:49 2009 -0600" }, "message": "[SCSI] libfc, fcoe: fixed locking issues with lport-\u003elp_mutex around lport-\u003elink_status\n\nThe fcoe_xmit could call fc_pause in case the pending skb queue len is larger\nthan FCOE_MAX_QUEUE_DEPTH, the fc_pause was trying to grab lport-\u003elp_muex to\nchange lport-\u003elink_status and that had these issues :-\n\n1. The fcoe_xmit was getting called with bh disabled, thus causing\n\"BUG: scheduling while atomic\" when grabbing lport-\u003elp_muex with bh disabled.\n\n2. fc_linkup and fc_linkdown function calls lport_enter function with\nlport-\u003elp_mutex held and these enter function in turn calls fcoe_xmit to send\nlport related FC frame, e.g. fc_linkup \u003d\u003e fc_lport_enter_flogi to send flogi\nreq. In this case grabbing the same lport-\u003elp_mutex again in fc_puase from\nfcoe_xmit would cause deadlock.\n\nThe lport-\u003elp_mutex was used for setting FC_PAUSE in fcoe_xmit path but\nFC_PAUSE bit was not used anywhere beside just setting and clear this\nbit in lport-\u003elink_status, instead used a separate field qfull in fc_lport\nto eliminate need for lport-\u003elp_mutex to track pending queue full condition\nand in turn avoid above described two locking issues.\n\nAlso added check for lp-\u003eqfull in fc_fcp_lport_queue_ready to trigger\nSCSI_MLQUEUE_HOST_BUSY when lp-\u003eqfull is set to prevent more scsi-ml cmds\nwhile lp-\u003eqfull is set.\n\nThis patch eliminated FC_LINK_UP and FC_PAUSE and instead used dedicated\nfields in fc_lport for this, this simplified all related conditional\ncode.\n\nAlso removed fc_pause and fc_unpause functions and instead used newly added\nlport-\u003eqfull directly in fcoe.\n\nSigned-off-by: Vasu Dev \u003cvasu.dev@intel.com\u003e\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\n" }, { "commit": "1f6ff364ceda516f88351a8ab640e656beed0b26", "tree": "227a5f6e39372eaf3c898d1de8feb2b88df707b0", "parents": [ "33dd6f92a1a7ad85c54d47fd9d73371a32c0bde4" ], "author": { "name": "Abhijeet Joglekar", "email": "abjoglek@cisco.com", "time": "Fri Feb 27 10:54:35 2009 -0800" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@HansenPartnership.com", "time": "Fri Mar 06 15:35:47 2009 -0600" }, "message": "[SCSI] libfc: Pass lport in exch_mgr_reset\n\nfc_exch_mgr structure is private to fc_exch.c. To export exch_mgr_reset to\ntransport, transport needs access to the exch manager. Change\nexch_mgr_reset to use lport param which is the shared structure between\nlibFC and transport.\n\nAlternatively, fc_exch_mgr definition can be moved to libfc.h so that lport\ncan be accessed from mp*.\n\nSigned-off-by: Abhijeet Joglekar \u003cabjoglek@cisco.com\u003e\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\n" }, { "commit": "42e9a92fe6a9095bd68a379aaec7ad2be0337f7a", "tree": "344f8d9f72a3d926d652632abb8d319f8e32343a", "parents": [ "f032c2f7cdaae0e8907cd3b26426fc651dc5c275" ], "author": { "name": "Robert Love", "email": "robert.w.love@intel.com", "time": "Tue Dec 09 15:10:17 2008 -0800" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@HansenPartnership.com", "time": "Mon Dec 29 11:24:33 2008 -0600" }, "message": "[SCSI] libfc: A modular Fibre Channel library\n\nlibFC is composed of 4 blocks supported by an exchange manager\nand a framing library. The upper 4 layers are fc_lport, fc_disc,\nfc_rport and fc_fcp. A LLD that uses libfc could choose to\neither use libfc\u0027s block, or using the transport template\ndefined in libfc.h, override one or more blocks with its own\nimplementation.\n\nThe EM (Exchange Manager) manages exhcanges/sequences for all\ncommands- ELS, CT and FCP.\n\nThe framing library frames ELS and CT commands.\n\nThe fc_lport block manages the library\u0027s representation of the\nhost\u0027s FC enabled ports.\n\nThe fc_disc block manages discovery of targets as well as\nhandling changes that occur in the FC fabric (via. RSCN events).\n\nThe fc_rport block manages the library\u0027s representation of other\nentities in the FC fabric. Currently the library uses this block\nfor targets, its peer when in point-to-point mode and the\ndirectory server, but can be extended for other entities if\nneeded.\n\nThe fc_fcp block interacts with the scsi-ml and handles all\nI/O.\n\nSigned-off-by: Robert Love \u003crobert.w.love@intel.com\u003e\n[jejb: added include of delay.h to fix ppc64 compile prob spotted by sfr]\nSigned-off-by: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\n" } ] }