RE: [PATCH v2] megaraid_sas : Add locking to megasas_aen_polling

2015-11-03 Thread Sumit Saxena
> -Original Message- > From: Ben Guthro [mailto:ben.gut...@gmail.com] On Behalf Of Ben Guthro > Sent: Monday, November 02, 2015 5:49 PM > To: megaraidlinux@avagotech.com; linux-scsi@vger.kernel.org > Cc: Glenn Watkins; Ben Guthro; Yang, Bo; sta...@vger.kernel.org > Subject: [PATCH v2]

ATTENTION

2015-11-03 Thread
Good day My Dear, I am seeking the immediate assistance,of a reliable and trustworthy Indian businessman who understands India local language (Hindi) and a little English; to handle a profitable business for our mutual benefit. I await your reply for more details. Thanks and best regards, Dr.

Re: [PATCH v4 11/11] scsi: ufs-exynos: add UFS host support for Exynos SoCs

2015-11-03 Thread Alim Akhtar
Hi Kishon, Thanks for your time. On 10/28/2015 06:23 PM, Kishon Vijay Abraham I wrote: Hi, On Sunday 25 October 2015 05:34 PM, Alim Akhtar wrote: Hi Kishon Thanks again for you review. On Fri, Oct 23, 2015 at 8:48 PM, Kishon Vijay Abraham I wrote: Hi, On Thursday 15 October

Re: [PATCH 1 22/25] hpsa: enhance device messages

2015-11-03 Thread Tomas Henzl
On 2.11.2015 17:54, Don Brace wrote: > On 10/30/2015 09:32 AM, Tomas Henzl wrote: >> RAID_UNKNOWN is used in few other places - raid_level_show for example, > raid_level_show handles physical devices by using "N/A", otherwise it > displays > the RAID level for logical devices. > > Other functions

Re: [PATCH v2 27/32] scsi: hisi_sas: add smp protocol support

2015-11-03 Thread Arnd Bergmann
On Tuesday 03 November 2015 11:42:30 John Garry wrote: > > > There seems to be some misunderstanding. Are you suggesting I change > sas_smp_task? > ./include/scsi/libsas.h > struct sas_smp_task { > struct scatterlist smp_req; > struct scatterlist smp_resp; > }; > Ah, no. I had

Re: [PATCH] pm80xx: remove the SCSI host before detaching from SAS transport

2015-11-03 Thread Christoph Hellwig
On Mon, Nov 02, 2015 at 09:29:13AM +0100, Jack Wang wrote: > Thanks for digging it down. Looks good to me. > > Seems other libsas based driver all affected, maybe we should also fix them? > Christoph (cc-ed), could you share your opinion about this fix, as the > commit cff549e4860f is from you?

Re: [PATCH v2 27/32] scsi: hisi_sas: add smp protocol support

2015-11-03 Thread John Garry
On 02/11/2015 20:29, Arnd Bergmann wrote: On Monday 02 November 2015 17:03:58 John Garry wrote: Can do. Actually sg_req seems only ever has one element: expander.c, smp_execute_task() sg_init_one(>smp_task.smp_req, req, req_size); I tried replacing with dma_map_single, but I feel the code

[PATCH v2 2/5] ipr: Don't set NO_ULEN_CHK bit when resource is a vset.

2015-11-03 Thread Gabriel Krisman Bertazi
According to the IPR specification, Inhibit Underlength Checking bit must be disabled when issuing commands to vsets. Enabling it in this case might cause SCSI commands to fail with an Illegal Request, so make sure we keep this bit cleared when resource is a vset. Changes since v1: - Put

[PATCH v2 3/5] ipr: Inquiry IOA page 0xC4 during initialization.

2015-11-03 Thread Gabriel Krisman Bertazi
Add an IOA Inquiry command for Page 0xC4 during IOA initialization to collect cache capabilities, particularly to check if Sync IOA Write Cache is supported. Inquiry will happen right after Cap Inquiry on page 0xD0; and will execute only if the "Supported Pages" field in Inquiry Page 0x0 shows

