Skip to content
  • Kai Makisara's avatar
    [SCSI] Fix st oops with new scsi_execute infrastructure · 787926b1
    Kai Makisara authored
    
    
    Patch from Kai minus last sg_segs clearing which was merged already.
    
    > > 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.
    >
    It was an oops in sgl_unmap_user_pages(). The reason is this:
    
    		/* XXX: just for debug. Remove when PageReserved is removed */
    		BUG_ON(PageReserved(page));
    
    I was using /dev/zero as input and it triggers this. When I used a file as
    input, this did not trigger. Should this BUG_ON be removed?
    
    In the same log I noticed that there was another ->sg_segs inconsistency.
    Also, the field ->last_SRpnt was not reset when scsi_execute_async()
    failed. This caused the error message "Async command already active"
    later and prevented proper close.
    
    While doing the changes, I noticed that the current code (since
    2.6.0-test4) does not set the pages dirty when reading with direct i/o.
    
    All of these st problems (including the one I sent earlier) are fixed in
    the patch at the end of this message. These fixes should probably be
    included already in 2.6.15.
    
    After these fixes, the tape seems to operate as expected. Without other
    changes, the largest block size with sym53c896 SCSI adapter is 384 kB. The
    maximum number of sg segments is set to 96 and clustering is disabled in
    the driver. 96 x 4 kB = 384 kB. OK.
    
    I enabled clustering and set max_sectors to 10000 in the SCSI HBA driver.
    Now the block size limit is 5000 kB as expected.
    
    Signed-off-by: default avatarJames Bottomley <James.Bottomley@SteelEye.com>
    787926b1