Re: [PATCH] scsi: mvsas: Replace pci_pool_alloc by pci_pool_zalloc

2016-12-14 Thread Souptick Joarder
Martin, On Wed, Dec 14, 2016 at 7:22 AM, Martin K. Petersen wrote: >> "Souptick" == Souptick Joarder writes: > > Souptick, > > Sorry about the delay. Been out for a few days. > Inside mvs_task_prep(), pci_pool_alloc() followed by memset will be replaced by pci_pool_zalloc() > > Sou

Re: [PATCH] scsi: mvsas: Replace pci_pool_alloc by pci_pool_zalloc

2016-12-13 Thread Martin K. Petersen
> "Souptick" == Souptick Joarder writes: Souptick, Sorry about the delay. Been out for a few days. >>> Inside mvs_task_prep(), pci_pool_alloc() followed by memset will be >>> replaced by pci_pool_zalloc() Souptick> Any further comment on this ? I took one of your other patches because the

Re: [PATCH] scsi: mvsas: Replace pci_pool_alloc by pci_pool_zalloc

2016-12-11 Thread Souptick Joarder
Hi Martin, On Tue, Dec 6, 2016 at 6:58 PM, Johannes Thumshirn wrote: > On Mon, Nov 28, 2016 at 04:56:26PM +0530, Souptick Joarder wrote: >> Inside mvs_task_prep(), pci_pool_alloc() followed by memset will be >> replaced by pci_pool_zalloc() >> >> Signed-off-by: Souptick joarder >> --- > > FWIW,

Re: [PATCH] scsi: mvsas: Replace pci_pool_alloc by pci_pool_zalloc

2016-12-06 Thread Johannes Thumshirn
On Mon, Nov 28, 2016 at 04:56:26PM +0530, Souptick Joarder wrote: > Inside mvs_task_prep(), pci_pool_alloc() followed by memset will be > replaced by pci_pool_zalloc() > > Signed-off-by: Souptick joarder > --- FWIW, Reviewed-by: Johannes Thumshirn -- Johannes Thumshirn

Re: [PATCH] scsi: mvsas: Replace pci_pool_alloc by pci_pool_zalloc

2016-12-06 Thread Souptick Joarder
Hi Martin, On Tue, Dec 6, 2016 at 3:34 AM, Martin K. Petersen wrote: >> "Souptick" == Souptick Joarder writes: > > Souptick, > > Souptick> Any comment on this patch? > > The patch looked OK to me when you posted it. However, you need one > person in addition to me to review it. And you need

Re: [PATCH] scsi: mvsas: Replace pci_pool_alloc by pci_pool_zalloc

2016-12-05 Thread Martin K. Petersen
> "Souptick" == Souptick Joarder writes: Souptick, Souptick> Any comment on this patch? The patch looked OK to me when you posted it. However, you need one person in addition to me to review it. And you need convince us of the merit of a presumably untested change to an unmaintained driver.

Re: [PATCH] scsi: mvsas: Replace pci_pool_alloc by pci_pool_zalloc

2016-12-04 Thread Souptick Joarder
Hi Martin, Any comment on this patch? On Thu, Dec 1, 2016 at 11:34 AM, Souptick Joarder wrote: > On Mon, Nov 28, 2016 at 4:56 PM, Souptick Joarder > wrote: >> Inside mvs_task_prep(), pci_pool_alloc() followed by memset will be >> replaced by pci_pool_zalloc() >> >> Signed-off-by: Souptick joar

Re: [PATCH] scsi: mvsas: Replace pci_pool_alloc by pci_pool_zalloc

2016-11-30 Thread Souptick Joarder
On Mon, Nov 28, 2016 at 4:56 PM, Souptick Joarder wrote: > Inside mvs_task_prep(), pci_pool_alloc() followed by memset will be > replaced by pci_pool_zalloc() > > Signed-off-by: Souptick joarder > --- > drivers/scsi/mvsas/mv_sas.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > dif

[PATCH] scsi: mvsas: Replace pci_pool_alloc by pci_pool_zalloc

2016-11-28 Thread Souptick Joarder
Inside mvs_task_prep(), pci_pool_alloc() followed by memset will be replaced by pci_pool_zalloc() Signed-off-by: Souptick joarder --- drivers/scsi/mvsas/mv_sas.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/scsi/mvsas/mv_sas.c b/drivers/scsi/mvsas/mv_sas.c index