Re: [PATCHv2 00/12] Re-implement am53c974 driver

2014-11-24 Thread Christoph Hellwig
I'm tempted to merge this into drivers-for-3.19 ASAP, any argument against that? Guennadi, do you have any additional hardware to Hannes one real and one virtual adapter to test this new driver on before? -- To unsubscribe from this list: send the line unsubscribe linux-scsi in the body of a

Re: Another (ESP?) scsi blk-mq problem on sparc64

2014-11-24 Thread Christoph Hellwig
On Fri, Nov 21, 2014 at 02:56:00PM -0500, David Miller wrote: I would suggest looking into the possibility that we allocate the memory using the count of valid cpus, rather than the largest cpu number. That's a common error that runs into problems with discontiguous cpu numbering like Sparc

Re: [PATCHv2 00/12] Re-implement am53c974 driver

2014-11-24 Thread Hannes Reinecke
On 11/24/2014 09:14 AM, Christoph Hellwig wrote: I'm tempted to merge this into drivers-for-3.19 ASAP, any argument against that? Guennadi, do you have any additional hardware to Hannes one real and one virtual adapter to test this new driver on before? Certainly not from me :-) It would

[PATCH] blk-mq: handle the single queue case in blk_mq_hctx_next_cpu

2014-11-24 Thread Christoph Hellwig
Don't duplicate the code to handle the not cpu bounce case in the caller, do it inside blk_mq_hctx_next_cpu instead. Signed-off-by: Christoph Hellwig h...@lst.de --- block/blk-mq.c | 31 ++- 1 file changed, 10 insertions(+), 21 deletions(-) diff --git

[PATCH] libsas: allow async aborts

2014-11-24 Thread Christoph Hellwig
We now first try to call -eh_abort_handler from a work queue, but libsas was always failing that for no good reason. Allow async aborts. Signed-off-by: Christoph Hellwig h...@lst.de --- drivers/scsi/libsas/sas_scsi_host.c | 3 --- 1 file changed, 3 deletions(-) diff --git

[PATCH] libsas: remove task_collector mode

2014-11-24 Thread Christoph Hellwig
The task_collector mode (or latency_injector, (C) Dan Willians) is an optional I/O path in libsas that queues up scsi commands instead of directly sending it to the hardware. It generall increases latencies to in the optiomal case slightly reduce mmio traffic to the hardware. Only the obsolete

Re: [PATCH] scsi_debug: switch to table based parser

2014-11-24 Thread christoph Hellwig
Any chance to get a review for this one? -- 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

Re: [PATCH] libsas: remove task_collector mode

2014-11-24 Thread Hannes Reinecke
On 11/24/2014 09:41 AM, Christoph Hellwig wrote: The task_collector mode (or latency_injector, (C) Dan Willians) is an optional I/O path in libsas that queues up scsi commands instead of directly sending it to the hardware. It generall increases latencies to in the optiomal case slightly

Re: [PATCH] libsas: allow async aborts

2014-11-24 Thread Hannes Reinecke
On 11/24/2014 09:40 AM, Christoph Hellwig wrote: We now first try to call -eh_abort_handler from a work queue, but libsas was always failing that for no good reason. Allow async aborts. Signed-off-by: Christoph Hellwig h...@lst.de --- drivers/scsi/libsas/sas_scsi_host.c | 3 --- 1 file

Re: [PATCH] libsas: remove task_collector mode

2014-11-24 Thread Christoph Hellwig
Why did you not remove the last argument from asd_can_queue() and asd_can_dequeue(), too? With that patch it's always '1' ... Because I wanted to remove the libsas (mis-)feature and avoid to many boad driver changes. Especially for a unmaintained big driver like aic94xx. This 'while' loop

Re: [PATCH] scsi_debug: switch to table based parser

2014-11-24 Thread Hannes Reinecke
On 10/23/2014 10:33 PM, Douglas Gilbert wrote: Changing a frequently hacked, big switch parser to being table based is, of necessity, not a small patch. Testing showed up some breakages which required extra code and re-factoring. Since supporting the REPORT SUPPORTED OPERATION CODES command,

Re: [PATCH] libsas: remove task_collector mode

2014-11-24 Thread Hannes Reinecke
On 11/24/2014 09:54 AM, Christoph Hellwig wrote: Why did you not remove the last argument from asd_can_queue() and asd_can_dequeue(), too? With that patch it's always '1' ... Because I wanted to remove the libsas (mis-)feature and avoid to many boad driver changes. Especially for a

Re: BUG in scsi_lib.c due to a bad commit

