linux-scsi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Mike Christie <michaelc@cs.wisc.edu>
To: Kai Makisara <Kai.Makisara@kolumbus.fi>
Cc: Jens Axboe <axboe@suse.de>, linux-scsi@vger.kernel.org
Subject: Re: [PATCH 5/10] convert st to use scsi_execute_async
Date: Sat, 12 Nov 2005 13:12:20 -0600	[thread overview]
Message-ID: <43763E94.40406@cs.wisc.edu> (raw)
In-Reply-To: <Pine.LNX.4.63.0511121859070.10335@kai.makisara.local>

Kai Makisara wrote:
> On Tue, 8 Nov 2005, Mike Christie wrote:
> 
> 
>>convert st to always send scatterlists and kill scsi_request
>>usage.
>>
>>This is the same as last time as it was posted, but with Kai's patches
>>merged and we now pass the bytes value to scsi_execute_async.
>>
> 
> This does not work "out of the box". The changes in st are almost the same 
> ones than before but the world around st has changed ;-) This has revealed 
> at least one bug in st that is fixed by the patch at the end of this 

ok thanks.

> message. After this patch some simple tests succeed but there seem to be 
> still problems with larger requests. I don't have time to continue 
> debugging just now but I should be able to post more results during this 
> weekend.
>

Was there a oops or lockup or any debug output you can send me? I will 
try some more large request tests with scsi_debug. You also have to 
compile your kernel with SCSI_MAX_PHYS_SEGMENTS == 255 to get larger 
requests now.

  reply	other threads:[~2005-11-12 19:12 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-11-08 10:06 [PATCH 5/10] convert st to use scsi_execute_async Mike Christie
2005-11-12 17:03 ` Kai Makisara
2005-11-12 19:12   ` Mike Christie [this message]
2005-11-12 19:54     ` Mike Christie
2005-11-13  8:04       ` Kai Makisara
2005-11-13 17:07         ` Doug Ledford
2005-11-13 18:08           ` Kai Makisara
2005-11-13 19:49             ` Doug Ledford
2005-11-13 22:12               ` Kai Makisara
2005-11-14  7:55           ` Douglas Gilbert
2005-11-14 15:15             ` Jens Axboe
2005-11-14 15:39             ` Doug Ledford

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=43763E94.40406@cs.wisc.edu \
    --to=michaelc@cs.wisc.edu \
    --cc=Kai.Makisara@kolumbus.fi \
    --cc=axboe@suse.de \
    --cc=linux-scsi@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).