On Thu, 2015-02-05 at 14:25 -0500, James Smart wrote: > --- This is a bit big for an empty explanation. You're adding a lot of unexplained debug statements, but your own words in the comment seem to be the best changelog:
> + * lpfc_sli4_unreg_rpi_cmpl_clr - mailbox completion handler > + * @phba: Pointer to HBA context object. > + * @pmb: Pointer to mailbox object. > + * > + * This function is the unreg rpi mailbox completion handler. It > + * frees the memory resources associated with the completed mailbox > + * command. An additional refrenece is put on the ndlp to prevent > + * lpfc_nlp_release from freeing the rpi bit in the bitmask before > + * the unreg mailbox command completes, this routine puts the > + * reference back. I know its in the code, but the reason we have changelogs is so that when distros ask "is this fixed upstream" they can look through the git log for the driver to get an idea. For them you need to say that this is SLI rev 4 only and then give the above explanation. James -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
