Re: [PATCH 0/2] target: Fixes for AllRegistrants reservation handling

2014-12-19 Thread Nicholas A. Bellinger
Hi Doug, On Thu, 2014-12-18 at 11:53 -0500, Douglas Gilbert wrote: SNIP FreeNAS's iSCSI target also implements ODX which is one reason why I have been looking at it (hint to NAB). With ODX it does cut one significant corner in only supporting access upon reference RODs. Its ODX

[PATCH V2 1/4] be2iscsi : Fix the retry count for boot targets

2014-12-19 Thread John Soni Jose
Increment the retry count to get the boot target info when port async event is received by the driver. Update sysfs enteries with the boot target parameters. Signed-off-by: Minh Tran minhduc.t...@emulex.com Signed-off-by: John Soni Jose sony.joh...@emulex.com Signed-off-by: Jayamohan

[PATCH V2 2/4] be2iscsi : Fix the PCI request region reserving.

2014-12-19 Thread John Soni Jose
Reserve the device PCI I/O and Memory resources. Signed-off-by: John Soni Jose sony.joh...@emulex.com Signed-off-by: Jayamohan Kallickal jayamohan.kallic...@emulex.com --- drivers/scsi/be2iscsi/be_main.c | 22 ++ 1 files changed, 18 insertions(+), 4 deletions(-) diff

[PATCH V2 0/4] be2iscsi driver update to 10.4.200.0

2014-12-19 Thread John Soni Jose
The patchset updates be2iscsi driver to 10.4.200.0. These patches are generated aganist the SCSI for-next branch. The changes suggested by Mike C is also done in this patch-set. Please ingnore the earlier sent patches. John Soni Jose (4): be2iscsi : Fix the retry count for boot

[PATCH V2 2/4] be2iscsi : Fix the PCI request region reserving.

2014-12-19 Thread John Soni Jose
Reserve the device PCI I/O and Memory resources. Signed-off-by: John Soni Jose sony.joh...@emulex.com Signed-off-by: Jayamohan Kallickal jayamohan.kallic...@emulex.com --- drivers/scsi/be2iscsi/be_main.c | 22 ++ 1 files changed, 18 insertions(+), 4 deletions(-) diff

[PATCH V2 3/4] be2iscsi : Fix memory leak in the unload path

2014-12-19 Thread John Soni Jose
Driver was not freeing the DMA memory allocated for EQ/CQ in the unload path. This patch frees the DMA memory during the driver unload. Signed-off-by: John Soni Jose sony.joh...@emulex.com Signed-off-by: Jayamohan Kallickal jayamohan.kallic...@emulex.com --- drivers/scsi/be2iscsi/be_main.c |

[PATCH V2 4/4] be2iscsi : Bump the driver version

2014-12-19 Thread John Soni Jose
Bump the driver version Signed-off-by: John Soni Jose sony.joh...@emulex.com Signed-off-by: Jayamohan Kallickal jayamohan.kallic...@emulex.com --- drivers/scsi/be2iscsi/be_main.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/scsi/be2iscsi/be_main.h

[Patch] Fix regression in 3.18 caused by e05fe29248 qla2xxx: Honor FCP_RSP retry delay timer field.

2014-12-19 Thread Bruno Prémont
Commit e05fe29248 (qla2xxx: Honor FCP_RSP retry delay timer field.) causes systems to busy-wait for about 3 minutes after boot prior to detecting SAN disks. During this wait period one kworker is running full-time (though /proc/pid/stack has no useful data). Another kworker is waiting for IO to

[GIT PULL] target updates for v3.19-rc1

2014-12-19 Thread Nicholas A. Bellinger
Hello Linus, Here are the target-pending updates for v3.19-rc1 code. Please go ahead and pull from: git://git.kernel.org/pub/scm/linux/kernel/git/nab/target-pending.git for-next The highlights this merge window include: - Allow target fabric drivers to function as built-in. (Roland) -

[PATCH 0/3] Minor clean up in mpt2sas

2014-12-19 Thread Quentin Lambert
The first patch merges two similar functions. Since the resulting function is only called three times, it adds a parameter to differenciate between the two behaviours. The second patch proposes a simplification of the function code. The function previously used a variable to skip through the rest

[PATCH 1/3] [SCSI] mpt2sas: Merge two similar functions

2014-12-19 Thread Quentin Lambert
mpt2sas_scsih_set_tm_flag and mpt2sas_scsih_clear_tm_flag shared a lot of code. Therefore, they are merged into one function whose behaviour is given by a new parameter named action. Signed-off-by: Quentin Lambert lambert.quen...@gmail.com --- drivers/scsi/mpt2sas/mpt2sas_scsih.c | 43

[PATCH 2/3] [SCSI] mpt2sas: Simplify the code of mpt2sas_scsih_tm_flag

