Re: [PATCH 10/10] scsi: Do not display buffer pointers in scsi_log_send()

2014-11-17 Thread Hannes Reinecke
On 11/14/2014 11:53 PM, Elliott, Robert (Server Storage) wrote: > > >> -Original Message- >> From: Hannes Reinecke [mailto:h...@suse.de] >> Sent: Thursday, 06 November, 2014 2:31 AM >> To: James Bottomley >> Cc: Christoph Hellwig; Ewan Milne; Elliott, Robert (Server Storage); >> linux-scs

[PATCH RESEND 1/7] megaraid_sas : Driver version upgrade and remove some meta data of driver

2014-11-17 Thread Sumit.Saxena
Update driver version and remove some meta data(release date and extended version) about megaraid_sas driver. Signed-off-by: Sumit Saxena --- drivers/scsi/megaraid/megaraid_sas.h |4 +--- drivers/scsi/megaraid/megaraid_sas_base.c | 24 +--- 2 files changed, 2 inse

[PATCH RESEND 3/7] megaraid_sas : Do not process IOCTLs and SCSI commands during driver removal

2014-11-17 Thread Sumit.Saxena
Do not process any SCSI and IOCTL command further(return them with appropriate return values to callers), while driver removal is in progress/PCI shutdown is invoked. Signed-off-by: Sumit Saxena Signed-off-by: Kashyap Desai --- drivers/scsi/megaraid/megaraid_sas_base.c | 19 +++-

[PATCH RESEND 2/7] megaraid_sas : Update MAINTAINERS and copyright information for megaraid drivers

2014-11-17 Thread Sumit.Saxena
Update MAINTAINERS list and copyright information for megaraid_sas driver. Signed-off-by: Sumit Saxena --- MAINTAINERS |9 ++--- drivers/scsi/megaraid/megaraid_sas.h| 16 +--- drivers/scsi/megaraid/megaraid_sas_base.c | 21 +++

[PATCH RESEND 5/7] megaraid_sas : Make HBA operational after LD_MAP_SYNC DCMD in OCR path

2014-11-17 Thread Sumit.Saxena
In OCR(Online Controller Reset) path, driver sets adapter state to MEGASAS_HBA_OPERATIONAL before getting new RAID map. There will be a small window where IO will come from OS with old RAID map. This patch will update adapter state to MEGASAS_HBA_OPERATIONAL, only after driver has new RAID map to

[PATCH RESEND 0/7] megaraid_sas : Description Patch

2014-11-17 Thread Sumit.Saxena
megaraid_sas driver changes. This resent patch series has some additional changes in few patches on top of last patch series. [PATCH RESNED 1/7] additionally rmeoves some meta data of megaraid_sas driver. [PATCH RESEND 2/7] has additional copyright changes as well. Please consider this patch set f

[PATCH RESEND 4/7] megaraid_sas : Online Firmware upgrade support for Extended VD feature

2014-11-17 Thread Sumit.Saxena
In OCR(Online Controller Reset) path, driver sets adapter state to MEGASAS_HBA_OPERATIONAL before getting new RAID map. There will be a small window where IO will come from OS with old RAID map. This patch will update adapter state to MEGASAS_HBA_OPERATIONAL, only after driver has new RAID map to

[PATCH RESEND 6/7] megaraid_sas : Corrected return of wait_event from abort frame path

2014-11-17 Thread Sumit.Saxena
Corrected wait_event() call which was waiting for wrong completion status(0xFF). Cc: Signed-off-by: Sumit Saxena Signed-off-by: Kashyap Desai --- drivers/scsi/megaraid/megaraid_sas_base.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/scsi/megaraid/megaraid_

[PATCH RESEND 7/7] megaraid_sas : Endinaness related bug fixes

2014-11-17 Thread Sumit.Saxena
This patch addresses few endianness related bug fixes. Cc: Signed-off-by: Sumit Saxena Signed-off-by: Kashyap Desai --- drivers/scsi/megaraid/megaraid_sas_fp.c | 17 + drivers/scsi/megaraid/megaraid_sas_fusion.c | 13 +++-- 2 files changed, 16 insertions(+), 14

RE: [PATCH] drivers: scsi: megaraid: Add "megaraid_sas_internal.h" for internal shared functions using

2014-11-17 Thread Sumit Saxena
>-Original Message- >From: Chen Gang [mailto:gang.chen.5...@gmail.com] >Sent: Sunday, November 09, 2014 3:47 PM >To: megaraidli...@lsi.com; jbottom...@parallels.com >Cc: linux-scsi@vger.kernel.org; linux-ker...@vger.kernel.org >Subject: [PATCH] drivers: scsi: megaraid: Add "megaraid_sas_int

