tags 666108 - moreinfo
quit
Torbjørn Thorsen wrote:
> The kernel boots with no problems, the storage controller seems to be
> working nicely.
>
> root@xen14:~# dmesg | grep -i mega
> [3.340638] megasas: 00.00.05.38-rc1 Wed. May. 11 17:00:00 PDT 2011
> [3.340742] megasas: 0x1000:0x005b:0x1
On 12/04/2012 02:56 AM, Jeff Garzik wrote:
> On 12/03/2012 11:23 AM, Tejun Heo wrote:
>> Hello, James.
>>
>> On Mon, Dec 03, 2012 at 08:25:43AM +, James Bottomley wrote:
diff --git a/include/scsi/scsi_device.h b/include/scsi/scsi_device.h
index e65c62e..1756151 100644
--- a/inclu
On Wed, Nov 21, 2012 at 12:54 AM, Bjørn Mork wrote:
> commit 98cb7e44 ([SCSI] megaraid_sas: Sanity check user
> supplied length before passing it to dma_alloc_coherent())
> introduced a memory leak. Memory allocated for entries
> following zero length SGL entries will not be freed.
>
> Reference:
On Tue, Nov 27, 2012 at 8:46 AM, Jeff Moyer wrote:
>
> Signed-off-by: Jeff Moyer
> ---
> drivers/scsi/megaraid/megaraid_sas_base.c |5 +++--
> 1 files changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/scsi/megaraid/megaraid_sas_base.c
> b/drivers/scsi/megaraid/megaraid_sas_b
1. Handling overlapped firmware resets
This fix serialize multiple firmware resets to avoid situation where fnic
device fails to come up for link up event, when firmware resets are issued
back to back. If there are overlapped firmware resets are issued,
the firmware reset operat
Driver allows IOs with more SGEs than max SGEs supported by Palo. The current
max SGEs
supported by the fnic driver is 1024. The current register settings on Palo
supports
a max of 256 only. Palo would return any IO with more than 256 SGEs with an
error
indicating INVALID_SGLS. Fnic driver shoul
Modified fnic driver to let hardware insert the COS value. Set bit
in descriptor to 0 telling hardware to use its lif COS configurations
to insert the COS value in the frames.
Signed-off-by: Narsimhulu Musini
Signed-off-by: Hiral Patel
---
drivers/scsi/fnic/fnic_fcs.c |6 --
1 file chan
> Well, looking at QLogic and Emulex both emulate a bus reset with a
> loop over each target and invoke a target reset there.
> I somewhat fail to see the rationale behind it, other than emulating
> the bus reset behaviour on SPI.
> Given that the original target reset already failed (otherwise we
On Sat, Dec 1, 2012 at 6:40 PM, Cyril Roelandt wrote:
> This call is followed by a call to memcpy() on the same memory area, so it can
> be safely removed.
>
> Signed-off-by: Cyril Roelandt
> ---
> drivers/scsi/megaraid/megaraid_sas_fusion.c |2 --
> 1 file changed, 2 deletions(-)
>
> diff -
On 12/01/2012 06:40 PM, Cyril Roelandt wrote:
These calls are followed by calls to memcpy() on the same memory area, so they
can be safely removed.
Signed-off-by: Cyril Roelandt
---
drivers/scsi/bnx2fc/bnx2fc_hwi.c |4
1 file changed, 4 deletions(-)
diff --git a/drivers/scsi/bnx2fc
On Mon, 2012-12-03 at 14:48 -0500, Gregory Haskins wrote:
> Hello,
>
> I am experiencing a significant performance issue in write thoughput with SAS
> devices connected via an LSI 9207-8e (LSISAS2308 based) HBA. SATA devices
> plugged into the same rig appear to be ok. I first noticed this in
On 12/03/2012 02:36 PM, Chris Friesen wrote:
> On 12/03/2012 03:21 PM, Dave Jiang wrote:
>> On 12/03/2012 02:08 PM, Chris Friesen wrote:
>>> On 12/03/2012 02:52 PM, Ric Wheeler wrote:
>>>
I jumped into this thread late - can you repost detail on the specific
drive and HBA used here? In an
On 12/03/2012 04:08 PM, Chris Friesen wrote:
On 12/03/2012 02:52 PM, Ric Wheeler wrote:
I jumped into this thread late - can you repost detail on the specific
drive and HBA used here? In any case, it sounds like this is a better
topic for the linux-scsi or linux-ide list where most of the low l
On 12/03/2012 03:21 PM, Dave Jiang wrote:
On 12/03/2012 02:08 PM, Chris Friesen wrote:
On 12/03/2012 02:52 PM, Ric Wheeler wrote:
I jumped into this thread late - can you repost detail on the specific
drive and HBA used here? In any case, it sounds like this is a better
topic for the linux-scs
Quoting James Bottomley :
On Mon, 2012-11-26 at 09:55 -0600, wenxi...@linux.vnet.ibm.com wrote:
plain text document attachment (fix_mem_alloc_fail)
Add code to handle memory allocation failures at module load time.
Reported-by: Fengguang Wu
Signed-off-by: Brian King
The signoff chain on
On 12/03/2012 02:08 PM, Chris Friesen wrote:
> On 12/03/2012 02:52 PM, Ric Wheeler wrote:
>
>> I jumped into this thread late - can you repost detail on the specific
>> drive and HBA used here? In any case, it sounds like this is a better
>> topic for the linux-scsi or linux-ide list where most o
On 12/03/2012 02:52 PM, Ric Wheeler wrote:
> I jumped into this thread late - can you repost detail on the specific
> drive and HBA used here? In any case, it sounds like this is a better
> topic for the linux-scsi or linux-ide list where most of the low level
> storage people lurk :)
Okay, ex
Hello,
I am experiencing a significant performance issue in write thoughput with SAS
devices connected via an LSI 9207-8e (LSISAS2308 based) HBA. SATA devices
plugged into the same rig appear to be ok. I first noticed this in kernel
v3.0.13, but I confirmed the problem exists even in 3.7.0-r
On 12/03/2012 11:23 AM, Tejun Heo wrote:
Hello, James.
On Mon, Dec 03, 2012 at 08:25:43AM +, James Bottomley wrote:
diff --git a/include/scsi/scsi_device.h b/include/scsi/scsi_device.h
index e65c62e..1756151 100644
--- a/include/scsi/scsi_device.h
+++ b/include/scsi/scsi_device.h
@@ -160,6
The new generation IBM SAS Controllers will support MSI-X interrupts and
Distributed Completion Processing features. This patch add these support
in ipr device driver.
Signed-off-by: Wen Xiong
---
drivers/scsi/ipr.c | 716 -
drivers/scsi/ipr.h
Bump driver version.
Signed-off-by: Wen Xiong
---
drivers/scsi/ipr.h |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
Index: b/drivers/scsi/ipr.h
===
--- a/drivers/scsi/ipr.h2012-11-19 12:18:58.263935158 -0600
+
This patch reduces lock contention while implementing distributed
completion processing.
Signed-off-by: Wen Xiong
---
drivers/scsi/ipr.c | 322 +
drivers/scsi/ipr.h | 21 +--
2 files changed, 239 insertions(+), 104 deletions(-)
Index: b/d
This patch implements blk iopoll in ipr driver for performance improvement.
Signed-off-by: Wen Xiong
---
drivers/scsi/ipr.c | 218 +
drivers/scsi/ipr.h |6 +
2 files changed, 175 insertions(+), 49 deletions(-)
Index: b/drivers/scsi/ipr.c
From: Brian King
The resource path as displayed by the ipr driver is the
location string identifying a location on the SAS fabric.
This patch adds the SCSI host number such that error logs
can be more easily correlated in multiple adapter configurations.
Signed-off-by: Brian King
Signed-off-by:
Add the appropriate definitions and table entries for new adapter support.
Signed-off-by: Wen Xiong
---
drivers/scsi/ipr.c | 10 ++
drivers/scsi/ipr.h |5 +
2 files changed, 15 insertions(+)
Index: b/drivers/scsi/ipr.c
==
The new generation IBM SAS controllers will support MSI-X interrupt and
Distributed Completion Processing features. The patches in this series
add the code to support these new hardware features and also add some
patches for performance improvement such as block iopoll and reducing
lock contention
From: Brian King
Add code to handle memory allocation failures at module load time.
Reported-by: Fengguang Wu
Signed-off-by: Brian King
Signed-off-by: Wen Xiong
---
drivers/scsi/ipr.c |7 +++
1 file changed, 7 insertions(+)
Index: b/drivers/scsi/ipr.c
=
On 12/3/2012 1:15 AM, Hannes Reinecke wrote:
> Well, looking at QLogic and Emulex both emulate a bus reset with a loop
> over each target and invoke a target reset there. I somewhat fail to see
> the rationale behind it, other than emulating the bus reset behaviour on
> SPI.
It is actually
Hello, Bart.
On Mon, Dec 03, 2012 at 05:38:52PM +0100, Bart Van Assche wrote:
> It is indeed possible to invoke complete() only if the device list
> became empty with the host state equal to SHOST_CANCEL,
> SHOST_CANCEL_RECOVERY, SHOST_DEL or SHOST_DEL_RECOVERY and in
We can test this with !scsi_
On 12/03/12 17:15, Tejun Heo wrote:
On Mon, Dec 03, 2012 at 09:23:59AM +0100, Bart Van Assche wrote:
On 12/02/12 14:45, Tejun Heo wrote:
On Wed, Nov 28, 2012 at 01:51:13PM +0100, Bart Van Assche wrote:
+/* Test whether scsi_remove_host() may finish, and if so, wake it up. */
+void __scsi_check
Hello, James.
On Mon, Dec 03, 2012 at 08:25:43AM +, James Bottomley wrote:
> > diff --git a/include/scsi/scsi_device.h b/include/scsi/scsi_device.h
> > index e65c62e..1756151 100644
> > --- a/include/scsi/scsi_device.h
> > +++ b/include/scsi/scsi_device.h
> > @@ -160,6 +160,7 @@ struct scsi_de
Hello, Bart.
On Mon, Dec 03, 2012 at 09:23:59AM +0100, Bart Van Assche wrote:
> On 12/02/12 14:45, Tejun Heo wrote:
> >On Wed, Nov 28, 2012 at 01:51:13PM +0100, Bart Van Assche wrote:
> >>+/* Test whether scsi_remove_host() may finish, and if so, wake it up. */
> >>+void __scsi_check_remove_host_d
I think we should free also the cfgtable allocated a few lines
above this point.
Signed-off-by: Tomas Henzl
---
drivers/scsi/hpsa.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c
index 796482b..3cc18aa 100644
--- a/drivers/scsi/hpsa.
We do not allow old-style function definition. Always spell foo(void) if
a function does not take any parameters.
Signed-off-by: Cong Ding
---
drivers/scsi/gdth.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/scsi/gdth.c b/drivers/scsi/gdth.c
index 5d72274..9
linux/version.h is not necessary as detected by checkversion.pl script.
Signed-off-by: Sachin Kamat
---
drivers/scsi/mpt3sas/mpt3sas_base.c |1 -
drivers/scsi/mpt3sas/mpt3sas_config.c |1 -
drivers/scsi/mpt3sas/mpt3sas_ctl.c |1 -
drivers/scsi/mpt3sas/mpt3sas_s
On Mon, Dec 03, 2012 at 08:25:43AM +, James Bottomley wrote:
> On Mon, 2012-12-03 at 16:13 +0800, Aaron Lu wrote:
> > On Wed, Nov 28, 2012 at 08:56:09AM +, James Bottomley wrote:
> > > On Tue, 2012-11-27 at 17:39 -0800, Tejun Heo wrote:
> > > > Hey, Rafael.
> > > >
> > > > On Wed, Nov 28,
On Mon, 2012-12-03 at 16:13 +0800, Aaron Lu wrote:
> On Wed, Nov 28, 2012 at 08:56:09AM +, James Bottomley wrote:
> > On Tue, 2012-11-27 at 17:39 -0800, Tejun Heo wrote:
> > > Hey, Rafael.
> > >
> > > On Wed, Nov 28, 2012 at 01:51:00AM +0100, Rafael J. Wysocki wrote:
> > > > Having considered
On 12/02/12 14:45, Tejun Heo wrote:
On Wed, Nov 28, 2012 at 01:51:13PM +0100, Bart Van Assche wrote:
+/* Test whether scsi_remove_host() may finish, and if so, wake it up. */
+void __scsi_check_remove_host_done(struct Scsi_Host *shost)
+{
+ lockdep_assert_held(shost->host_lock);
+
+
On Wed, Nov 28, 2012 at 08:56:09AM +, James Bottomley wrote:
> On Tue, 2012-11-27 at 17:39 -0800, Tejun Heo wrote:
> > Hey, Rafael.
> >
> > On Wed, Nov 28, 2012 at 01:51:00AM +0100, Rafael J. Wysocki wrote:
> > > Having considered that a bit more I'm now thinking that in fact the power
> > >
39 matches
Mail list logo