Re: [PATCH 425/493] scsi: remove use of __devexit

2012-11-22 Thread Guennadi Liakhovetski
On Mon, 19 Nov 2012, Bill Pemberton wrote: > CONFIG_HOTPLUG is going away as an option so __devexit is no > longer needed. > drivers/scsi/tmscsim.c| 2 +- Acked-by: Guennadi Liakhovetski Thanks Guennadi --- Guennadi Liakhovetski, Ph.D. Freelance Open-Source Software Develop

[PATCH] iscsit: use GFP_ATOMIC under spin lock

2012-11-22 Thread Wei Yongjun
From: Wei Yongjun The function iscsit_build_conn_drop_async_message() is called from iscsit_close_connection() with spin lock 'sess->conn_lock' held, so we should use GFP_ATOMIC instead of GFP_KERNEL. Signed-off-by: Wei Yongjun --- drivers/target/iscsi/iscsi_target.c | 2 +- 1 file changed, 1

[PATCH 1/3] scsi: sd: set valid return value in init_sd()

2012-11-22 Thread Guo Chao
init_sd() forgets to set error code when we fail to create slab cache, thus return 0 in a failed path. Return ENOMEM in this case. Signed-off-by: Guo Chao --- drivers/scsi/sd.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/scsi/sd.c b/drivers/scsi/sd.c index 352bc77..4cc277d 100

[PATCH 3/3] scsi: sd: remove unnecessary initialization in sd_probe()

2012-11-22 Thread Guo Chao
We use kzalloc() to allocate scsi_disk, no need to reset ->openers. Signed-off-by: Guo Chao --- drivers/scsi/sd.c |1 - 1 file changed, 1 deletion(-) diff --git a/drivers/scsi/sd.c b/drivers/scsi/sd.c index e934ffc..d3b66a2 100644 --- a/drivers/scsi/sd.c +++ b/drivers/scsi/sd.c @@ -2897,7 +

[PATCH 2/3] scsi: sd: no need to set gendisk->minors in sd_probe_async()

2012-11-22 Thread Guo Chao
We already call alloc_disk() with SD_MINORS and ->minors will be always set there. Signed-off-by: Guo Chao --- drivers/scsi/sd.c |1 - 1 file changed, 1 deletion(-) diff --git a/drivers/scsi/sd.c b/drivers/scsi/sd.c index 4cc277d..e934ffc 100644 --- a/drivers/scsi/sd.c +++ b/drivers/scsi/sd

Re: [PATCH 097/493] scsi: remove use of __devexit_p

2012-11-22 Thread Guennadi Liakhovetski
On Mon, 19 Nov 2012, Bill Pemberton wrote: > CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer > needed. > drivers/scsi/tmscsim.c| 2 +- Acked-by: Guennadi Liakhovetski Thanks Guennadi --- Guennadi Liakhovetski, Ph.D. Freelance Open-Source Software Devel

Re: [PATCH 319/493] scsi: remove use of __devinitdata

2012-11-22 Thread Guennadi Liakhovetski
On Mon, 19 Nov 2012, Bill Pemberton wrote: > drivers/scsi/tmscsim.c | 2 +- Acked-by: Guennadi Liakhovetski Thanks Guennadi --- Guennadi Liakhovetski, Ph.D. Freelance Open-Source Software Developer http://www.open-technology.de/ -- To unsubscribe from this list: send the line "uns

[GIT PULL] SCSI fixes for 3.7-rc6

2012-11-22 Thread James Bottomley
This is a set of four bug fixes. The isci one is an obvious thinko (using request buffer instead of response buffer) which causes a command to fail. The three others are DIF/DIX updates which are required because they're part of a series of ten patches, the other seven of which went into the bloc