On 07/05/2014 11:41 AM, Christoph Hellwig wrote:
On Fri, Jul 04, 2014 at 04:12:22PM +0200, Hannes Reinecke wrote:
So what would be set in the upper 48 bits for those drivers? We receive
the LUN over the wire to start with.
None. All these devices are physically incapable of receiving or sendi
On 07/07/2014 09:54 AM, Hannes Reinecke wrote:
On 07/05/2014 11:41 AM, Christoph Hellwig wrote:
On Fri, Jul 04, 2014 at 04:12:22PM +0200, Hannes Reinecke wrote:
So what would be set in the upper 48 bits for those drivers? We
receive
the LUN over the wire to start with.
None. All these device
On Sun, Jul 06, 2014 at 01:18:03AM +0800, loody wrote:
> hi all:
> we met a USB Hard Disk that will go to suspend if host stop
> sending scsi command over 5mins.
> To save the IO, kernel will keep the file in page cache as much as
> he can and under this circumstances, the scsi command may
On Mon, Jul 07, 2014 at 12:01:53AM +, Elliott, Robert (Server Storage)
wrote:
> > nr_sectors >>= ilog2(sdp->sector_size) - 9;
> > - rq->timeout = SD_TIMEOUT;
> > -
> > - memset(rq->cmd, 0, rq->cmd_len);
>
> Is *cmd guaranteed to be zero at this point?
Yes, scsi_setup_fs_cmnd takes ca
On Mon, Jul 07, 2014 at 09:54:51AM +0200, Hannes Reinecke wrote:
> 1) Isolate from any fallout due to the 64bit LUN changes. By introducing
> accessors we are guaranteed that the drivers will only ever see LUN numbers
> they are expecting.
I really don't see the point. The LUNs have to be passed
On 07/07/2014 11:37 AM, Christoph Hellwig wrote:
On Mon, Jul 07, 2014 at 09:54:51AM +0200, Hannes Reinecke wrote:
1) Isolate from any fallout due to the 64bit LUN changes. By introducing
accessors we are guaranteed that the drivers will only ever see LUN numbers
they are expecting.
I really do
On Sun, Jul 06, 2014 at 02:24:03PM +0200, Hans de Goede wrote:
> Hi,
>
> On 07/03/2014 08:01 AM, Christoph Hellwig wrote:
> > Hi Hans,
> >
> > please test the path below:
>
> Note, I did a similar patch a while back, but then it was decided to
> try and fix the tear-down ordering instead (which
On Mon, Jul 07, 2014 at 12:05:04PM +0200, Hannes Reinecke wrote:
> Ok, so I'll be sending a patchset for fixing up the build warnings
> (and keeping the current accesses to ->lun), and prepare a different
> patchset moving everything onto accessors so that we can use struct scsi_lun
> directly.
> A
Thanks Paolo,
I've applied the patches.
--
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
This produces a compiler warning for me that looks serious:
../drivers/scsi/ufs/ufshcd.c: In function ???ufshcd_link_startup???:
../drivers/scsi/ufs/ufshcd.c:1938:3: warning: passing argument 1 of
???ufshcd_is_device_present??? makes integer from pointer without a
cast [enabled by default]
../driv
On 07/01/2014 08:53 PM, Christoph Hellwig wrote:
> On Sun, Jun 29, 2014 at 08:28:34PM +0200, Tomas Henzl wrote:
>> On 06/27/2014 09:38 PM, Maurizio Lombardi wrote:
> diff --git a/drivers/scsi/be2iscsi/be_mgmt.c
> b/drivers/scsi/be2iscsi/be_mgmt.c
> index 07934b0..a3e5648 100644
> -
Update some missed printk() statements and fixed up some drivers.
This fixes the build failures seen by the kbuild test robot.
Signed-off-by: Hannes Reinecke
---
drivers/scsi/53c700.c | 7 ++--
drivers/scsi/NCR5380.c | 21 +-
drivers/scsi/NCR53
Hi,
On 07/07/2014 12:10 PM, Christoph Hellwig wrote:
> On Sun, Jul 06, 2014 at 02:24:03PM +0200, Hans de Goede wrote:
>> Hi,
>>
>> On 07/03/2014 08:01 AM, Christoph Hellwig wrote:
>>> Hi Hans,
>>>
>>> please test the path below:
>>
>> Note, I did a similar patch a while back, but then it was decid
On Thu, Jul 3, 2014 at 2:15 PM, Christoph Hellwig wrote:
> Suresh,
>
> do you maintain pm8001 for pmcs now? There have been lots of pm8001
> patches on the list, many of them with reviews from various people
> including Jack. I'd be really greatful if someone could collect them
> and resend th
On 07/07/2014 03:12 PM, Suresh Thiagarajan wrote:
>
> On Thu, Jul 3, 2014 at 2:15 PM, Christoph Hellwig wrote:
>> Suresh,
>>
>> do you maintain pm8001 for pmcs now? There have been lots of pm8001
>> patches on the list, many of them with reviews from various people
>> including Jack. I'd be real
On Mon, Jul 7, 2014 at 7:14 PM, Tomas Henzl wrote:
> On 07/07/2014 03:12 PM, Suresh Thiagarajan wrote:
>>
>> On Thu, Jul 3, 2014 at 2:15 PM, Christoph Hellwig wrote:
>>> Suresh,
>>>
>>> do you maintain pm8001 for pmcs now? There have been lots of pm8001
>>> patches on the list, many of them wi
On Thu, Jun 26, 2014 at 11:16:26AM +0200, Maurizio Lombardi wrote:
> In the beiscsi_conn_get_stats() function, custom_length should be set
> to 1 to take into account the "eh_abort_cnt" field at custom[0].
>
> Signed-off-by: Maurizio Lombardi
Any chance to get a review on this one?
> ---
> dri
On 07/07/2014 12:11 PM, Christoph Hellwig wrote:
On Mon, Jul 07, 2014 at 12:05:04PM +0200, Hannes Reinecke wrote:
Ok, so I'll be sending a patchset for fixing up the build warnings
(and keeping the current accesses to ->lun), and prepare a different
patchset moving everything onto accessors so t
Steve, Joe,
any chance you could review this patch and verify the sw reset case?
Thanks, Tomas
On 06/12/2014 05:29 PM, Tomas Henzl wrote:
> When a second(kdump) kernel starts and the hard reset method is used
> the driver calls pci_disable_device without previously enabling it,
> so the kernel sho
On Thu, 2014-07-03 at 15:08 +0200, Chi Pham wrote:
> Added explicit void declarations to zero-argument function headers.
> The following coccinelle script was used:
> @addvoid@
> identifier f;
> @@
>
> f(
> + void
> ) { ... }
>
> Signed-off-by: Chi Pham
> ---
> drivers/scsi/FlashPoint.c |2
hi Bryn:
2014-07-07 17:20 GMT+08:00 Bryn M. Reeves :
> On Sun, Jul 06, 2014 at 01:18:03AM +0800, loody wrote:
>> hi all:
>> we met a USB Hard Disk that will go to suspend if host stop
>> sending scsi command over 5mins.
>> To save the IO, kernel will keep the file in page cache as much as
On 2014-07-07 21:37, loody wrote:
hi Bryn:
2014-07-07 17:20 GMT+08:00 Bryn M. Reeves :
On Sun, Jul 06, 2014 at 01:18:03AM +0800, loody wrote:
hi all:
we met a USB Hard Disk that will go to suspend if host stop
sending scsi command over 5mins.
To save the IO, kernel will keep the file
From: Lars Melin
...
> sgread is not included in BusyBox but you should have "touch".
> Create a dummy file on the disk and let cron touch it every 4 minutes.
You don't need 'touch' a shell redirect eg ": >file" will do open(...,
O_CREAT|O_TRUNC).
However that still might not force an actual disc
The current implementation may mix the negative value returned
from pm8001_set_nvmd with with count. -(-ENOMEM) could be interpreted
as bytes programmed, this patch fixes it.
Signed-off-by: Tomas Henzl
---
drivers/scsi/pm8001/pm8001_ctl.c | 20 ++--
1 file changed, 10 insertions(
Instead of copying information to fw_control_context free it.
The task is forgotten thus also the reference to fw_control_context
and the completion thread takes the info from virt_ptr again.
Signed-off-by: Tomas Henzl
---
drivers/scsi/pm8001/pm8001_hwi.c | 8 ++--
1 file changed, 2 inserti
ccb->fw_control_context is copied to local fw_control_context and
the local variable is never used later
Free ccb->fw_control_context.
The task is forgotten thus also the reference to fw_control_context
and the completion thread takes the info from virt_ptr again.
Signed-off-by: Tomas Henzl
---
The driver checks the return valu, but after he tries to wait_for_completion
which might never happen.
Also the ioctlbuffer is freed at the end of the function,
so the first removal is not needed.
Signed-off-by: Tomas Henzl
---
drivers/scsi/pm8001/pm8001_ctl.c | 6 +++---
1 file changed, 3 inser
hi David:
2014-07-07 23:06 GMT+08:00 David Laight :
> From: Lars Melin
> ...
>> sgread is not included in BusyBox but you should have "touch".
>> Create a dummy file on the disk and let cron touch it every 4 minutes.
>
> You don't need 'touch' a shell redirect eg ": >file" will do open(...,
> O_C
On 06/26/2014 11:16 AM, Maurizio Lombardi wrote:
> In the beiscsi_conn_get_stats() function, custom_length should be set
> to 1 to take into account the "eh_abort_cnt" field at custom[0].
>
> Signed-off-by: Maurizio Lombardi
> ---
> drivers/scsi/be2iscsi/be_iscsi.c | 2 +-
> 1 file changed, 1 ins
I'll take a look when I get a chance, Steve's out on vacation all week. At
first glance it looks good, but like you said...gotta test it.
Joe
-Original Message-
From: Tomas Henzl [mailto:the...@redhat.com]
Sent: Monday, July 07, 2014 9:13 AM
To: 'linux-scsi@vger.kernel.org'
Cc: stephenm
On 2014-07-07 22:39, loody wrote:
hi David:
2014-07-07 23:06 GMT+08:00 David Laight :
From: Lars Melin
...
sgread is not included in BusyBox but you should have "touch".
Create a dummy file on the disk and let cron touch it every 4 minutes.
You don't need 'touch' a shell redirect eg ": >file"
On Mon, Jul 07, 2014 at 11:39:05PM +0800, loody wrote:
> hi David:
>
> 2014-07-07 23:06 GMT+08:00 David Laight :
> > From: Lars Melin
> > ...
> >> sgread is not included in BusyBox but you should have "touch".
> >> Create a dummy file on the disk and let cron touch it every 4 minutes.
> >
> > You
Oops, I just noticed that
bnx2i, be2iscsi: fix custom stats length
included this one liner change. Sorry for the stir!
--
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/
On Mon, 7 Jul 2014, loody wrote:
> hi Bryn:
>
> 2014-07-07 17:20 GMT+08:00 Bryn M. Reeves :
> > On Sun, Jul 06, 2014 at 01:18:03AM +0800, loody wrote:
> >> hi all:
> >> we met a USB Hard Disk that will go to suspend if host stop
> >> sending scsi command over 5mins.
> >> To save the IO, k
hi Alan:
>> > E.g. issue an sg_read for one sector to the device every 4m:
>> >
>> > */4 * * * * sg_read count=1 if=/dev/
>> Since my target platform arm embedded system,
>> does that mean I should include sg_read in my Busybox
>> or
>> cross-compile sg_read from sg3_utils?
>
> An alternative is
hi Bryn:
2014-07-07 23:52 GMT+08:00 Bryn M. Reeves :
> On Mon, Jul 07, 2014 at 11:39:05PM +0800, loody wrote:
>> hi David:
>>
>> 2014-07-07 23:06 GMT+08:00 David Laight :
>> > From: Lars Melin
>> > ...
>> >> sgread is not included in BusyBox but you should have "touch".
>> >> Create a dummy file
On Tue, Jul 08, 2014 at 12:15:54AM +0800, loody wrote:
> so sg_read will not hammer on the page cache like dd without "iflags=direct"
>
> thanks for your kind help,
The sg_read program (and other programs in sg3_utils) sends a command directly
to the device using an SG_IO ioctl. This bypasses all
On 07/04/2014 03:37 PM, Julia Lawall wrote:
>
>
> On Fri, 4 Jul 2014, Elliott, Robert (Server Storage) wrote:
>
>>
>>
>>> -Original Message-
>>> From: linux-scsi-ow...@vger.kernel.org [mailto:linux-scsi-
>>> ow...@vger.kernel.org] On Behalf Of Himangi Saraogi
>>> Sent: Friday, 04 July, 2
On Tue, 8 Jul 2014, loody wrote:
> > An alternative is to write a positive value, such as 24, to
> >
> > /sys/block/sdX/events_poll_msecs
> >
> > where X is replaced with the proper drive letter. (24 ms = 240 s =
> > 4 m.) I'm not sure this will work, but it's worth a try.
>
> I
This patch addresses the dma state machine lockup for APM X-Gene SoC.
Signed-off-by: Loc Ho
Signed-off-by: Suman Tripathi
---
Suman Tripathi (2):
libahci: Export some of the libahci functions.
ata: Fix the dma state machine lockup for the IDENTIFY DEVICE PIO mode
command.
drivers/ata/
This patch fixes the dma state machine lockup due to the processing
of IDENTIFY DEVICE PIO mode command. The X-Gene AHCI controller
has an errata in which it cannot clear the BSY bit after the PIO setup
FIS. The dma state machine enters CMFatalErrorUpdate state and locks
up. This patch also removes
This patch exports the function ahci_qc_issue and ahci_start_fis_rx.
The subsequent patches will make use of them.
Signed-off-by: Loc Ho
Signed-off-by: Suman Tripathi
---
drivers/ata/ahci.h| 2 ++
drivers/ata/libahci.c | 7 ---
2 files changed, 6 insertions(+), 3 deletions(-)
diff --gi
On 07/07/14 10:01 pm, "Mike Christie" wrote:
>On 07/04/2014 03:37 PM, Julia Lawall wrote:
>>
>>
>> On Fri, 4 Jul 2014, Elliott, Robert (Server Storage) wrote:
>>
>>>
>>>
-Original Message-
From: linux-scsi-ow...@vger.kernel.org [mailto:linux-scsi-
ow...@vger.kernel.org
On Thu, Apr 24, 2014 at 12:07:12PM -0600, Bjorn Helgaas wrote:
> >> These depend on f7fc32c, which appeared in v3.15-rc1. I had intended
> >> to merge these scsi updates at the same time, but I failed. Would you
> >> like to take them now that f7fc32c is upstream?
> >
> > OK, will add them. I'm
Hi, here's some additional beginning bits for the configuration and use
of this code:
Changes to targetcli-fb and rtslib-fb:
https://github.com/agrover/targetcli-fb/tree/userback
https://github.com/agrover/rtslib-fb/tree/userback
An incredibly-crappy stand-in for the user process that will han
On 06/26/2014 07:03 PM, Tyrel Datwyler wrote:
> Added big endian annotations to relevant data structure fields, and necessary
> byte swappings to support little endian builds.
Acked-by: Brian King
Thanks,
Brian
--
Brian King
Power Linux I/O
IBM Linux Technology Center
--
To unsubscribe from
46 matches
Mail list logo
