[PATCH RESEND 3/3] zfcp: remove access control tables interface (keep sysfs files)

2013-08-22 Thread Steffen Maier
orever for whatever user space tools that may rely on them. Since these files always return a constant value, a new stripped down show-macro was required. Otherwise build warnings would have been introduced. Signed-off-by: Martin Peschke Cc: stable@vger.kernel.org #3.11+ Signed-off-by: Steffen Maier --

[PATCH RESEND 1/3] zfcp: fix lock imbalance by reworking request queue locking

2013-08-22 Thread Steffen Maier
-off-by: Martin Peschke Cc: stable@vger.kernel.org #2.6.35+ Cc: Andrew Morton Cc: David Howells Cc: Jens Axboe Cc: "Paul E. McKenney" Cc: Daniel Vetter Cc: linux-ker...@vger.kernel.org Signed-off-by: Steffen Maier --- drivers/s390/scsi/zfcp_qdio.c |

[PATCH RESEND 2/3] zfcp: fix schedule-inside-lock in scsi_device list loops

2013-08-22 Thread Steffen Maier
d-by: Christian Borntraeger Signed-off-by: Martin Peschke Cc: stable@vger.kernel.org #2.6.37+ Signed-off-by: Steffen Maier --- drivers/s390/scsi/zfcp_erp.c | 29 ++--- 1 file changed, 22 insertions(+), 7 deletions(-) --- a/drivers/s390/scsi/zfcp_erp.c +++ b/drivers/s39

[PATCH 5/8] zfcp: fix schedule-inside-lock in scsi_device list loops

2013-08-21 Thread Steffen Maier
d-by: Christian Borntraeger Signed-off-by: Martin Peschke Cc: stable@vger.kernel.org #2.6.37+ Signed-off-by: Steffen Maier --- drivers/s390/scsi/zfcp_erp.c | 29 ++--- 1 file changed, 22 insertions(+), 7 deletions(-) --- a/drivers/s390/scsi/zfcp_erp.c +++ b/drivers/s39

[PATCH 4/8] zfcp: fix lock imbalance by reworking request queue locking

2013-08-21 Thread Steffen Maier
-off-by: Martin Peschke Cc: stable@vger.kernel.org #2.6.35+ Cc: Andrew Morton Cc: David Howells Cc: Jens Axboe Cc: "Paul E. McKenney" Cc: Daniel Vetter Cc: linux-ker...@vger.kernel.org Signed-off-by: Steffen Maier --- drivers/s390/scsi/zfcp_qdio.c |

[PATCH 6/8] zfcp: remove access control tables interface (keep sysfs files)

2013-08-21 Thread Steffen Maier
orever for whatever user space tools that may rely on them. Since these files always return a constant value, a new stripped down show-macro was required. Otherwise build warnings would have been introduced. Signed-off-by: Martin Peschke Cc: stable@vger.kernel.org #3.11+ Signed-off-by: Steffen Maier --

Re: [PATCH] [SCSI] zfcp: status read buffers on first adapter open with link down

2013-07-29 Thread Steffen Maier
On 07/26/2013 10:35 PM, Greg KH wrote: On Fri, Jul 26, 2013 at 12:56:13PM +0200, Steffen Maier wrote: From: Steffen Maier commit 9edf7d75ee5f21663a0183d21f702682d0ef132f upstream backported for 3.0 stable Thanks, and can you also provide a version for 3.4, as it should go there as well

[PATCH] [SCSI] zfcp: status read buffers on first adapter open with link down

2013-07-26 Thread Steffen Maier
From: Steffen Maier commit 9edf7d75ee5f21663a0183d21f702682d0ef132f upstream backported for 3.0 stable Commit 64deb6efdc5504ce97b5c1c6f281fffbc150bd93 "[SCSI] zfcp: Use status_read_buf_num provided by FCP channel" started using a value returned by the channel but only evaluated the va

[PATCH RESEND 9/9] zfcp: status read buffers on first adapter open with link down

2013-04-26 Thread Steffen Maier
to inform us on a future link up event. While we are not aware of any other code path that could result in mempool resize attempts of size zero, we still also initialize the number of status read buffers to be posted to a static minimum number on adapter object allocation. Signed-off-by: Ste

[PATCH RESEND 6/9] zfcp: block queue limits with data router

2013-04-26 Thread Steffen Maier
cause fc_host and Scsi_Host do not transport FCP requests including SCSI commands using protection data. Signed-off-by: Steffen Maier Reviewed-by: Martin Peschke Cc: #3.2+ --- drivers/s390/scsi/zfcp_scsi.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) --- a/driv

[PATCH RESEND 5/9] zfcp: fix adapter (re)open recovery while link to SAN is down

2013-04-26 Thread Steffen Maier
FSF_EXCHANGE_CONFIG_DATA_INCOMPLETE is received the adapter recovery will be finished without any retries. QDIO will be up now and status changes such as LINK UP will be received now. Signed-off-by: Daniel Hansel Signed-off-by: Steffen Maier Cc: #2.6.37+ --- drivers/s390/scsi/zfcp_fsf.c |4 1 file

[PATCH 1/8] zfcp: Adapt to new FC_PORTSPEED semantics