2014-11-24 Thread Christoph Hellwig
On Thu, Nov 20, 2014 at 07:27:49PM +0100, Barto wrote: but the sata DVD burner is now connected on a Jmicron sata PCIe card ( with this configuration the bug doesn't occur ), I don't know if this change can modify the result of your command, here is the result of another command : $ for i

LSF/MM 2015 Call For Proposals

2014-11-24 Thread Mel Gorman
The annual Linux Storage, Filesystem and Memory Management Summit for 2015 will be held on March 9th and 10th before the Linux Foundation Vault conference at the Revere Hotel, Boston MA. For those that do not know, Vault is designed to be an event where open source storage and filesystem

ARCMSR - Notify sense data report - Lost patch?

2014-11-24 Thread Markus Stockhausen
Hi, sorting out some diffs between the official Arcea driver and the newest 3.18 patches I found a patch that might got lost: https://lkml.org/lkml/2014/2/20/956 Could you check if that is desired? Thanks in advance. Markus

[PATCH v3 37/41] virtio_scsi: v1.0 support

2014-11-24 Thread Michael S. Tsirkin
Note: for consistency, and to avoid sparse errors, convert all fields, even those no longer in use for virtio v1.0. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- include/linux/virtio_scsi.h | 32 +++- drivers/scsi/virtio_scsi.c | 51

[PATCH 2/3] wd719x: Introduce Western Digital WD7193/7197/7296 PCI SCSI card driver

2014-11-24 Thread Ondrej Zary
Introduce wd719x, a driver for Western Digital WD7193, WD7197 and WD7296 PCI SCSI controllers based on WD33C296A chip. Tested with WD7193 card. Reviewed-by: Christoph Hellwig h...@lst.de Signed-off-by: Ondrej Zary li...@rainbow-software.org --- drivers/scsi/Kconfig |8 +

[PATCH 1/3] eeprom-93cx6: Add (read-only) support for 8-bit mode

2014-11-24 Thread Ondrej Zary
Add read-only support for EEPROMs configured in 8-bit mode (ORG pin connected to GND). This will be used by wd719x driver. Signed-off-by: Ondrej Zary li...@rainbow-software.org --- drivers/misc/eeprom/eeprom_93cx6.c | 62 +++- include/linux/eeprom_93cx6.h

[PATCH 0/3] wd719x: Introduce Western Digital WD7193/7197/7296 PCI SCSI card driver

2014-11-24 Thread Ondrej Zary
Hello, this is a new driver for Western Digital WD7193, WD7197 and WD7296 PCI SCSI controllers based on WD33C296A chip. These nice cards never worked good with Linux. It's based on old and ugly wd719x driver written back in 2.0 days, then hacked to 2.2 and finally to 2.4 kernels. Most of the

[PATCH 3/3] wd719x: Add firmware documentation

2014-11-24 Thread Ondrej Zary
Add documentation and script to obtain required firmware. Signed-off-by: Ondrej Zary li...@rainbow-software.org --- Documentation/scsi/wd719x.txt | 21 + 1 file changed, 21 insertions(+) create mode 100644 Documentation/scsi/wd719x.txt diff --git

Re: [PATCH 1/3] eeprom-93cx6: Add (read-only) support for 8-bit mode

2014-11-24 Thread Hannes Reinecke
On 11/24/2014 01:11 PM, Ondrej Zary wrote: Add read-only support for EEPROMs configured in 8-bit mode (ORG pin connected to GND). This will be used by wd719x driver. Signed-off-by: Ondrej Zary li...@rainbow-software.org Reviewed-by: Hannes Reinecke h...@suse.de Cheers, Hannes -- Dr.

Re: [PATCH 2/3] wd719x: Introduce Western Digital WD7193/7197/7296 PCI SCSI card driver

2014-11-24 Thread Hannes Reinecke
On 11/24/2014 01:11 PM, Ondrej Zary wrote: Introduce wd719x, a driver for Western Digital WD7193, WD7197 and WD7296 PCI SCSI controllers based on WD33C296A chip. Tested with WD7193 card. Reviewed-by: Christoph Hellwig h...@lst.de Signed-off-by: Ondrej Zary li...@rainbow-software.org ---

Re: ARCMSR - Notify sense data report - Lost patch?

2014-11-24 Thread Christoph Hellwig
On Mon, Nov 24, 2014 at 11:47:02AM +, Markus Stockhausen wrote: Hi, sorting out some diffs between the official Arcea driver and the newest 3.18 patches I found a patch that might got lost: https://lkml.org/lkml/2014/2/20/956 Could you check if that is desired? Looks sensible to

Re: [PATCH 07/12] scsi: add 'am53c974' driver

2014-11-24 Thread Christoph Hellwig
Can someone give me a review for this patch as well? Fortuately the driver seems incredibly simple. -- 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

Re: [PATCH] aic94xx: Fixup compilation warning

2014-11-24 Thread Christoph Hellwig
Can someone review this trivial patch for me? Thanks! -- 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

AW: ARCMSR - Notify sense data report - Lost patch?

2014-11-24 Thread Markus Stockhausen
Von: linux-scsi-ow...@vger.kernel.org [linux-scsi-ow...@vger.kernel.org]quot; im Auftrag von quot;Christoph Hellwig [h...@infradead.org] Gesendet: Montag, 24. November 2014 14:23 An: Markus Stockhausen Cc: ching2...@areca.com.tw; the...@redhat.com; linux-scsi@vger.kernel.org Betreff: Re:

Re: [PATCH 07/12] scsi: add 'am53c974' driver

2014-11-24 Thread Christoph Hellwig
+/* + * AMD am53c974 driver. + * Copyright (c) 2014 Hannes Reinecke, SUSE Linux GmbH + */ Is there anything left from the old driver that should get credit here? +#define PCI_ESP_GET_PRIV(esp) ((struct pci_esp_priv *) \ +pci_get_drvdata((struct pci_dev *) \ +

Re: [PATCH 07/12] scsi: add 'am53c974' driver

2014-11-24 Thread Hannes Reinecke
On 11/24/2014 02:50 PM, Christoph Hellwig wrote: +/* + * AMD am53c974 driver. + * Copyright (c) 2014 Hannes Reinecke, SUSE Linux GmbH + */ Is there anything left from the old driver that should get credit here? Hmm. I copied the EEprom handling more-or-less verbatim from the tmscsim

[PATCH] bfa: switch to mid layer queue full tracking

2014-11-24 Thread Christoph Hellwig
As far as I can tell the queue tracking in the bfa driver is a copypaste copy of the midlayer version, just without support to configure it through sysfs. Remove the bfa copy and rely on the mid layer version. Signed-off-by: Christoph Hellwig h...@lst.de --- drivers/scsi/bfa/bfad_im.c | 66

[PATCH 05/12] esp_scsi: read status registers

2014-11-24 Thread Hannes Reinecke
A read to ESP_INTRPT will clear ESP_STATUS and ESP_SSTEP. So read all status registers in one go to avoid losing information. Reviewed-by: Paolo Bonzini pbonz...@redhat.com Acked-by: David S. Miller da...@davemloft.net Signed-off-by: Hannes Reinecke h...@suse.de --- drivers/scsi/esp_scsi.c | 8

[PATCH 1/4] scsi: remove -change_queue_type method

2014-11-24 Thread Christoph Hellwig
Since we got rid of ordered tag support in 2010 the prime use case of switching on and off ordered tags has been obsolete. The other function of enabling/dsiabling tagging entirely has only been correctly implemented by the 53c700 driver and isn't generally useful. Signed-off-by: Christoph

[PATCHv3 00/12] Re-implement am53c974 driver

2014-11-24 Thread Hannes Reinecke
Hi all, having found some issues with the current tmscsim driver I looked at the code and found is really awkward to clean up. Seeing that the am53c974 chip is actually based on the 'ESP' chip design I've re-implemented it based on the common esp_scsi routines. This new driver (cunningly named

[PATCH 08/12] am53c974: BLAST residual handling

2014-11-24 Thread Hannes Reinecke
The am53c974 has an design issue where a single byte might be left in the SCSI FIFO after a DMA transfer. As the handling code is currently untested add a WARN_ON() statement here. Reviewed-by: Paolo Bonzini pbonz...@redhat.com Acked-by: David S. Miller da...@davemloft.net Signed-off-by: Hannes

[PATCH 04/12] esp_scsi: debug event and command

2014-11-24 Thread Hannes Reinecke
Add new debug definitions for event and command logging. Reviewed-by: Paolo Bonzini pbonz...@redhat.com Acked-by: David S. Miller da...@davemloft.net Signed-off-by: Hannes Reinecke h...@suse.de --- drivers/scsi/esp_scsi.c | 15 +++ 1 file changed, 15 insertions(+) diff --git

[PATCH 3/4] scsi: remove scsi_get_tag_type

2014-11-24 Thread Christoph Hellwig
Both remaining users are better of just checking sdev-simple_tags directly. Signed-off-by: Christoph Hellwig h...@lst.de --- drivers/scsi/53c700.c | 4 ++-- drivers/scsi/ipr.c | 2 +- include/scsi/scsi_tcq.h | 13 - 3 files changed, 3 insertions(+), 16 deletions(-) diff

[PATCH 10/12] esp_scsi: let DMA driver provide a config2 value

2014-11-24 Thread Hannes Reinecke
From: Paolo Bonzini pbonz...@redhat.com On PCscsi, the FENAB configuration also enables 24-bit DMA transfer lengths (and provides the chip id in TCHI after reset). We want to be able to enable this parameter from the DMA driver. Check if the caller of scsi_esp_register provided a value for

[PATCH 09/12] esp: correctly detect am53c974

2014-11-24 Thread Hannes Reinecke
The am53c974 returns the same ID as the FAS236, but implements things slightly differently. So detect the am53c974 by checking for ESP_CONFIG4 register. Reviewed-by: Paolo Bonzini pbonz...@redhat.com Signed-off-by: Hannes Reinecke h...@suse.de --- drivers/scsi/am53c974.c | 2 ++

[PATCH 03/12] esp_scsi: convert to dev_printk

2014-11-24 Thread Hannes Reinecke
Use dev_printk functions for correct device annotations. Reviewed-by: Paolo Bonzini pbonz...@redhat.com Acked-by: David S. Miller da...@davemloft.net Signed-off-by: Hannes Reinecke h...@suse.de --- drivers/scsi/esp_scsi.c | 212 1 file changed,

[PATCH 06/12] esp_scsi: use FIFO for command submission

2014-11-24 Thread Hannes Reinecke
Using DMA for command submission has the drawback that it might generate additional DMA completion interrupts after the command has been submitted to the device. Additionally the am53c974 has a design flaw causing it to generate spurious interrupts even though DMA completion interrupts are not

[PATCH 02/12] esp_scsi: make number of tags configurable

2014-11-24 Thread Hannes Reinecke
Add a field 'num_tags' to the esp structure to allow drivers to overwrite the number of avialable tags if required. Default is ESP_DEFAULT_TAGS. Reviewed-by: Paolo Bonzini pbonz...@redhat.com Acked-by: David S. Miller da...@davemloft.net Signed-off-by: Hannes Reinecke h...@suse.de ---

[PATCH 4/4] scsi: remove scsi_set_tag_type

2014-11-24 Thread Christoph Hellwig
There is no benefit over just setting sdev-simple_tags directly. Signed-off-by: Christoph Hellwig h...@lst.de --- drivers/scsi/53c700.c | 2 +- drivers/scsi/tmscsim.c | 2 +- include/scsi/scsi_tcq.h | 15 --- 3 files changed, 2 insertions(+), 17 deletions(-) diff --git

[PATCH 2/4] scsi: never drop to untagged mode during queue ramp down

2014-11-24 Thread Christoph Hellwig
Dropping to untagged mode when ramping down a queue due to QUEUE FULL events has two problems: - nothing in the midlayer or drivers ever moves back to tagged mode during queue ramp up. - cmd_per_lun isn't the untagged queue depth for many modern drivers that can handle multiple untagged

queue_type fixups

2014-11-24 Thread Christoph Hellwig
These are a few patches that follow up my previous cleanups for the queue depth and type handling. The first patch removes the now pointless write support for the queue_type sysfs attribute. The second changes the queue full tracking code to never drop out of tagged mode, as we can't recover

[PATCH 11/12] esp: enable CONFIG2_FENAB for am53c974

2014-11-24 Thread Hannes Reinecke
CONFIG2_FENAB ('feature enable') changed definition between chip revisions, from 'Latch SCSI Phase' to 'Latch SCSI Phase, display chip ID upon reset, and enable 24 bit addresses'. So only enable it for am53c974 where we know what it's doing. Acked-by: David S. Miller da...@davemloft.net

[PATCH 07/12] scsi: add 'am53c974' driver

2014-11-24 Thread Hannes Reinecke
This patch adds a new implementation for the Tekram DC-390T / AMD AM53c974 SCSI controller, based on the generic esp_scsi infrastructure. Reviewed-by: Paolo Bonzini pbonz...@redhat.com Signed-off-by: Hannes Reinecke h...@suse.de --- drivers/scsi/Kconfig| 18 ++ drivers/scsi/Makefile | 1

[PATCH 01/12] esp_scsi: spellcheck 'driver'

2014-11-24 Thread Hannes Reinecke
Reviewed-by: Paolo Bonzini pbonz...@redhat.com Acked-by: David S. Miller da...@davemloft.net Signed-off-by: Hannes Reinecke h...@suse.de --- drivers/scsi/esp_scsi.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/esp_scsi.h b/drivers/scsi/esp_scsi.h index

Re: [PATCH] blk-mq: handle the single queue case in blk_mq_hctx_next_cpu

2014-11-24 Thread Jens Axboe
On 11/24/2014 01:27 AM, Christoph Hellwig wrote: Don't duplicate the code to handle the not cpu bounce case in the caller, do it inside blk_mq_hctx_next_cpu instead. Thanks, applied. -- Jens Axboe -- To unsubscribe from this list: send the line unsubscribe linux-scsi in the body of a message

Re: [PATCHv3 00/12] Re-implement am53c974 driver

2014-11-24 Thread Christoph Hellwig
Thanks, applied to drivers-for-3.19. -- 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

Re: AW: ARCMSR - Notify sense data report - Lost patch?

2014-11-24 Thread Tomas Henzl
On 11/24/2014 02:45 PM, Markus Stockhausen wrote: Von: linux-scsi-ow...@vger.kernel.org [linux-scsi-ow...@vger.kernel.org]quot; im Auftrag von quot;Christoph Hellwig [h...@infradead.org] Gesendet: Montag, 24. November 2014 14:23 An: Markus Stockhausen Cc: ching2...@areca.com.tw;

Re: Another (ESP?) scsi blk-mq problem on sparc64

2014-11-24 Thread Jens Axboe
On 11/24/2014 01:21 AM, Christoph Hellwig wrote: On Fri, Nov 21, 2014 at 02:56:00PM -0500, David Miller wrote: I would suggest looking into the possibility that we allocate the memory using the count of valid cpus, rather than the largest cpu number. That's a common error that runs into

Re: [PATCH] megaraid_sas: Enable shared host tag map

2014-11-24 Thread Christoph Hellwig
On Mon, Nov 24, 2014 at 04:33:55PM +0100, Hannes Reinecke wrote: The megaraid SAS driver uses a shared host tag map internally, so we should be telling the block layer about it. But it doesn't make use of request-tag yet. This would only be useful if you got rid of the internal tag allocator.

Re: [PATCH] aic94xx: Fixup compilation warning

2014-11-24 Thread James Bottomley
On Mon, 2014-11-24 at 05:40 -0800, Christoph Hellwig wrote: Can someone review this trivial patch for me? Thanks! I don't really think we need reviewers for trivial patches, do we? However, the patch is clearly bogus. What's supposed to happen is that the unset variables (size and offs) get

Re: [PATCH v3 09/11] IB/srp: Use block layer tags

2014-11-24 Thread Bart Van Assche
On 11/12/14 11:45, Christoph Hellwig wrote: FYI, I've updated this to use .use_blk_tags = 1 isntead of scsi_activate_tcq now that I've rebased the drivers-for-3.19 branch over the tcq rework. Please verify that it's workign as intented. [ just arrived home from traveling ] Hello Christoph,

Re: [PATCH] megaraid_sas: Enable shared host tag map

2014-11-24 Thread Hannes Reinecke
On 11/24/2014 04:35 PM, Christoph Hellwig wrote: On Mon, Nov 24, 2014 at 04:33:55PM +0100, Hannes Reinecke wrote: The megaraid SAS driver uses a shared host tag map internally, so we should be telling the block layer about it. But it doesn't make use of request-tag yet. This would only be

RE: [PATCH] megaraid_sas: Enable shared host tag map

2014-11-24 Thread Kashyap Desai
-Original Message- From: Christoph Hellwig [mailto:h...@lst.de] Sent: Monday, November 24, 2014 9:06 PM To: Hannes Reinecke Cc: James Bottomley; Sumit Saxena; Kashyap Desai; Christoph Hellwig; linux- s...@vger.kernel.org Subject: Re: [PATCH] megaraid_sas: Enable shared host tag map

Re: [PATCH] aic94xx: Fixup compilation warning

2014-11-24 Thread Hannes Reinecke
On 11/24/2014 04:39 PM, James Bottomley wrote: On Mon, 2014-11-24 at 05:40 -0800, Christoph Hellwig wrote: Can someone review this trivial patch for me? Thanks! I don't really think we need reviewers for trivial patches, do we? However, the patch is clearly bogus. What's supposed to

RE: [PATCH] megaraid_sas: Enable shared host tag map

2014-11-24 Thread Kashyap Desai
-Original Message- From: Hannes Reinecke [mailto:h...@suse.de] Sent: Monday, November 24, 2014 9:21 PM To: Christoph Hellwig Cc: James Bottomley; Sumit Saxena; Kashyap Desai; linux- s...@vger.kernel.org Subject: Re: [PATCH] megaraid_sas: Enable shared host tag map On 11/24/2014

Re: [PATCH] aic94xx: Fixup compilation warning

2014-11-24 Thread Tomas Henzl
On 11/24/2014 02:40 PM, Christoph Hellwig wrote: Can someone review this trivial patch for me? Thanks! The compiler complains because when asd_find_flash_de fails, the offs is not initialized. When that happens this code is invoked : if (err) { ASD_DPRINTK(couldn't find

Re: [PATCH] aic94xx: Fixup compilation warning

2014-11-24 Thread James Bottomley
On Mon, 2014-11-24 at 17:04 +0100, Tomas Henzl wrote: On 11/24/2014 02:40 PM, Christoph Hellwig wrote: Can someone review this trivial patch for me? Thanks! The compiler complains because when asd_find_flash_de fails, the offs is not initialized. When that happens this code is invoked :

Re: Another (ESP?) scsi blk-mq problem on sparc64

2014-11-24 Thread Paul E. McKenney
On Mon, Nov 24, 2014 at 08:35:46AM -0700, Jens Axboe wrote: On 11/24/2014 01:21 AM, Christoph Hellwig wrote: On Fri, Nov 21, 2014 at 02:56:00PM -0500, David Miller wrote: I would suggest looking into the possibility that we allocate the memory using the count of valid cpus, rather than the

Re: [PATCH] aic94xx: Fixup compilation warning

2014-11-24 Thread Tomas Henzl
On 11/24/2014 05:10 PM, James Bottomley wrote: On Mon, 2014-11-24 at 17:04 +0100, Tomas Henzl wrote: On 11/24/2014 02:40 PM, Christoph Hellwig wrote: Can someone review this trivial patch for me? Thanks! The compiler complains because when asd_find_flash_de fails, the offs is not

Re: [PATCH] libsas: remove task_collector mode

2014-11-24 Thread Dan Williams
On Mon, Nov 24, 2014 at 12:41 AM, Christoph Hellwig h...@lst.de wrote: The task_collector mode (or latency_injector, (C) Dan Willians) is an optional I/O path in libsas that queues up scsi commands instead of directly sending it to the hardware. It generall increases latencies to in the

[PATCH 1/6] isci: Fix a typo for the word destroy

2014-11-24 Thread SF Markus Elfring
From: Markus Elfring elfr...@users.sourceforge.net Date: Mon, 24 Nov 2014 15:35:48 +0100 A word was corrected in the description for the sci_remote_device_destruct() function. Signed-off-by: Markus Elfring elfr...@users.sourceforge.net --- drivers/scsi/isci/remote_device.c | 2 +- 1 file

[PATCH 2/6] SCSI-libfc: Fix typos for the word destroy

2014-11-24 Thread SF Markus Elfring
From: Markus Elfring elfr...@users.sourceforge.net Date: Mon, 24 Nov 2014 15:40:41 +0100 Identifiers were corrected in descriptions for the functions fc_fcp_pkt_destroy() and fc_fcp_destroy(). Signed-off-by: Markus Elfring elfr...@users.sourceforge.net --- drivers/scsi/libfc/fc_fcp.c | 4 ++--

Re: Another (ESP?) scsi blk-mq problem on sparc64

2014-11-24 Thread Jens Axboe
On 11/24/2014 09:22 AM, Paul E. McKenney wrote: On Mon, Nov 24, 2014 at 08:35:46AM -0700, Jens Axboe wrote: On 11/24/2014 01:21 AM, Christoph Hellwig wrote: On Fri, Nov 21, 2014 at 02:56:00PM -0500, David Miller wrote: I would suggest looking into the possibility that we allocate the memory

Re: Another (ESP?) scsi blk-mq problem on sparc64

2014-11-24 Thread Paul E. McKenney
On Mon, Nov 24, 2014 at 10:16:15AM -0700, Jens Axboe wrote: On 11/24/2014 09:22 AM, Paul E. McKenney wrote: On Mon, Nov 24, 2014 at 08:35:46AM -0700, Jens Axboe wrote: On 11/24/2014 01:21 AM, Christoph Hellwig wrote: On Fri, Nov 21, 2014 at 02:56:00PM -0500, David Miller wrote: I would

Re: Another (ESP?) scsi blk-mq problem on sparc64

2014-11-24 Thread Jens Axboe
On 11/24/2014 10:31 AM, Paul E. McKenney wrote: On Mon, Nov 24, 2014 at 10:16:15AM -0700, Jens Axboe wrote: On 11/24/2014 09:22 AM, Paul E. McKenney wrote: On Mon, Nov 24, 2014 at 08:35:46AM -0700, Jens Axboe wrote: On 11/24/2014 01:21 AM, Christoph Hellwig wrote: On Fri, Nov 21, 2014 at

Re: Another (ESP?) scsi blk-mq problem on sparc64

2014-11-24 Thread Paul E. McKenney
On Mon, Nov 24, 2014 at 10:33:37AM -0700, Jens Axboe wrote: On 11/24/2014 10:31 AM, Paul E. McKenney wrote: On Mon, Nov 24, 2014 at 10:16:15AM -0700, Jens Axboe wrote: On 11/24/2014 09:22 AM, Paul E. McKenney wrote: On Mon, Nov 24, 2014 at 08:35:46AM -0700, Jens Axboe wrote: On 11/24/2014

Fix Penguin Penalty 17th October2014 ( mail-archive.com )

2014-11-24 Thread cessations00624
Dear Sir Did your website get hit by Google Penguin update on October 17th 2014? What basically is Google Penguin Update? It is actually a code name for Google algorithm which aims at decreasing your websites search engine rankings that violate Google’s guidelines by using black hat SEO

AW: AW: ARCMSR - Notify sense data report - Lost patch?

2014-11-24 Thread Markus Stockhausen
Von: Tomas Henzl [the...@redhat.com] Gesendet: Montag, 24. November 2014 15:47 An: Markus Stockhausen; Christoph Hellwig Cc: ching2...@areca.com.tw; linux-scsi@vger.kernel.org Betreff: Re: AW: ARCMSR - Notify sense data report - Lost patch? On 11/24/2014 02:45 PM, Markus Stockhausen wrote:

RE: [PATCH 00/13] hpsa update

2014-11-24 Thread brace
-Original Message- From: Christoph Hellwig [mailto:h...@infradead.org] Sent: Thursday, November 20, 2014 9:55 AM To: Don Brace Cc: h...@infradead.org; webb.sca...@hp.com; james.bottom...@parallels.com; brace; linux-scsi@vger.kernel.org Subject: Re: [PATCH 00/13] hpsa update I've

RE: [PATCH] scsi: ufs: ensure clk gating work is finished before module unloading

2014-11-24 Thread Subhash Jadavani
Looks good to me. Reviewed-by: Subhash Jadavani subha...@codeaurora.org -Original Message- From: linux-scsi-ow...@vger.kernel.org [mailto:linux-scsi-ow...@vger.kernel.org] On Behalf Of Akinobu Mita Sent: Sunday, November 23, 2014 9:24 PM To: linux-scsi@vger.kernel.org Cc: Akinobu Mita;

Re: [PATCH 2/3] wd719x: Introduce Western Digital WD7193/7197/7296 PCI SCSI card driver

2014-11-24 Thread Ondrej Zary
On Monday 24 November 2014 14:15:47 Hannes Reinecke wrote: On 11/24/2014 01:11 PM, Ondrej Zary wrote: Introduce wd719x, a driver for Western Digital WD7193, WD7197 and WD7296 PCI SCSI controllers based on WD33C296A chip. Tested with WD7193 card. Reviewed-by: Christoph Hellwig

Re: [PATCH 3/3] esas2r: fix an oversight in setting return value

2014-11-24 Thread Bradley Grove
Acked-by: Bradley Grove bgr...@attotech.com Thanks, Brad On 11/16/2014 08:35 AM, Tomas Henzl wrote: The patch moves an error code assigment to a 'default' case in the previous switch statement. Signed-off-by: Tomas Henzl the...@redhat.com --- drivers/scsi/esas2r/esas2r_ioctl.c | 5 - 1

Re: [PATCH 2/3] esas2r: fix an error path in esas2r_ioctl_handler

2014-11-24 Thread Bradley Grove
Acked-by: Bradley Grove bgr...@attotech.com Thanks, Brad On 11/16/2014 08:35 AM, Tomas Henzl wrote: Is seems strange to manipulate nvram_semaphore when in this place, this patch fixes it. Signed-off-by: Tomas Henzl the...@redhat.com --- drivers/scsi/esas2r/esas2r_ioctl.c | 7 --- 1

Re: [PATCH 3/3] esas2r: fix an oversight in setting return value

2014-11-24 Thread Bradley Grove
Acked-by: Bradley Grove bgr...@attotech.com Thanks, Brad On 11/16/2014 08:35 AM, Tomas Henzl wrote: The patch moves an error code assigment to a 'default' case in the previous switch statement. Signed-off-by: Tomas Henzl the...@redhat.com --- drivers/scsi/esas2r/esas2r_ioctl.c | 5 - 1

Re: [PATCH 1/3] esas2r: fir error handling in do_fm_api

2014-11-24 Thread Bradley Grove
Looks good. Acked-by: Bradley Grove bgr...@attotech.com Thanks, Brad On 11/16/2014 08:35 AM, Tomas Henzl wrote: This patch fixes an error path and rearranges error handling. Signed-off-by: Tomas Henzl the...@redhat.com --- drivers/scsi/esas2r/esas2r_ioctl.c | 10 +- 1 file

Re: Another (ESP?) scsi blk-mq problem on sparc64

2014-11-24 Thread David Miller
From: Jens Axboe ax...@kernel.dk Date: Mon, 24 Nov 2014 10:16:15 -0700 How about this one? The num in num_possible_cpus() means a count, as in how many are there. It doesn't mean largest ID of members of set X, which is what you are asking for. Even worse, having num_online_cpus() and

Re: Another (ESP?) scsi blk-mq problem on sparc64

2014-11-24 Thread Jens Axboe
On 11/24/2014 02:56 PM, David Miller wrote: From: Jens Axboe ax...@kernel.dk Date: Mon, 24 Nov 2014 10:16:15 -0700 How about this one? The num in num_possible_cpus() means a count, as in how many are there. It doesn't mean largest ID of members of set X, which is what you are asking for.

Re: Another (ESP?) scsi blk-mq problem on sparc64

2014-11-24 Thread David Miller
From: Jens Axboe ax...@kernel.dk Date: Mon, 24 Nov 2014 15:01:55 -0700 I'll just updated blk-mq to use nr_cpu_ids and be done with it. Wow, a grep on nr_cpu_ids gets a lot of hits on people allocating just these kinds of tables :) -- To unsubscribe from this list: send the line unsubscribe

Re: Another (ESP?) scsi blk-mq problem on sparc64

2014-11-24 Thread Jens Axboe
On 11/24/2014 03:09 PM, David Miller wrote: From: Jens Axboe ax...@kernel.dk Date: Mon, 24 Nov 2014 15:01:55 -0700 I'll just updated blk-mq to use nr_cpu_ids and be done with it. Wow, a grep on nr_cpu_ids gets a lot of hits on people allocating just these kinds of tables :) Yep! It'd be

Re: Another (ESP?) scsi blk-mq problem on sparc64

2014-11-24 Thread mroos
Yes, that does look like the case. Do you have a good trick on how to allocate a map for the highest possible cpu number without first iterating the cpu map? I couldn't find something that looks like a highest_possible_cpu() helper. Honestly I think that num_posible_cpus()

[PATCH 3/3] wd719x: Add firmware documentation

2014-11-24 Thread Ondrej Zary
Add documentation and script to obtain required firmware. Signed-off-by: Ondrej Zary li...@rainbow-software.org --- Documentation/scsi/wd719x.txt | 21 + 1 file changed, 21 insertions(+) create mode 100644 Documentation/scsi/wd719x.txt diff --git

[PATCH v2 0/3] wd719x: Introduce Western Digital WD7193/7197/7296 PCI SCSI card driver

2014-11-24 Thread Ondrej Zary
Hello, this is a new driver for Western Digital WD7193, WD7197 and WD7296 PCI SCSI controllers based on WD33C296A chip. These nice cards never worked good with Linux. It's based on old and ugly wd719x driver written back in 2.0 days, then hacked to 2.2 and finally to 2.4 kernels. Most of the

[PATCH 1/3] eeprom-93cx6: Add (read-only) support for 8-bit mode

2014-11-24 Thread Ondrej Zary
Add read-only support for EEPROMs configured in 8-bit mode (ORG pin connected to GND). This will be used by wd719x driver. Signed-off-by: Ondrej Zary li...@rainbow-software.org Reviewed-by: Hannes Reinecke h...@suse.de --- drivers/misc/eeprom/eeprom_93cx6.c | 62

[PATCH 2/3] wd719x: Introduce Western Digital WD7193/7197/7296 PCI SCSI card driver

2014-11-24 Thread Ondrej Zary
Introduce wd719x, a driver for Western Digital WD7193, WD7197 and WD7296 PCI SCSI controllers based on WD33C296A chip. Tested with WD7193 card. Reviewed-by: Christoph Hellwig h...@lst.de Signed-off-by: Ondrej Zary li...@rainbow-software.org --- drivers/scsi/Kconfig |8 +

Re: Another (ESP?) scsi blk-mq problem on sparc64

2014-11-24 Thread David Miller
From: mr...@linux.ee Date: Tue, 25 Nov 2014 00:23:20 +0200 (EET) Yes, that does look like the case. Do you have a good trick on how to allocate a map for the highest possible cpu number without first iterating the cpu map? I couldn't find something that looks like a

[PATCH net-next] cxgb4/cxgb4vf/csiostor: Add T4/T5 PCI ID Table

2014-11-24 Thread Hariprasad Shenai
Add a new file t4_pci_id_tbl.h that contains T4/T5 PCI ID Table so that for all drivers that uses T4/T5 PCI functions changes can be done in one place. checkpatch.pl script reports following error, which if tried to fix ends up in compilation error. ERROR: Macros with complex values

[PATCH v2 0/5] scsi_debug: switch to table based parser

2014-11-24 Thread Douglas Gilbert
This series of patches is in response to Hannes' comments earlier today. He wanted the single large patch broken up. Changes since v1: - break single big patch into 4 smaller ones and one less large - resync with drivers-for-3.19 Patches: [PATCH v2 1/5] pinpoint invalid field in

[PATCH v2 1/5] scsi_debug: pinpoint invalid field in sense data

2014-11-24 Thread Douglas Gilbert
Use Sense Key Specific field in the sense data of an ILLEGAL REQUEST to optionally pinpoint the location of the problem field. This may be either in the cdb or the associated parameter list. --- drivers/scsi/scsi_debug.c | 138 +- 1 file changed, 88

[PATCH v2 5/5] scsi_debug: Add Report supported opcodes+tmfs; Compare and write

2014-11-24 Thread Douglas Gilbert
From: Douglas Gilbert dgilb...@interlog.com Date: Mon, 24 Nov 2014 21:27:12 -0500 Subject: [PATCH 5/5] Add Report supported opcodes+tmfs; Compare and write The Report supported operation codes command is very closely integrated into the table driven parser and very useful for testing it. Its cdb

[PATCH v2 4/5] scsi_debug: change SCSI command parser to table driven

2014-11-24 Thread Douglas Gilbert
From: Douglas Gilbert dgilb...@interlog.com Date: Mon, 24 Nov 2014 20:46:29 -0500 Subject: [PATCH 4/5] change SCSI command parser to table driven The existing 'big switch' parser in queuecommand() is changed to a table driven parser. The old and new queuecommand() were moved in the source so

[PATCH v2 2/5] scsi_debug: append inject error flags onto scsi_cmnd object

2014-11-24 Thread Douglas Gilbert
From: Douglas Gilbert dgilb...@interlog.com Date: Mon, 24 Nov 2014 20:18:02 -0500 Subject: [PATCH 2/5] append inject error flags onto scsi_cmnd object The way the existing scsi_debug command parser associated various inject error flags to a command was difficult to replicate in the table driven

[PATCH v2 3/5] scsi_debug: add Capacity Changed Unit Attention

2014-11-24 Thread Douglas Gilbert
From: Douglas Gilbert dgilb...@interlog.com Date: Mon, 24 Nov 2014 20:27:51 -0500 Subject: [PATCH 3/5] add Capacity Changed Unit Attention Via sysfs the virtual_gb scsi_debug parameter can be changed while LUs are in use. If that changes, the 'Capacity data has changed' Unit Attention is queued

Re: [PATCH v2 1/5] scsi_debug: pinpoint invalid field in sense data

2014-11-24 Thread Hannes Reinecke
On 11/25/2014 05:04 AM, Douglas Gilbert wrote: Use Sense Key Specific field in the sense data of an ILLEGAL REQUEST to optionally pinpoint the location of the problem field. This may be either in the cdb or the associated parameter list. --- drivers/scsi/scsi_debug.c | 138

Re: [PATCH v2 2/5] scsi_debug: append inject error flags onto scsi_cmnd object

2014-11-24 Thread Hannes Reinecke
On 11/25/2014 05:05 AM, Douglas Gilbert wrote: From: Douglas Gilbert dgilb...@interlog.com Date: Mon, 24 Nov 2014 20:18:02 -0500 Subject: [PATCH 2/5] append inject error flags onto scsi_cmnd object The way the existing scsi_debug command parser associated various inject error flags to a

Re: [PATCH v2 3/5] scsi_debug: add Capacity Changed Unit Attention

2014-11-24 Thread Hannes Reinecke
On 11/25/2014 05:05 AM, Douglas Gilbert wrote: From: Douglas Gilbert dgilb...@interlog.com Date: Mon, 24 Nov 2014 20:27:51 -0500 Subject: [PATCH 3/5] add Capacity Changed Unit Attention Via sysfs the virtual_gb scsi_debug parameter can be changed while LUs are in use. If that changes, the

Re: [PATCH v2 4/5] scsi_debug: change SCSI command parser to table driven

2014-11-24 Thread Hannes Reinecke
On 11/25/2014 05:05 AM, Douglas Gilbert wrote: + case 0x53: /* XDWRITEREAD(10) */ + ei_lba = 0; + lba = get_unaligned_be32(cmd + 2); + num = get_unaligned_be16(cmd + 7); + check_prot = false; + break; Why not use

Re: [PATCH v2 5/5] scsi_debug: Add Report supported opcodes+tmfs; Compare and write

2014-11-24 Thread Hannes Reinecke
On 11/25/2014 05:05 AM, Douglas Gilbert wrote: From: Douglas Gilbert dgilb...@interlog.com Date: Mon, 24 Nov 2014 21:27:12 -0500 Subject: [PATCH 5/5] Add Report supported opcodes+tmfs; Compare and write The Report supported operation codes command is very closely integrated into the table