2014-12-19 Thread Quentin Lambert
Since skip was only used to pass through the rest of the loop, a break statment is called where skip was previously assigned 1. Signed-off-by: Quentin Lambert lambert.quen...@gmail.com --- drivers/scsi/mpt2sas/mpt2sas_scsih.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git

[PATCH 3/3] [SCSI] mpt2sas: Remove unnecessary use of a boolean variable

2014-12-19 Thread Quentin Lambert
Signed-off-by: Quentin Lambert lambert.quen...@gmail.com --- drivers/scsi/mpt2sas/mpt2sas_base.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/scsi/mpt2sas/mpt2sas_base.c b/drivers/scsi/mpt2sas/mpt2sas_base.c index 58e4521..c31de9d 100644 ---

Re: [PATCH 3/3] [SCSI] mpt2sas: Remove unnecessary use of a boolean variable

2014-12-19 Thread Joe Perches
On Fri, 2014-12-19 at 12:13 +0100, Quentin Lambert wrote: [] diff --git a/drivers/scsi/mpt2sas/mpt2sas_base.c b/drivers/scsi/mpt2sas/mpt2sas_base.c [] @@ -1414,12 +1414,8 @@ _base_enable_msix(struct MPT2SAS_ADAPTER *ioc) struct msix_entry *entries, *a; int r; int i; -

[PATCH v2 0/3] Minor clean up in mpt2sas

2014-12-19 Thread Quentin Lambert
The version 2 of this patch fix the logic error pointed out by Joe Perches. Quentin Lambert (3): [SCSI] mpt2sas: Merge two similar functions [SCSI] mpt2sas: Simplify the code of mpt2sas_scsih_tm_flag [SCSI] mpt2sas: Remove unnecessary use of a boolean variable

[PATCH v2 1/3] [SCSI] mpt2sas: Merge two similar functions

2014-12-19 Thread Quentin Lambert
mpt2sas_scsih_set_tm_flag and mpt2sas_scsih_clear_tm_flag shared a lot of code. Therefore, they are merged into one function whose behaviour is given by a new parameter named action. Signed-off-by: Quentin Lambert lambert.quen...@gmail.com --- drivers/scsi/mpt2sas/mpt2sas_scsih.c | 43

[PATCH v2 2/3] [SCSI] mpt2sas: Simplify the code of mpt2sas_scsih_tm_flag

2014-12-19 Thread Quentin Lambert
Since skip was only used to pass through the rest of the loop, a break statment is called where skip was previously assigned 1. Signed-off-by: Quentin Lambert lambert.quen...@gmail.com --- drivers/scsi/mpt2sas/mpt2sas_scsih.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git

[PATCH v2 3/3] [SCSI] mpt2sas: Remove unnecessary use of a boolean variable

2014-12-19 Thread Quentin Lambert
Signed-off-by: Quentin Lambert lambert.quen...@gmail.com --- drivers/scsi/mpt2sas/mpt2sas_base.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/scsi/mpt2sas/mpt2sas_base.c b/drivers/scsi/mpt2sas/mpt2sas_base.c index 58e4521..6cc0591 100644 ---

[PATCH v2 0/1] int to bool conversion

2014-12-19 Thread Quentin Lambert
The second version of this patch did not treat the cases where the use of boolean variable was unnecessary. These cases are now treated in a patchset that was already submitted. It also left out the cases treated by one Dan Carpenter's patch currently being reviewed:

[PATCH v2 1/1] [SCSI] mpt2sas: Convert non-returned local variable to boolean when relevant

2014-12-19 Thread Quentin Lambert
This patch was produced using Coccinelle. A simplified version of the semantic patch is: @r exists@ identifier f; local idexpression u8 x; identifier xname; @@ f(...) { ...when any ( x@xname = 1; | x@xname = 0; ) ...when any } @bad exists@ identifier r.f; local idexpression u8 r.x

[LSF/MM TOPIC] ZAC/ZBC Update

2014-12-19 Thread Hannes Reinecke
Hi all, (my favourite topic again :-) I'd like to give a short update for ZAC/ZBC implementation. The T10 and T13 standards have been updated with some features like SATA autosense which requires changes to the various subsystems and the implementation itself. These features would also allow

Re: [PATCH v5 2/3] libata: async resume

2014-12-19 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On 03/05/2014 03:17 PM, Dan Williams wrote: From: Todd Brandt todd.e.bra...@linux.intel.com Improve overall system resume time by making libata link recovery actions asynchronous relative to other resume events. Link resume operations are

Re: [PATCH v5 2/3] libata: async resume

2014-12-19 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On 12/19/2014 06:46 PM, Phillip Susi wrote: I realize this is a little late but I finally started looking at the patch set I was working on last year again, and now that I look at your version that was accepted, I realize that it only addresses