[PATCH 06/32] xen blkback: prepare for bi_rw split

2015-11-04 Thread mchristi
From: Mike Christie This patch prepares xen blkback submit_bio use for the next patches that split bi_rw into a operation and flags field. Instead of passing in a bitmap with both the operation and flags mixed in, the callers will now pass them in seperately. This patch modifies the code related

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

2015-11-04 Thread Hannes Reinecke
On 11/04/2015 12:46 PM, Laurent Vivier wrote: > > > On 04/11/2015 12:16, Hannes Reinecke wrote: >> On 11/04/2015 11:20 AM, Laurent Vivier wrote: >>> QEMU allows until 32 LUNs. >>> >>> Signed-off-by: Laurent Vivier >>> --- >>> drivers/scsi/ibmvscsi/ibmvscsi.c | 5 - >>> 1 file changed, 4 ins

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

2015-11-04 Thread Laurent Vivier
On 04/11/2015 12:16, Hannes Reinecke wrote: > On 11/04/2015 11:20 AM, Laurent Vivier wrote: >> QEMU allows until 32 LUNs. >> >> Signed-off-by: Laurent Vivier >> --- >> drivers/scsi/ibmvscsi/ibmvscsi.c | 5 - >> 1 file changed, 4 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/scsi/i

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

2015-11-04 Thread Hannes Reinecke
On 11/04/2015 11:20 AM, Laurent Vivier wrote: > QEMU allows until 32 LUNs. > > Signed-off-by: Laurent Vivier > --- > drivers/scsi/ibmvscsi/ibmvscsi.c | 5 - > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/drivers/scsi/ibmvscsi/ibmvscsi.c > b/drivers/scsi/ibmvscsi/ibmvscsi

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

2015-11-04 Thread Laurent Vivier
On 04/11/2015 11:20, Laurent Vivier wrote: > QEMU allows until 32 LUNs. > > Signed-off-by: Laurent Vivier I forget the: Reviewed-by: Brian King Acked-by: Tyrel Datwyler > --- > drivers/scsi/ibmvscsi/ibmvscsi.c | 5 - > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/d

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

2015-11-04 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 --- v2: fix format of max_lun (u64): use %llu drivers/scsi/ibmvscsi/ib

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

2015-11-04 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 --- drivers/scsi/ibmvscsi/ibmvscsi.c | 4 ++-- 1 file change

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

2015-11-04 Thread Laurent Vivier
QEMU allows until 32 LUNs. Signed-off-by: Laurent Vivier --- drivers/scsi/ibmvscsi/ibmvscsi.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/scsi/ibmvscsi/ibmvscsi.c b/drivers/scsi/ibmvscsi/ibmvscsi.c index 04de287..4480d3e 100644 --- a/drivers/scsi/ibmvscsi/ibmv

[PATCH v2 0/3] ibmvscsi parameter cleanup

2015-11-04 Thread Laurent Vivier
v2 of this series only fix the format type of max_lun: drivers/scsi/ibmvscsi/ibmvscsi.c:2298:4: warning: format '%d' expects argument of type 'int', but argument 4 has type 'u64 {aka long long unsigned int}' [-Wformat=] "Maximum ID: %d Maximum LUN: %d Maximum Channel: %d\n", ^ Laur

[PATCH] st: trivial: remove form feed characters

2015-11-04 Thread Maurizio Lombardi
Signed-off-by: Maurizio Lombardi --- drivers/scsi/st.c | 24 1 file changed, 8 insertions(+), 16 deletions(-) diff --git a/drivers/scsi/st.c b/drivers/scsi/st.c index b37b9b0..7c4e518 100644 --- a/drivers/scsi/st.c +++ b/drivers/scsi/st.c @@ -226,7 +226,6 @@ static DEFIN

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

2015-11-04 Thread Rasmus Villemoes
On Wed, Nov 04 2015, James Bottomley wrote: > On Wed, 2015-11-04 at 00:26 +0100, Rasmus Villemoes wrote: >> On Tue, Nov 03 2015, James Bottomley >> wrote: >> >> Please spell it U32_MAX >> > >> > Why? there's no reason not to use the arithmetic UINT_MAX here. Either >> > works, of course but

<    1   2