[PATCH v2 5/5] ipr: Driver version 2.6.3.

2015-11-03 Thread Gabriel Krisman Bertazi
Signed-off-by: Gabriel Krisman Bertazi --- drivers/scsi/ipr.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/ipr.h b/drivers/scsi/ipr.h index b16bcd1..a34c7a5 100644 --- a/drivers/scsi/ipr.h +++ b/drivers/scsi/ipr.h @@ -39,8 +39,8

[PATCH v2 1/5] ipr: Add delay to ensure coherent dumps.

2015-11-03 Thread Gabriel Krisman Bertazi
Add a holding pattern prior to collecting dump data, to wait for the IOA indication that the Mailbox register is stable and won't change without an explicit reset. This ensures we'll be collecting meaningful dump data, even when dumping right after an adapter reset. In the event of a timeout, we

[PATCH v2 4/5] ipr: Issue Configure Cache Parameters command.

2015-11-03 Thread Gabriel Krisman Bertazi
Some new adapters require a special Configure Cache Parameters command to enable the adapter write cache, so send this during the adapter initialization if the adapter requires it. Changes since v1: - Fix checkpatch.pl warnings. Signed-off-by: Gabriel Krisman Bertazi

Re: [PATCH v2] string_helpers: fix precision loss for some inputs

2015-11-03 Thread James Bottomley
On Tue, 2015-11-03 at 23:13 +0100, Rasmus Villemoes wrote: > On Tue, Nov 03 2015, James Bottomley > wrote: > > > From: James Bottomley > > > > It was noticed that we lose precision in the final calculation for some > > inputs. The

[PATCH v3] string_helpers: fix precision loss for some inputs

2015-11-03 Thread James Bottomley
From: James Bottomley It was noticed that we lose precision in the final calculation for some inputs. The most egregious example is size=3000 blk_size=1900 in units of 10 should yield 5.70 MB but in fact yields 3.00 MB (oops). This is because the current algorithm doesn't

Re: [PATCH v2] string_helpers: fix precision loss for some inputs

2015-11-03 Thread Rasmus Villemoes
On Tue, Nov 03 2015, James Bottomley wrote: > From: James Bottomley > > It was noticed that we lose precision in the final calculation for some > inputs. The most egregious example is size=3000 blk_size=1900 in units of 10 > should

Re: [PATCH v2 2/5] ipr: Don't set NO_ULEN_CHK bit when resource is a vset.

2015-11-03 Thread Manoj Kumar
Gabriel: On applying this patch, I noticed that this statement seems to be unnecessary: else ioarcb->cmd_pkt.flags_lo |= IPR_FLAGS_LO_UNTAGGED_TASK; As the value is 0x00: #define IPR_FLAGS_LO_UNTAGGED_TASK 0x00 You can resolve this in a

Re: [PATCH v2] string_helpers: fix precision loss for some inputs

2015-11-03 Thread James Bottomley
From: James Bottomley It was noticed that we lose precision in the final calculation for some inputs. The most egregious example is size=3000 blk_size=1900 in units of 10 should yield 5.70 MB but in fact yields 3.00 MB (oops). This is because the current algorithm doesn't

[PATCH] string_helpers: fix precision loss for some inputs

2015-11-03 Thread James Bottomley
From: James Bottomley It was noticed that we lose precision in the final calculation for some inputs. The most egregious example is size=3000 blk_size=1900 in units of 10 should yield 5.70 MB but in fact yields 3.00 MB (oops). This is because the current algorithm doesn't

Re: [PATCH v2] string_helpers: fix precision loss for some inputs

2015-11-03 Thread Rasmus Villemoes
On Tue, Nov 03 2015, James Bottomley wrote: > On Tue, 2015-11-03 at 23:13 +0100, Rasmus Villemoes wrote: >> On Tue, Nov 03 2015, James Bottomley >> wrote: >> >> > From: James Bottomley >> > >>