Re: [PATCH V1] virtio_scsi: support multi hw queue of blk-mq

2014-11-17 Thread Paolo Bonzini
On 15/11/2014 04:47, Ming Lei wrote: > Since virtio_scsi has supported multi virtqueue already, > it is natural to map the virtque to hw-queue of blk-mq. > > Cc: Paolo Bonzini > Signed-off-by: Ming Lei > --- > V1: > - support non-mq too > > drivers/scsi/virtio_scsi.c | 17 +++

RE: [PATCH] drivers: scsi: megaraid: Add "megaraid_sas_internal.h" for internal shared functions using

2014-11-17 Thread Chen Gang
OK, I will send patch v2 for it. I sent this patch based on Linux next tree, please tell me the tree location (git address) which I shall base on for sending patch v2. Thanks. Send from Lenovo A788t. Sumit Saxena wrote: >>-Original Message- >>From: Chen Gang [mailto:gang.chen.5...@gm

RE: [PATCH] drivers: scsi: megaraid: Add "megaraid_sas_internal.h" for internal shared functions using

2014-11-17 Thread Sumit Saxena
>-Original Message- >From: Chen Gang [mailto:gang.chen.5...@gmail.com] >Sent: Monday, November 17, 2014 5:17 PM >To: Sumit Saxena >Cc: megaraidli...@lsi.com; linux-scsi@vger.kernel.org; linux- >ker...@vger.kernel.org >Subject: RE: [PATCH] drivers: scsi: megaraid: Add "megaraid_sas_internal.

[PATCH 04/10] scsi: log request tag for scmd_printk()

2014-11-17 Thread Hannes Reinecke
The request tag provides a concise identification of a SCSI command, so we should be printing that out for scmd_printk(). Suggested-by: Christoph Hellwig Tested-by: Robert Elliott Reviewed-by: Robert Elliott Reviewed-by: Christoph Hellwig Signed-off-by: Hannes Reinecke --- drivers/scsi/scsi_

[PATCHv3 00/10] scsi logging update: the real thing

2014-11-17 Thread Hannes Reinecke
Hi all, this is the second part of my scsi logging update, covering CDB and sense code printing. With this patchset CDBs and sense code bytes will be formatted on a single line (where possible), and prefixed with the device and command tag. To ensure CDBs and sense code bytes are not broken up du

[PATCH 01/10] scsi: Rename SERVICE_ACTION_IN to SERVICE_ACTION_IN_16

2014-11-17 Thread Hannes Reinecke
SPC-3 defines SERVICE ACTION IN(12) and SERVICE ACTION IN(16). So rename SERVICE_ACTION_IN to SERVICE_ACTION_IN_16 to be consistent with SPC and to allow for better distinction. Tested-by: Robert Elliott Reviewed-by: Robert Elliott Reviewed-by: Christoph Hellwig Signed-off-by: Hannes Reinecke

[PATCH 08/10] scsi: use per-cpu buffer for formatting scsi_print_result()

2014-11-17 Thread Hannes Reinecke
Convert scsi_print_result() to use the per-cpu buffer for decoding the command result and disposition. Tested-by: Robert Elliott Reviewed-by: Robert Elliott Signed-off-by: Hannes Reinecke --- drivers/scsi/constants.c| 22 - drivers/scsi/scsi.c | 2 +- drivers/scsi/

[PATCH 07/10] scsi: use per-cpu buffer for formatting sense

2014-11-17 Thread Hannes Reinecke
Convert sense buffer logging to use the per-cpu buffer to avoid line breakup. Tested-by: Robert Elliott Reviewed-by: Robert Elliott Signed-off-by: Hannes Reinecke --- drivers/scsi/constants.c| 107 drivers/scsi/scsi_logging.c | 166 +

[PATCH 10/10] scsi: Do not display buffer pointers in scsi_log_send()

2014-11-17 Thread Hannes Reinecke
scsi_log_send() would display buffer pointer for higher logging levels. This is not only of questionable value but also exposes kernel pointer to userspace, which is discouraged in some setups. So drop this message altogether. Tested-by: Robert Elliott Reviewed-by: Robert Elliott Signed-off-by:

[PATCH 03/10] scsi: Implement per-cpu logging buffer

2014-11-17 Thread Hannes Reinecke
Implement a per-cpu buffer for formatting messages to avoid line break up under high load. This patch implements scmd_printk() and sdev_prefix_printk() using the per-cpu buffer and makes sdev_printk() a wrapper for sdev_prefix_printk(). Tested-by: Robert Elliott Reviewed-by: Robert Elliott Signe

[PATCH 06/10] libata: use __scsi_format_command()

2014-11-17 Thread Hannes Reinecke
libata already uses an internal buffer, so we should be using __scsi_format_command() here. Tested-by: Robert Elliott Reviewed-by: Robert Elliott Reviewed-by: Christoph Hellwig Acked-by: Tejun Heo Signed-off-by: Hannes Reinecke --- drivers/ata/libata-eh.c | 17 +++-- 1 file chang

[PATCH 05/10] scsi: use external buffer for command logging

2014-11-17 Thread Hannes Reinecke
Use an external buffer for __scsi_print_command() and move command logging over to use the per-cpu logging buffer. With that we can guarantee the command always will always be formatted in one line. So we can even print out a variable length command correctly across several lines. Finally rename __

[PATCH 02/10] scsi: Add SPC-3 command definitions

2014-11-17 Thread Hannes Reinecke
SPC-3 defines SERVICE ACTION IN(12), SERVICE_ACTION OUT(12), SERVICE ACTION OUT(16), and SERVICE ACTION BIDIRECTIONAL. And READ MEDIA SERIAL NUMBER has long since been deprecated. So update callers to refer to the new cdb name. Tested-by: Robert Elliott Reviewed-by: Robert Elliott Reviewed-by: C

[PATCH 09/10] scsi: Conditionally compile in constants.c

2014-11-17 Thread Hannes Reinecke
Instead of having constants.c littered with ifdef statements we should be moving dummy functions into the header and condintionally compile in constants.c if selected. And update the Kconfig description to reflect the actual size difference. Suggested-by: Christoph Hellwig Tested-by: Robert Ellio

Re: [PATCH 1/3] zfcp: remove access control tables interface (port leftovers)

2014-11-17 Thread Hannes Reinecke
On 11/13/2014 02:59 PM, Steffen Maier wrote: > From: Martin Peschke > > This patch removes some leftovers for commit > 663e0890e31cb85f0cca5ac1faaee0d2d52880b5 > "[SCSI] zfcp: remove access control tables interface". > > The "access denied" case for ports is gone, as well. > The corresponding fl

Re: [PATCH 2/3] zfcp: bring back unit sysfs attributes for automatic LUN scan

2014-11-17 Thread Hannes Reinecke
On 11/13/2014 02:59 PM, Steffen Maier wrote: > Through sysfs attributes, zfcp unit objects > provide a trigger for manual LUN recovery > and export information for problem determination. > > With commit > f8210e34887e1feb977a9b6b8caa086855af40c9 > "[SCSI] zfcp: Allow midlayer to scan for LUNs when

Re: [PATCH 1/3] zfcp: remove access control tables interface (port leftovers)

2014-11-17 Thread Hannes Reinecke
On 11/17/2014 02:29 PM, Hannes Reinecke wrote: > On 11/13/2014 02:59 PM, Steffen Maier wrote: >> From: Martin Peschke >> >> This patch removes some leftovers for commit >> 663e0890e31cb85f0cca5ac1faaee0d2d52880b5 >> "[SCSI] zfcp: remove access control tables interface". >> >> The "access denied" c

Re: [PATCH 3/3] zfcp: auto port scan resiliency

2014-11-17 Thread Hannes Reinecke
On 11/13/2014 02:59 PM, Steffen Maier wrote: > From: Martin Peschke > > This patch improves the Fibre Channel port scan behaviour of the zfcp lldd. > Without it the zfcp device driver may churn up the storage area network by > excessive scanning and scan bursts, particularly in big virtual server

[PATCH V4 3/5] scsi: Encapsulate scsi_do_report_luns

2014-11-17 Thread Rob Evers
Report luns functionality is encapsulated as it will be re-used in the next patch in this series if a large number of LUs are present. Signed-off-by: Rob Evers --- drivers/scsi/scsi_scan.c | 114 ++- 1 file changed, 62 insertions(+), 52 deletions(-) d

[PATCH V4 5/5] scsi: Use set/get_unaligned_be32 in report_luns

2014-11-17 Thread Rob Evers
Signed-off-by: Rob Evers --- drivers/scsi/scsi_scan.c | 21 ++--- 1 file changed, 6 insertions(+), 15 deletions(-) diff --git a/drivers/scsi/scsi_scan.c b/drivers/scsi/scsi_scan.c index 0d9108e..fa6ac51 100644 --- a/drivers/scsi/scsi_scan.c +++ b/drivers/scsi/scsi_scan.c @@ -34,6

[PATCH V4 4/5] scsi: Change default value of max_report_luns to 16k-1

2014-11-17 Thread Rob Evers
First, allocate a buffer large enough for report luns command to report on 511 LUs and issue a report luns command. This buffer size matches the previous report lun behavior based on the original default max_report_luns value. If more that 511 LUs are reported in the first report luns, allocate a

[PATCH V4 0/5] scsi: Configure number of LUs reported by 'report-luns'

2014-11-17 Thread Rob Evers
This patch set retrieves the number of LUs available on a target using the report-luns command. The initial size of the report-luns command is 512 entries, as the previous default initial number was. If more LUs than 511 are present on a target, the report-luns is re-issued with the size indicated

[PATCH V4 2/5] scsi: Fix scsi_report_lun_scan return when kmalloc fails

2014-11-17 Thread Rob Evers
Signed-off-by: Rob Evers --- drivers/scsi/scsi_scan.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/scsi/scsi_scan.c b/drivers/scsi/scsi_scan.c index e460b35..69291bf3 100644 --- a/drivers/scsi/scsi_scan.c +++ b/drivers/scsi/scsi_scan.c @@ -1412,6 +1412,7 @@ static int scsi_report_l

[PATCH V4 1/5] scsi: Fix scsi_report_lun_scan kmalloc flag

2014-11-17 Thread Rob Evers
=== Signed-off-by: Rob Evers --- drivers/scsi/scsi_scan.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/scsi_scan.c b/drivers/scsi/scsi_scan.c index ba3f1e8..e460b35 100644 --- a/drivers/scsi/scsi_

Re: [PATCH 3/3] block: Introduce blkdev_issue_zeroout_discard() function

2014-11-17 Thread Darrick J. Wong
On Fri, Nov 14, 2014 at 03:22:05PM -0500, Martin K. Petersen wrote: > > "Martin" == Martin K Petersen writes: > > Martin> What would you prefer as the default for the ext4 use case? To > Martin> allocate or to discard? > > I didn't get a preference for whether sb_issue_zeroout() should disca

[PATCH] block: create ioctl to discard-or-zeroout a range of blocks

2014-11-17 Thread Darrick J. Wong
Create a new ioctl to expose the block layer's newfound ability to issue either a zeroing discard, a WRITE SAME with a zero page, or a regular write with the zero page. This BLKZEROOUT2 ioctl takes {start, length, flags} as parameters. So far, the only flag available is to enable the zeroing disc

Re: [PATCH 03/10] scsi: Implement per-cpu logging buffer

2014-11-17 Thread Douglas Gilbert
On 14-11-17 08:25 AM, Hannes Reinecke wrote: Implement a per-cpu buffer for formatting messages to avoid line break up under high load. This patch implements scmd_printk() and sdev_prefix_printk() using the per-cpu buffer and makes sdev_printk() a wrapper for sdev_prefix_printk(). Tested-by: Rob

[PATCH] SCSI Improved spelling and documentation.

2014-11-17 Thread Terence Eden
From: Terence Eden I found the comments to be slightly confusing. Hopefully I've clarified some points. Signed-off-by: Terence Eden --- drivers/scsi/isci/port_config.c | 158 +--- 1 file changed, 83 insertions(+), 75 deletions(-) diff --git a/drivers/scsi/

[PATCH 1/1] IOMMU Fault occurs when IO and abort IO is out of order

2014-11-17 Thread Anil Chintalapati
When I/O is aborted by mid-layer, fnic FW will complete the I/O before completing the abort task. In some cases abort request is completed before the I/O, which could lead to inconsistent driver and firmware states. In this case firmware reset would clear the inconsistent state Signed-off-by: A

Re: [PATCH 3/3] target: Change some fabric functions to return bool

2014-11-17 Thread Andy Grover
On 11/13/2014 11:06 PM, Christoph Hellwig wrote: On Thu, Nov 13, 2014 at 12:50:58PM -0800, Andy Grover wrote: The tpg_check_* functions in the fabric_ops struct return 1 or 0, so we can make their return type bool. Change fabrics to match. All these true/false stub sounds like some of them met

COMPANY REPRESENTATIVE NEEDED

2014-11-17 Thread ST CHANG
China Steel Corporation (CSAC), is in urgent need of a reputable company,/firm or individual to serve as our financial coordinator in Canada, America, Europe, Uk. It's a part time job and pays well. If you are interested in working with us please reply: stchang...@yahoo.com.hk Thank you very m