scsi_debug.c:1626: possible bad test ?

2014-12-22 Thread David Binderman
Hello there, drivers/scsi/scsi_debug.c:1626:20: warning: logical ‘and’ of mutually exclusive tests is always false [-Wlogical-op] Source code is     if (alloc_len 4 alloc_len 0x) { Suggest swap for ||. Adding -Wlogical-op to the build might also help Regards David Binderman

[PATCH] scsi_debug: fix sanity check in resp_rsup_opcodes

2014-12-22 Thread Douglas Gilbert
David Binderman pointed out in this post: http://marc.info/?l=linux-scsim=141924469523474w=2 a sanity check that was flawed. See attached fix. Should be applied to lk 3.19.0-rc1 and Christoph's branches. ChangeLog: fix alloc_len sanity check in resp_rsup_opcodes() ---

Re: [PATCH] scsi_debug: fix sanity check in resp_rsup_opcodes

2014-12-22 Thread Ewan Milne
On Mon, 2014-12-22 at 11:37 -0500, Douglas Gilbert wrote: David Binderman pointed out in this post: http://marc.info/?l=linux-scsim=141924469523474w=2 a sanity check that was flawed. See attached fix. Should be applied to lk 3.19.0-rc1 and Christoph's branches. ChangeLog: fix

Re: [PATCH] scsi: 3w-9xxx.c: Cleaning up missing null-terminate in conjunction with strncpy

2014-12-22 Thread Rickard Strandqvist
2014-08-01 0:19 GMT+02:00 adam radford aradf...@gmail.com: On Sun, Jul 27, 2014 at 8:11 AM, Rickard Strandqvist rickard_strandqv...@spectrumdigital.se wrote: Replacing strncpy with strlcpy to avoid strings that lacks null terminate. And use the sizeof on the to string rather than strlen on the

Re: [PATCH] scsi: 3w-9xxx.c: Cleaning up missing null-terminate in conjunction with strncpy

2014-12-22 Thread adam radford
On Mon, Dec 22, 2014 at 2:52 PM, Rickard Strandqvist rickard_strandqv...@spectrumdigital.se wrote: 2014-08-01 0:19 GMT+02:00 adam radford aradf...@gmail.com: On Sun, Jul 27, 2014 at 8:11 AM, Rickard Strandqvist rickard_strandqv...@spectrumdigital.se wrote: Replacing strncpy with strlcpy to

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

2014-12-22 Thread Mike Christie
On 12/19/2014 06:51 PM, John Soni Jose wrote: @@ -5376,8 +5387,14 @@ beiscsi_hw_health_check(struct work_struct *work) be_eqd_update(phba); if (phba-state BE_ADAPTER_CHECK_BOOT) { - phba-state = ~BE_ADAPTER_CHECK_BOOT; - be_check_boot_session(phba); +

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

2014-12-22 Thread Mike Christie
On 12/22/2014 05:47 PM, Mike Christie wrote: On 12/19/2014 06:51 PM, John Soni Jose wrote: @@ -5376,8 +5387,14 @@ beiscsi_hw_health_check(struct work_struct *work) be_eqd_update(phba); if (phba-state BE_ADAPTER_CHECK_BOOT) { -phba-state = ~BE_ADAPTER_CHECK_BOOT; -

Re: [PATCH 3/4 v3] scsi:stex.c Add reboot support

2014-12-22 Thread Charles Chiou
On 12/18/2014 11:41 AM, Charles Chiou wrote: On 12/17/2014 05:30 PM, Christoph Hellwig wrote: I'm not going to add a new reboot notifier for a driver. If you can convince the driver model / PM people to pass this information to -shutdown we can add support for this difference, but not in