Can I get another review for this series? On Wed, Jun 04, 2014 at 12:49:42PM -0400, Joe Lawrence wrote: > While reviewing the mpt2/mpt3 static checker fixup patchset, Christoph > inquired about mptfusion. None of the sparse / smatch warnings from the > earlier patchset directly apply to fusion, but there were a few easy to > fix warnings (compile tested only). > > The patchset is ordered from the smallest/easiest change up to the last > three, which are bit more involved and should be reviewed by LSI, > especially the last one "mptfusion: tweak null pointer checks". See the > commentary in those patches after the signed-off-by line. > > Cc: Christoph Hellwig <hch@xxxxxxxxxxxxx> > Cc: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > Cc: Sreekanth Reddy <Sreekanth.Reddy@xxxxxxx> > > Joe Lawrence (7): > mptfusion: mark file-private functions as static > mptfusion: remove redundant kfree checks > mptfusion: initChainBuffers should return errno > mptfusion: zero kernel-space source of copy_to_user > mptfusion: make adapter prod_name[] a pointer > mptfusion: combine fw_event_work and its event_data > mptfusion: tweak null pointer checks > > drivers/message/fusion/mptbase.c | 29 +++++++++---------- > drivers/message/fusion/mptbase.h | 2 +- > drivers/message/fusion/mptctl.c | 2 +- > drivers/message/fusion/mptfc.c | 3 +- > drivers/message/fusion/mptsas.c | 57 +++++++++++++++++++------------------ > drivers/message/fusion/mptsas.h | 2 +- > drivers/message/fusion/mptscsih.c | 7 +++-- > drivers/message/fusion/mptspi.c | 5 ++-- > 8 files changed, 52 insertions(+), 55 deletions(-) > > -- > 1.7.10.4 > > -- > 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 ---end quoted text--- -- 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
