Re: [PATCH 5/10] convert st to use scsi_execute_async

[Mike Christie <michaelc@cs.wisc.edu>]

Mike Christie wrote:
> 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.


Oh yeah I put the patches Christoph asked me to make, and changes that 
Jens and Pat Mansfield asked for, and a patch to increase 
SCSI_MAX_PHYS_SEGMENTS here 
http://www.cs.wisc.edu/~michaelc/block/use-sg/v11/

They were made against a scsi-misc tree I got today. But I was not 
actually able to test against that tree becuase the current git trees 
(scsi-misc, linux-2.6.git, etc) do not boot for me.