On Sun, 2009-02-22 at 16:19 +0530, Kashyap, Desai wrote: > This Patch is submitted to with reference to MSI issue with some of LSI > controller. Disabling it by default when module is loaded. Point of order here: this is *not* a resend of a previous patch. This is a bug fix of a previously sent patch (With a change in intended behaviour). > Signed-off-by: Kashyap Desai <kadesai@xxxxxxx> > --- > diff --git a/drivers/message/fusion/mptbase.c b/drivers/message/fusion/mptbase.c > index 96ac883..ea3aafb 100644 > --- a/drivers/message/fusion/mptbase.c > +++ b/drivers/message/fusion/mptbase.c > @@ -91,9 +91,9 @@ MODULE_PARM_DESC(mpt_msi_enable_fc, " Enable MSI Support for FC \ > controllers (default=0)"); > > static int mpt_msi_enable_sas; > -module_param(mpt_msi_enable_sas, int, 1); > +module_param(mpt_msi_enable_sas, int, 0); This is the actual bug fix, which isn't mentioned anywhere in the change log > MODULE_PARM_DESC(mpt_msi_enable_sas, " Enable MSI Support for SAS \ > - controllers (default=1)"); > + controllers (default=0)"); This entire patch is also functionally identical to the second patch in this email from Yinghai Lu: http://marc.info/?l=linux-scsi&m=123498532419609 Open Source is a co-operative effort: The only thing you get at the end of the day is kudos for finding (and possibly fixing) a bug, so since Yinghai Lu did all the heavy lifting in this regard, I'll take this as an ack of his patch. James -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
