Converted dh_state to use DEVICE_ATTR_RW macro
instead of __ATTR. That forced a change to the associated
show/store function names and the name of the attribute.
Changed usage of snprintf in show function to scnprintf per
Documentation/filesystems/sysfs.txt.
Signed-off-by: Shane Seymour
---
--
Converted dh_state to use DEVICE_ATTR_RW macro
instead of __ATTR. That forced a change to the associated
show/store function names and the name of the attribute.
Changed usage of snprintf in show function to scnprintf per
Documentation/filesystems/sysfs.txt.
Signed-off-by: Shane Seymour
---
--
This patch fix the macro transport_class_to_sdev to the resonable name
and use the macro.
Signed-off-by: vonnyfly
---
drivers/scsi/sg.c |6 +++---
include/scsi/scsi_device.h |2 +-
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/scsi/sg.c b/drivers/scsi/s
On 06/11/2015 03:42 AM, rajinikanth.panduran...@pmcs.com wrote:
> From: Rajinikanth Pandurangan
>
> Description:
> If 'IsFastPath' bit is set, then response path assumes no error
> and skips error check.
>
> Signed-off-by: Rajinikanth Pandurangan
> ---
> drivers/scsi/aacraid/aa
On 06/25/2015 06:54 PM, Rajinikanth Pandurangan wrote:
> Hello Tomas,
>
> Please see my response below.
>
> Thanks,
> -Raj P.
>
> -Original Message-
> From: Tomas Henzl [mailto:the...@redhat.com]
> Sent: Thursday, June 25, 2015 8:43 AM
> To: Rajinikanth Pandurangan; jbottom...@parallels
On Wed, 2015-06-24 at 17:54 +0200, SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Wed, 24 Jun 2015 17:50:32 +0200
>
> The fc_fcp_ddp_done() function tests whether its argument is NULL and then
> returns immediately. Thus the test around the call is not needed.
>
> This issue was detecte
Make sense.
Acked-by: Chad Dupuis
On Tue, 23 Jun 2015, Nicholas Krause wrote:
This makes the function bnx_2fc_alloc_work static now due
to this function's only caller being in this particular
function's definition/declaration file of bnx2fc_hwi.c.
Signed-off-by: Nicholas Krause
---
drivers/
From: Alexei Potashnik
There are multiple reasons for disabling this:
1. It provides no functional benefit. We pretty much only get a few more
sysfs entries for each port, but all that information is already
available from /sys/kernel/debug/target/qla-session-X
2. It already only works in priva
From: Alexei Potashnik
1. Initiator A is logged in with fc_id(1)/loop_id(1)
2. Initiator A re-logs in with fc_id(2)/loop_id(2)
3. Part of old session deletion async logoout for 1/1 is queued
4. Initiator B logs in with fc_id(1)/loop_id(1), starts
passing data and creates session.
5. Async logo
From: Swapnil Nagle
Since cmds go into qla_tgt_wq and TMRs don't, it's possible that TMR
like TASK_ABORT can be queued over the cmd for which it was meant.
To avoid this race, use a per-port list to keep track of cmds that
are enqueued to qla_tgt_wq but not yet processed. When a TMR arrives,
iter
Hi James,
This series is applied on top of patch series sent on June 10
[PATCH 0/9] qla2xxx: Patches for scsi "misc" branch
(http://marc.info/?l=linux-scsi&m=143395156920505&w=2)
These set of patches addresses issue with reuse of stale command found
in a customer enviorment. Here's sequence o
From: Roland Dreier
To fix some issues talking to ESX, this patch modifies the qla2xxx driver
so that it never logs into remote ports. This has the side effect of
getting rid of the "rports" entirely, which means we never log out of
initiators and never tear down sessions when an initiator goes
From: Alexei Potashnik
Until now ack'ing of a new PLOGI has only been delayed if there
was an existing session for the same WWN. Ack was released when
the session deletion completed.
If there was another WWN session with the same fc_id/loop_id pair
(aka "conflicting session"), PLOGI was still ac
From: Alexei Potashnik
RSCN processing in qla2xxx driver can run in parallel with ELS/IO
processing. As such the decision to remove disappeared fc port's
session could be stale, because a new login sequence has occurred
since and created a brand new session.
Previous mechanism of dealing with th
From: Alexei Potashnik
The newly introduced aborted_task TFO callback has to terminate
exchange with QLogic driver, since command is being deleted and
no status will be queued to the driver at a later point.
This patch also moves the burden of releasing one cmd refcount to
the aborted_task handl
From: Alexei Potashnik
cancel any commands from initiator's s_id that are still waiting
on qla_tgt_wq when PLOGI arrives.
Cc:
Signed-off-by: Alexei Potashnik
Acked-by: Quinn Tran
Signed-off-by: Himanshu Madhani
---
drivers/scsi/qla2xxx/qla_target.c | 35 +++
From: Alexei Potashnik
- keep qla_tgt_sess object on the session list until it's freed
- modify use of sess->deleted flag to differentiate delayed
session deletion that can be cancelled from irreversible one:
QLA_SESS_DELETION_PENDING vs QLA_SESS_DELETION_IN_PROGRESS
- during IN_PROGRESS de
From: Alexei Potashnik
If a new initiator (different WWN) shows up on the same fcport, old
initiator's session is scheduled for deletion. But there is a small
window between it being marked with QLA_SESS_DELETION_IN_PROGRESS
and qlt_unret_sess getting called when new session's commands will
keep
Cc:
Signed-off-by: Himanshu Madhani
Signed-off-by: Giridhar Malavali
---
drivers/scsi/qla2xxx/qla_attr.c | 36 +++
drivers/scsi/qla2xxx/qla_dbg.c|5 +-
drivers/scsi/qla2xxx/qla_def.h| 19 -
drivers/scsi/qla2xxx/qla_gbl.h|2 +
drivers/scsi/qla2xxx/qla_inline.h |
Hello Tomas,
Please see my response below.
Thanks,
-Raj P.
-Original Message-
From: Tomas Henzl [mailto:the...@redhat.com]
Sent: Thursday, June 25, 2015 8:43 AM
To: Rajinikanth Pandurangan; jbottom...@parallels.com;
linux-scsi@vger.kernel.org
Cc: aacr...@pmc-sierra.com; Harry Yang; Mah
Under the 'default' case in scmnd->sc_data_direction we have 3 options:
- DMA_NONE which we handle correctly.
- DMA_BIDIRECTIONAL which is never supposed to be set by SCSI stack.
- Garbage value.
Do WARN() and return -EINVAL in the last two cases. virtio_scsi does
BUG_ON() here but it looks like a
On 06/11/2015 03:42 AM, rajinikanth.panduran...@pmcs.com wrote:
> From: Rajinikanth Pandurangan
>
> Description:
> If writeq() not supported, then do atomic two 32bit write
>
> Signed-off-by: Rajinikanth Pandurangan
Reviewed-by: Tomas Henzl
Tomas
--
To unsubscribe from this list: sen
On 06/11/2015 03:42 AM, rajinikanth.panduran...@pmcs.com wrote:
> From: Rajinikanth Pandurangan
>
> Description:
> Enable MSI interrupt mode for series-6 controller.
What seem problematic to me is that you still keep the module
option 'aac_msi' and a poor user might try
"IRQ handling. 0=PIC
Updated version per comments from Jiri Slaby. Thanks!
8<
Fixes another signed / unsigned array indexing bug in the ipr driver.
Currently, when hrrq_index wraps, it becomes a negative number. We
do the modulo, but still have a negative number, so we end up indexing
backwards in the array. Given w
On 06/25/2015 09:36 AM, Jiri Slaby wrote:
> On 06/25/2015, 03:44 PM, Brian King wrote:
>> Fixes another signed / unsigned array indexing bug in the ipr driver.
>
> Could you be more specific? Specifically, I fail to see why you do +1
> twice now.
Regarding the extra +1, you are correct. Its not n
On 06/25/2015 09:36 AM, Jiri Slaby wrote:
> On 06/25/2015, 03:44 PM, Brian King wrote:
>> Fixes another signed / unsigned array indexing bug in the ipr driver.
>
> Could you be more specific? Specifically, I fail to see why you do +1
> twice now.
Sure. With the code that is currently upstream, wh
On 06/11/2015 03:42 AM, rajinikanth.panduran...@pmcs.com wrote:
> From: Rajinikanth Pandurangan
>
> Description:
> * .suspend() and .resume() routines implemented in the driver
> * aac_release_resources() initiates firmware shutdown
> * aac_acquire_resources re-initializes the h
On 06/11/2015 03:42 AM, rajinikanth.panduran...@pmcs.com wrote:
> From: Rajinikanth Pandurangan
>
> Description:
> Driver sends the right size of the response buffer.
>
> Signed-off-by: Rajinikanth Pandurangan
Reviewed-by: Tomas Henzl
Tomas
--
To unsubscribe from this list: send the lin
On 06/25/2015, 03:44 PM, Brian King wrote:
> Fixes another signed / unsigned array indexing bug in the ipr driver.
Could you be more specific? Specifically, I fail to see why you do +1
twice now.
> --- linux/drivers/scsi/ipr.c~ipr_hrrq_index_fix 2015-06-23
> 11:43:18.151741523 -0500
> +++
James,
Here is one more fix for a rather nasty bug where the ipr driver can start
accessing memory it doesn't own. I'd like to add to the queue
of ipr patches. There are now two patches on top of the previously submitted
series of four. If you want me to resend everything as a complete
series, pl
On Tue, Jun 23, 2015 at 2:06 AM, Martin K. Petersen
wrote:
>
> > "Sreekanth" == Sreekanth Reddy writes:
>
> Sreekanth> I agree with you that these changes belong in pci.ids. But
> Sreekanth> Customers are asking us to print the HBA branding name while
> Sreekanth> loading the driver on their
This patch contains the fix for kernel oops on issuing setpci reset
along with sysfs, cli access
spinlock initialization modified to DEFINE_SPINLOCK as per the
comments for previous version of the patch
>From edf1c00ddcc9b045fd1527a23631f3d4b1eea599 Mon Sep 17 00:00:00 2001
From: Nagarajkumar Na
This patch fixes duplicate cache names.
Signed-off-by: Narsimhulu Musini
---
drivers/scsi/snic/snic_main.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/scsi/snic/snic_main.c b/drivers/scsi/snic/snic_main.c
index b2b87ce..47ed593 100644
--- a/drivers/scsi/snic/snic_
On 25/06/15 3:22 pm, "Maninder Singh" wrote:
>Currently SNIC_HOST_ERR is called in both cases
>whether snic_disc_start returns zero or non-zero.
>
>From the code it looks like it has to print only if
>snic_disc_start return non-zero value, So calling
>SNIC_HOST_ERR only for non-zero return from
pci_dev_put checks for NULL Pointer itself.
Signed-off-by: Maninder Singh
---
drivers/scsi/eata_pio.c |3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/scsi/eata_pio.c b/drivers/scsi/eata_pio.c
index ca8003f..481ecac 100644
--- a/drivers/scsi/eata_pio.c
+++ b/drive
On Wed, Jun 24, 2015 at 05:33:50PM +0200, SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Wed, 24 Jun 2015 17:15:17 +0200
>
> The wd33c93_setup() function performs also input parameter validation.
> Thus the test around the call is not needed.
>
Guys, you should be aware that Markus aut
Currently SNIC_HOST_ERR is called in both cases
whether snic_disc_start returns zero or non-zero.
>From the code it looks like it has to print only if
snic_disc_start return non-zero value, So calling
SNIC_HOST_ERR only for non-zero return from snic_disc_start.
Signed-off-by: Maninder Singh
Revi
37 matches
Mail list logo
