On Mon, 2013-05-06 at 09:36 -0700, Anand wrote:
> diff --git a/drivers/scsi/pm8001/pm80xx_hwi.h
> b/drivers/scsi/pm8001/pm80xx_hwi.h
> index 0c15627..ea127b6 100644
> --- a/drivers/scsi/pm8001/pm80xx_hwi.h
> +++ b/drivers/scsi/pm8001/pm80xx_hwi.h
[...]
> @@ -996,6 +1020,23 @@ struct ssp_coalesced_
On Mon, 2013-05-06 at 09:35 -0700, Anand wrote:
> >From 2738e0b59fe103a363b0ea600b3eccc687a7fa3c Mon Sep 17 00:00:00 2001
> From: Sakthivel K
> Date: Tue, 19 Mar 2013 18:08:08 +0530
> Subject: [PATCH V5 11/12] pm80xx: NCQ error handling changes
>
> Handled NCQ errors in the low level driver as th
On Tuesday, May 07, 2013, Santosh Y wrote:
> On Mon, May 6, 2013 at 11:07 AM, Seungwon Jeon wrote:
> > Simplify operations with hiding mmio_base.
> >
> > Signed-off-by: Seungwon Jeon
> > Tested-by: Maya Erez
> > ---
> > drivers/scsi/ufs/ufshcd.c | 105
> > +++--
On Tuesday, May 07, 2013, Santosh Y wrote:
> > /**
> > + * ufshcd_dme_power_xxx - UIC command for DME_POWERON, DME_POWEROFF
> > + * @hba: per adapter instance
> > + * @on: indicate wherter power_on or power_off
> > + *
> > + * Returns 0 on success, non-zero value on failure
> > + */
> > +int ufshc
On Wed, 2013-05-01 at 12:52 -0700, Tim Chen wrote:
> Currently the CRC-T10DIF checksum is computed using a generic table lookup
> algorithm. By switching the checksum to PCLMULQDQ based computation,
> we can speedup the computation by 8x for checksumming 512 bytes and
> even more for larger buffer
Il 20/02/2013 17:12, Paolo Bonzini ha scritto:
> Il 06/02/2013 16:15, Paolo Bonzini ha scritto:
>> This series regards the whitelist that is used for the SG_IO ioctl. This
>> whitelist has three problems:
>>
>> * the bitmap of allowed commands is designed for MMC devices (roughly,
>> "play/burn
> /**
> + * ufshcd_dme_power_xxx - UIC command for DME_POWERON, DME_POWEROFF
> + * @hba: per adapter instance
> + * @on: indicate wherter power_on or power_off
> + *
> + * Returns 0 on success, non-zero value on failure
> + */
> +int ufshcd_dme_power_xxx(struct ufs_hba *hba, u8 on)
> +{
> +
http://marc.info/?l=linux-scsi&m=133769378525796&w=2
Did anything ever happen with this? I don't see that it did.
There seems to be a problem with udev processing an event
for a device at a time when the device can't be accessed.
Mike's original fix was to generate another KOBJ_CHANGE
event when
>
> /**
> + * ufshcd_dme_xxx_set - UIC command for DME_SET, DME_PEER_SET
> + * @hba: per adapter instance
> + * @attr_sel: uic command argument1
> + * @attr_set: attribute set type as uic command argument2
> + * @mib_val: setting value as uic command argument3
> + * @peer: indicate wherter peer or
On Mon, May 6, 2013 at 11:07 AM, Seungwon Jeon wrote:
> Simplify operations with hiding mmio_base.
>
> Signed-off-by: Seungwon Jeon
> Tested-by: Maya Erez
> ---
> drivers/scsi/ufs/ufshcd.c | 105
> +++--
> drivers/scsi/ufs/ufshcd.h |7 +++-
> 2 file
On Mon, 2013-05-06 at 11:41 -0700, Kamaljit Singh wrote:
> Are there any plans to add 32-byte CDB support in Libsas?
I don't understand the question ... it should just work; is it broken?
James
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to
Hello,
Are there any plans to add 32-byte CDB support in Libsas?
Thanks,
Kamaljit Singh
--
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
On Mon, 2013-05-06 at 09:48 -0700, Anand Kumar Santhanam wrote:
> Hi James,
>
> I have resubmitted the patchset after fixing the array out of bounds
> issue. Pls pull the same.
>
> I guess I did not make myself clear with my earlier mail. I compiled my
> earlier patchset using the
> Kernel Makef
Tested-by: Maya Erez
> From: Dolev Raviv
>
> Allow UFS device to complete its initialization and accept SCSI
> commands by setting fDeviceInit flag. The device may take time
> for this operation and hence the host should poll until fDeviceInit
> flag is toggled to zero. This step is mandated by
> On Monday, May 06, 2013, Sujit Reddy Thumma wrote:
>> On 5/6/2013 11:07 AM, Seungwon Jeon wrote:
>> > Link start-up requires long time with multiphase handshakes
>> > between UFS host and device. This affects driver's probe time.
>> > This patch let link start-up run asynchronously. Link start-u
til after v3.10-rc1 is released.
>>>
>>> I am receiving a (un)reasonable number of conflicts from there being
>>> multiple copies of some commits in various trees. Please clean this up
>>> and resist the temptataion to rebase your trees on the way to your
>>&
> "James" == James Bottomley writes:
>> drivers/built-in.o: In function `lpfc_bg_crc': (.text+0x3cb3c9):
>> undefined reference to `crc_t10dif'
James> That's the usual minor config cockup, isn't it? lpfc apparently
James> also needs the generic checksum, so the fix would seem to be
James> t
On 05/06/2013 06:55 AM, Zhao Hongjiang wrote:
> The qla4xxx_sysfs_ddb_add() function uses device_find_child() but it does not
> drop
> the reference of the retrieved child.
>
> Signed-off-by: Zhao Hongjiang
> ---
> drivers/scsi/qla4xxx/ql4_os.c |1 +
> 1 files changed, 1 insertions(+), 0 de
am receiving a (un)reasonable number of conflicts from there being
> > multiple copies of some commits in various trees. Please clean this up
> > and resist the temptataion to rebase your trees on the way to your
> > upstream ...
> >
> > Changes since 20130506:
> >
&g
Hi James,
I have resubmitted the patchset after fixing the array out of bounds
issue. Pls pull the same.
I guess I did not make myself clear with my earlier mail. I compiled my
earlier patchset using the
Kernel Makefile and I did not get the array out of bounds warning even I
had -Wall -O2 flags
On 05/06/2013 04:23 PM, Bjørn Mork wrote:
> "Patrick Monnerat" writes:
>>Create ioctl device node for megaraid_sas driver. Let this node be
>> managed by udev. Fix a typo.
>
> Or maybe just simplify it all and use a misc device instead? See how
> this is done in e.g. drivers/scsi/mpt3sas/mpt3
>From ffe82b1efc42510239046909a7ac0aff10bb4162 Mon Sep 17 00:00:00 2001
From: Sakthivel K
Date: Tue, 19 Mar 2013 18:08:40 +0530
Subject: [PATCH V5 12/12] pm80xx: thermal, sas controller config and error
handling update
Modified thermal configuration to happen after interrupt registration
Added S
>From 2738e0b59fe103a363b0ea600b3eccc687a7fa3c Mon Sep 17 00:00:00 2001
From: Sakthivel K
Date: Tue, 19 Mar 2013 18:08:08 +0530
Subject: [PATCH V5 11/12] pm80xx: NCQ error handling changes
Handled NCQ errors in the low level driver as the FW
is not providing the faulty tag for NCQ errors for libs
>From 6fb08e45a7272daecc4b8bdce643e40fd2062542 Mon Sep 17 00:00:00 2001
From: Sakthivel K
Date: Tue, 19 Mar 2013 18:07:35 +0530
Subject: [PATCH V5 10/12] pm80xx: WWN Modification for PM8081/88/89 controllers
Individual WWN read operations based on controller.
PM8081 - Read WWN from Flash VPD.
PM8
>From 8d5db4d3d00604c8dc3db6e616ed21e0c46bcbaf Mon Sep 17 00:00:00 2001
From: Sakthivel K
Date: Tue, 19 Mar 2013 18:07:09 +0530
Subject: [PATCH V5 09/12] pm80xx: Changed module name and debug messages update
Changed name in driver to pm80xx. Updated debug messages.
Signed-off-by: Sakthivel K
Si
>From 4043d1eb8813a2fc0656815fe2e6d21d4df0da66 Mon Sep 17 00:00:00 2001
From: Sakthivel K
Date: Tue, 19 Mar 2013 18:06:40 +0530
Subject: [PATCH V5 08/12] pm80xx: Firmware flash memory free fix, with addition
of new memory region for it
Performing pci_free_consistent in tasklet had result in a co
>From 2ce3ecb52358fcd358e1ea93a02522084fc7f968 Mon Sep 17 00:00:00 2001
From: Sakthivel K
Date: Tue, 19 Mar 2013 18:05:55 +0530
Subject: [PATCH V5 07/12] pm80xx: SPC new firmware changes for device id 0x8081
alone
Additional bar shift for new SPC firmware, applicable to device
id 0x8081 only.
S
>From c9ef2f3de75ccd7d33ac26ab3e40745ce3134a2b Mon Sep 17 00:00:00 2001
From: Sakthivel K
Date: Tue, 19 Mar 2013 17:56:17 +0530
Subject: [PATCH V5 5/12] pm80xx: MSI-X implementation for using 64 interrupts
Implementation of interrupt handlers and tasklets to support
upto 64 interrupt for the devi
>From c5a2c7eff684b5ece5320f070855e3c9321f9b94 Mon Sep 17 00:00:00 2001
From: Sakthivel K
Date: Wed, 27 Feb 2013 20:27:43 +0530
Subject: [PATCH V5 4/12] pm80xx: Updated common functions common for SPC and
SPCv/ve
Update of function prototype for common function to SPC and SPCv/ve.
Multiple queue
>From 6218dd8466f6771ff46dfcd0a820f853c62cde32 Mon Sep 17 00:00:00 2001
From: Sakthivel K
Date: Wed, 27 Feb 2013 20:25:25 +0530
Subject: [PATCH V5 3/12] pm80xx: Multiple inbound/outbound queue configuration
Memory allocation and configuration of multiple inbound and
outbound queues.
Signed-off-b
>From a33eba8de80389b80fd3a16437e475598854eed7 Mon Sep 17 00:00:00 2001
From: Sakthivel K
Date: Wed, 17 Apr 2013 16:26:36 +0530
Subject: [PATCH V5 02/12] pm80xx: Added SPCv/ve specific ids, variables and
modify for SPC
Updated pci id table with device, vendor, subdevice and subvendor ids
for 808
>From 8464bcd3b7789798a1f34826fe51eb833ae0578d Mon Sep 17 00:00:00 2001
From: Sakthivel K
Date: Mon, 4 Feb 2013 12:10:02 +0530
Subject: [PATCH V5 1/12] pm80xx: fix for memory region free
All memory regions are allocated based on variables total_len
and alignment but free was based on element_size
>From ffe82b1efc42510239046909a7ac0aff10bb4162 Mon Sep 17 00:00:00 2001
From: Sakthivel K
Date: Tue, 19 Mar 2013 18:32:55 +0530
Subject: [PATCH V5 00/12] Support for PMC 8081/8088/8089 controllers
V5
Resubmission of patchset to fix the array out of bounds
error reported by James.
V4
Resubmissi
mits in various trees. Please clean this up
> and resist the temptataion to rebase your trees on the way to your
> upstream ...
>
> Changes since 20130506:
>
on i386:
# CONFIG_CRC_T10DIF is not set
drivers/built-in.o: In function `lpfc_bg_crc':
(.text+0x3cb3c9): undefined
On 05/06/2013 04:23 PM, Bjørn Mork wrote:
"Patrick Monnerat" writes:
Create ioctl device node for megaraid_sas driver. Let this node be
managed by udev. Fix a typo.
Or maybe just simplify it all and use a misc device instead? See how
this is done in e.g. drivers/scsi/mpt3sas/mpt3sas_ctl.c
From: Stephen M. Cameron
In the event of a dma mapping error, we forgot to free
the command we had allocated.
Signed-off-by: Stephen M. Cameron
---
drivers/scsi/hpsa.c | 11 +--
1 files changed, 5 insertions(+), 6 deletions(-)
diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c
i
"Patrick Monnerat" writes:
> From: Patrick Monnerat
>
> Create ioctl device node for megaraid_sas driver. Let this node be
> managed by udev. Fix a typo.
Or maybe just simplify it all and use a misc device instead? See how
this is done in e.g. drivers/scsi/mpt3sas/mpt3sas_ctl.c or
drivers/me
On Mon, 2013-05-06 at 03:40 -0700, me...@codeaurora.org wrote:
> Hi James,
>
> Can you please merge this patch?
Well, no. a) because it's part of a series which still seems to be
under development and b) because it needs a maintainer ack.
James
--
To unsubscribe from this list: send the line "
From: Patrick Monnerat
Create ioctl device node for megaraid_sas driver. Let this node be
managed by udev. Fix a typo.
Reported-by: p...@datasphere.ch
To: Neela Syam Kolli
Cc: "James E.J. Bottomley"
Cc: linux-scsi@vger.kernel.org
Cc: linux-ker...@vger.kernel.org
Signed-off-by: Patrick Monner
The qla4xxx_sysfs_ddb_add() function uses device_find_child() but it does not
drop
the reference of the retrieved child.
Signed-off-by: Zhao Hongjiang
---
drivers/scsi/qla4xxx/ql4_os.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/drivers/scsi/qla4xxx/ql4_os.c b/drive
On Monday, May 06, 2013, Sujit Reddy Thumma wrote:
> On 5/6/2013 11:07 AM, Seungwon Jeon wrote:
> > Link start-up requires long time with multiphase handshakes
> > between UFS host and device. This affects driver's probe time.
> > This patch let link start-up run asynchronously. Link start-up
> > w
From: Dolev Raviv
Allow UFS device to complete its initialization and accept SCSI
commands by setting fDeviceInit flag. The device may take time
for this operation and hence the host should poll until fDeviceInit
flag is toggled to zero. This step is mandated by UFS device
specification for devic
Tested-by: Maya Erez
> There is no need to check the version to clear
> the interrupt status. And the order is changed
> prior to actual handling.
>
> Signed-off-by: Seungwon Jeon
> ---
> drivers/scsi/ufs/ufshcd.c |5 +
> 1 files changed, 1 insertions(+), 4 deletions(-)
>
> diff --git a
Tested-by: Maya Erez
> As part of device initialization sequence, sending NOP OUT UPIU and
> waiting for NOP IN UPIU response is mandatory. This confirms that the
> device UFS Transport (UTP) layer is functional and the host can configure
> the device with further commands. Add support for sendin
Tested-by: Maya Erez
>>> diff --git a/drivers/scsi/ufs/Kconfig b/drivers/scsi/ufs/Kconfig
>>> index 35faf24..82417d6 100644
>>> --- a/drivers/scsi/ufs/Kconfig
>>> +++ b/drivers/scsi/ufs/Kconfig
>>> @@ -35,6 +35,7 @@
>>> config SCSI_UFSHCD
>>> tristate "Universal Flash Storage Controller Dr
Tested-by: Maya Erez
> Implements to support the following operations.
> Currently, this patch doesn't introduce DME_ENABLE and DME_RESET
> because host controller's HCE enable contains these roles.
>
> DME_POWERON{OFF}, DME_HIBERNATE_ENTER{EXIT}, DME_ENDPOINTRESET.
>
> Signed-off-by: Seungwon Je
Tested-by: Maya Erez
> Implements to support Get and Set operation of the DME.
> And along with these operations, IDs of Attribute are added.
> It is used to configure the behavior of the UNIPRO.
>
> Signed-off-by: Seungwon Jeon
> ---
> drivers/scsi/ufs/ufshcd.c | 84 +
Hi James,
Can you please merge this patch?
Thanks,
Maya
> It makes interrupt setting more flexible especially
> for disabling. And wrong bit mask is fixed for ver 1.0.
> [17:16] is added for mask.
>
> Signed-off-by: Seungwon Jeon
> Tested-by: Maya Erez
> ---
> drivers/scsi/ufs/ufshcd.c | 84
Hi James,
Can you please merge this patch?
Thanks,
Maya
> It makes interrupt setting more flexible especially
> for disabling. And wrong bit mask is fixed for ver 1.0.
> [17:16] is added for mask.
>
> Signed-off-by: Seungwon Jeon
> Tested-by: Maya Erez
> ---
> drivers/scsi/ufs/ufshcd.c | 84
Hi James,
Can you please merge this patch on top of the query requests patch?
Thanks,
Maya
> Simplify operations with hiding mmio_base.
>
> Signed-off-by: Seungwon Jeon
> Tested-by: Maya Erez
> ---
> drivers/scsi/ufs/ufshcd.c | 105
> +++--
> drivers/sc
Hi,
The proposed fix fixes the timeout crash.
In normal operation the patch works, so after applying the fix you can add
my Tested-By.
One additional minor fix that is required:
In ufshcd_link_startup the following print-out is printed in success as well:
dev_err(hba->dev, "link startup failed %d
As part of device initialization sequence, sending NOP OUT UPIU and
waiting for NOP IN UPIU response is mandatory. This confirms that the
device UFS Transport (UTP) layer is functional and the host can configure
the device with further commands. Add support for sending NOP OUT UPIU to
check the dev
On 5/6/2013 11:07 AM, Seungwon Jeon wrote:
> Link start-up requires long time with multiphase handshakes
> between UFS host and device. This affects driver's probe time.
> This patch let link start-up run asynchronously. Link start-up
> will be executed at the end of prove separately.
> Along with
Field is now unused, so remove it.
Signed-off-by: Hannes Reinecke
---
include/scsi/scsi_host.h | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/include/scsi/scsi_host.h b/include/scsi/scsi_host.h
index af7ad0e..dcf17dc 100644
--- a/include/scsi/scsi_host.h
+++ b/include/scsi
'last_reset' is only used internally, so move it into the
internal host structure.
Signed-off-by: Hannes Reinecke
---
drivers/scsi/dc395x.c | 24 +---
1 file changed, 13 insertions(+), 11 deletions(-)
diff --git a/drivers/scsi/dc395x.c b/drivers/scsi/dc395x.c
index 694e13c..
struct Scsi_Host has a field 'resetting' which is only ever used
in dpt_i2o. And even there the usage is questionable.
So remove the field and have dpt_i2o use the standard mechanism
for delaying I/O.
Signed-off-by: Hannes Reinecke
---
drivers/scsi/dpt_i2o.c | 22 +++---
driver
The 'last_reset' value is only used internally, so move it into
the internal host structure.
Signed-off-by: Hannes Reinecke
---
drivers/scsi/tmscsim.c | 12 ++--
drivers/scsi/tmscsim.h | 1 +
2 files changed, 7 insertions(+), 6 deletions(-)
diff --git a/drivers/scsi/tmscsim.c b/drivers
The 'last_reset' field is only displayed in logging messages but
never actually used. Remove references to it.
Signed-off-by: Hannes Reinecke
---
drivers/scsi/advansys.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/scsi/advansys.c b/drivers/scsi/advansys.c
The 'Scsi_Host' structure has two fields 'resetting' and
'last_reset' which are basically unused. There are only
4 drivers referencing 'last_reset' (dpt_i2o, tmscsim, dc395, and advansys),
and only dpt_i2o makes usage of both fields.
However, dpt_i2o does _not_ require a delay after reset,
so even
59 matches
Mail list logo