Re: [PATCH v2] string_helpers: fix precision loss for some inputs

2015-11-03 Thread James Bottomley
On Wed, 2015-11-04 at 00:26 +0100, Rasmus Villemoes wrote: > On Tue, Nov 03 2015, James Bottomley > wrote: > > > On Tue, 2015-11-03 at 23:13 +0100, Rasmus Villemoes wrote: > >> On Tue, Nov 03 2015, James Bottomley > >>

Re: [PATCH v2 4/5] ipr: Issue Configure Cache Parameters command.

2015-11-03 Thread Brian King
Acked-by: Brian King -- Brian King Power Linux I/O IBM Linux Technology Center -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majord...@vger.kernel.org More majordomo info at

Re: [PATCH v2 5/5] ipr: Driver version 2.6.3.

2015-11-03 Thread Brian King
Acked-by: Brian King -- Brian King Power Linux I/O IBM Linux Technology Center -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majord...@vger.kernel.org More majordomo info at

Re: [PATCH v2 3/5] ipr: Inquiry IOA page 0xC4 during initialization.

2015-11-03 Thread Brian King
Acked-by: Brian King -- Brian King Power Linux I/O IBM Linux Technology Center -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majord...@vger.kernel.org More majordomo info at

Re: [PATCH v2 2/5] ipr: Don't set NO_ULEN_CHK bit when resource is a vset.

2015-11-03 Thread Brian King
Acked-by: Brian King -- Brian King Power Linux I/O IBM Linux Technology Center -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majord...@vger.kernel.org More majordomo info at

Re: [PATCH v2 1/5] ipr: Add delay to ensure coherent dumps.

2015-11-03 Thread Brian King
Acked-by: Brian King -- Brian King Power Linux I/O IBM Linux Technology Center -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majord...@vger.kernel.org More majordomo info at

Re: [PATCH v2] string_helpers: fix precision loss for some inputs

2015-11-03 Thread James Bottomley
On Tue, 2015-11-03 at 13:37 -0800, Bart Van Assche wrote: > On 11/03/2015 01:21 PM, James Bottomley wrote: > > + while (blk_size >= UINT_MAX) > > i++; > > (reduced CC-list) Let's keep at least the lists in cc. > Hello James, > > Is the above loop an infinite loop if blk_size >=

[PATCH 1/3] ibmvscsi: make parameters max_id and max_channel read-only

2015-11-03 Thread Laurent Vivier
The value of the parameter is never re-read by the driver, so a new value is ignored. Let know the user he can't modify it by removing writable attribute. Signed-off-by: Laurent Vivier Reviewed-by: Brian King Acked-by: Tyrel Datwyler

Re: [PATCH 2/3] ibmvscsi: display default value for max_id, max_lun and max_channel.

2015-11-03 Thread kbuild test robot
Hi Laurent, [auto build test WARNING on scsi/for-next] [also WARNING on: v4.3 next-20151103] url: https://github.com/0day-ci/linux/commits/Laurent-Vivier/ibmvscsi-make-parameters-max_id-and-max_channel-read-only/20151103-223706 base: https://github.com/0day-ci/linux Laurent-Vivier

[PATCH 2/3] ibmvscsi: display default value for max_id, max_lun and max_channel.

2015-11-03 Thread Laurent Vivier
As devices with values greater than that are silently ignored, this gives some hints to the sys admin to know why he doesn't see his devices... Signed-off-by: Laurent Vivier Reviewed-by: Brian King Acked-by: Tyrel Datwyler

[PATCH 3/3] ibmvscsi: Allow to configure maximum LUN

2015-11-03 Thread Laurent Vivier
QEMU allows until 32 LUNs. Signed-off-by: Laurent Vivier Reviewed-by: Brian King Acked-by: Tyrel Datwyler --- drivers/scsi/ibmvscsi/ibmvscsi.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git

Awards

2015-11-03 Thread RENEE CAMERON
Prize Notification Contact: unitednationdiplomaticen...@outlook.it -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html