2012-09-04 Thread Steffen Maier
From: Steffen Maier Commit a9277e7783651d4e0a849f7988340b1c1cf748a4 "[SCSI] scsi_transport_fc: Getting FC Port Speed in sync with FC-GS" changed the semantics of FC_PORTSPEED defines to FDMI port attributes of FC-HBA/SM-HBA which is different from the previous bit reversed Report

[PATCH 8/8] zfcp: only access zfcp_scsi_dev for valid scsi_device

2012-09-04 Thread Steffen Maier
local auto variable struct zfcp_scsi_dev *zfcp_sdev. Signed-off-by: Martin Peschke Signed-off-by: Steffen Maier Cc: #2.6.37+ --- drivers/s390/scsi/zfcp_fsf.c | 19 ++- 1 file changed, 14 insertions(+), 5 deletions(-) --- a/drivers/s390/scsi/zfcp_fsf.c +++ b/drivers/s390/scsi/

[PATCH 5/8] zfcp: remove invalid reference to list iterator variable

2012-09-04 Thread Steffen Maier
nged(). Signed-off-by: Julia Lawall Signed-off-by: Steffen Maier Reviewed-by: Martin Peschke Cc: #2.6.37+ --- drivers/s390/scsi/zfcp_cfdc.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- a/drivers/s390/scsi/zfcp_cfdc.c +++ b/drivers/s390/scsi/zfcp_cfdc.c @@ -2

[PATCH 6/8] zfcp: restore refcount check on port_remove

2012-09-04 Thread Steffen Maier
From: Steffen Maier Upstream commit f3450c7b917201bb49d67032e9f60d5125675d6a "[SCSI] zfcp: Replace local reference counting with common kref" accidentally dropped a reference count check before tearing down zfcp_ports that are potentially in use by zfcp_units. Even remote ports in

[PATCH 3/8] zfcp: Bounds checking for deferred error trace

2012-09-04 Thread Steffen Maier
From: Steffen Maier The pl vector has scount elements, i.e. pl[scount-1] is the last valid element. For maximum sized requests, payload->counter == scount after the last loop iteration. Therefore, do bounds checking first (with boolean shortcut) to not access the invalid element pl[scount].

[PATCH 4/8] zfcp: Do not wakeup while suspended

2012-09-04 Thread Steffen Maier
From: Steffen Maier If the mapping of FCP device bus ID and corresponding subchannel is modified while the Linux image is suspended, the resume of FCP devices can fail. During resume, zfcp gets callbacks from cio regarding the modified subchannels but they can be arbitrarily mixed with the

[PATCH 2/8] zfcp: Make trace record tags unique

2012-09-04 Thread Steffen Maier
From: Steffen Maier Duplicate fssrh_2 from a54ca0f62f953898b05549391ac2a8a4dad6482b "[SCSI] zfcp: Redesign of the debug tracing for HBA records." complicates distinction of generic status read response from local link up. Duplicate fsscth1 from 2c55b750a884b86dea8b4cc5f15e1484cc47a

Re: [PATCH 1/4] [RFC] Update SCSI git trees in MAINTAINERS

2012-08-14 Thread Steffen Maier
James, Greg, please disregard my four patches. This was an accident and I'm going to resend them correctly. I'm sorry. Steffen On Di, 2012-08-14 at 17:23 +0200, Steffen Maier wrote: plain text document attachment (704-scsi-git-tree-update.patch) From: Steffen Maier In http://ma

[PATCH 1/4] [RFC] Update SCSI git trees in MAINTAINERS

2012-08-14 Thread Steffen Maier
From: Steffen Maier In http://marc.info/?l=linux-scsi&m=133517794218707&w=2 James wrote on 2012-04-23: > Just to let you know that SCSI is moving on to a single tree model > from now on. The new tree is > > git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi.git > &g

[PATCH 3/4] zfcp: Make trace record tags unique

2012-08-14 Thread Steffen Maier
From: Steffen Maier Duplicate fssrh_2 from a54ca0f62f953898b05549391ac2a8a4dad6482b "[SCSI] zfcp: Redesign of the debug tracing for HBA records." Duplicate fsscth1 from 2c55b750a884b86dea8b4cc5f15e1484cc47a25c "[SCSI] zfcp: Redesign of the debug tracing for SAN records." Si

[PATCH 2/4] zfcp: Adapt to new FC_PORTSPEED semantics

2012-08-14 Thread Steffen Maier
From: Steffen Maier Commit a9277e7783651d4e0a849f7988340b1c1cf748a4 "[SCSI] scsi_transport_fc: Getting FC Port Speed in sync with FC-GS" changed the semantics of FC_PORTSPEED defines to FDMI port attributes of FC-HBA/SM-HBA which is different from the previous bit reversed Report

[PATCH 4/4] zfcp: Bounds checking for deferred error trace

2012-08-14 Thread Steffen Maier
From: Steffen Maier The pl vector has scount elements, i.e. pl[scount-1] is the last valid element. For maximum sized requests, payload->counter == scount after the last loop iteration. Therefore, do bounds checking first (with boolean shortcut) to not access the invalid element pl[scount].