[sane-devel] Problem with empty sense buffer in sanei_scsi (linux+hpusbscsi+vuescan)?

Thu, 4 Oct 2001 23:36:21 +0100

--------------Boundary-00=_LGDP714O5XTG771YA91A
Content-Type: text/plain;
  charset="us-ascii"
Content-Transfer-Encoding: 8bit

On Thursday 04 October 2001 21:01, abel deuring wrote:
> Jose Paulo Moitinho de Almeida wrote:
> > Hello
> >
>.....
>
> Jose, it seems that you uncovered a serious problem in sanei_scsi.c --
> but I'm afraid that a proper fix isn't that easy:
>
> 1. sanei_scsi.c passes the sense buffer to a callback function
> (parameter sense_handler in sanei_scsi_open[_extended]) So a fix would
> require patches to all backends that implement a sense handler.
>
> 2. As mentioned above, older versions of the Linux SG driver do not
> return the length of the sense buffer. So, even if you would start to
> patch sanei_scsi.c, the sense handlers would need to be aware that no
> length information is available.
>
> 3. The patch needs to be applied to the implementations of
> sanei_scsi_cmd resp. sanei_scsi_req_enter/sanei_scsi_req_wait for all
> the operating systems supported by Sane.
>
> An easier way might perhaps be to return some "obviously invalid data"
> like a string of 0 or 0xff in the case that the SG driver or its
> counterparts for other OS do not return a valid sense buffer. (BTW, does
> Byte 0 of the sense buffer contain the value 0xf0 or 0x70? If not, this
> could be used to determine, if a valid sense buffer is available.)
>

I now have the log, using the original sg driver, which is attached and 
states:

a) "sanei_scsi_open: using new SG header structure";

b) "sense buffer: 68 22 42 08 ff ff ff ff 00 00 00 00 00 00 00 00"

After reading Abel's message, rereading the code and looking at the log I 
think it is safe to say that the problem is here:

     if (   ((req->sgdata.sg3.hdr.info & SG_INFO_CHECK) != 0)
             || (req->sgdata.sg3.hdr.sb_len_wr > 0 &&
              ((req->sgdata.sg3.sense_buffer[0] & 0x7f) != 0)))

I don't have the scanner here, but from the log I had when tracing the 
problem, while in drivers/scsi/sg.c, sg_new_read, I dumped the following info:

masked_status 8 driver_status 0
sense 0 0 0 0 0 0 0 0
len=0 sbp 68 22 42 8 ff ff ff ff
sb_len_wr 0

using the following piece of code

    printk(KERN_INFO "masked_status %x driver_status %x\n", 
          hp->masked_status, hp->driver_status);
    printk(KERN_INFO "sense %x %x %x %x %x %x %x %x\n", srp->sense_b[0], 
          srp->sense_b[1], srp->sense_b[2], srp->sense_b[3], 
          srp->sense_b[4], srp->sense_b[5], srp->sense_b[6], srp->sense_b[7]);
        if ((CHECK_CONDITION & hp->masked_status) ||
            (DRIVER_SENSE & hp->driver_status)) {
            int sb_len = sizeof(dummy_cmdp->sr_sense_buffer);
            sb_len = (hp->mx_sb_len > sb_len) ? sb_len : hp->mx_sb_len;
            len = 8 + (int)srp->sense_b[7]; /* Additional sense length field*/
            len = (len > sb_len) ? sb_len : len;
            if ((err = verify_area(VERIFY_WRITE, hp->sbp, len)))
                goto err_out;
            __copy_to_user(hp->sbp, srp->sense_b, len);
            hp->sb_len_wr = len;
        }
    }
    printk(KERN_INFO "len=%d sbp %x %x %x %x %x %x %x %x\n", hp->sb_len_wr, 
                hp->sbp[0], hp->sbp[1], hp->sbp[2],
                hp->sbp[3], hp->sbp[4], hp->sbp[5], hp->sbp[6], hp->sbp[7]);
    printk(KERN_INFO "sb_len_wr %x\n", hp->sb_len_wr);                        
    if (hp->masked_status || hp->host_status || hp->driver_status)
          hp->info |= SG_INFO_CHECK;                            

So hrd.info ands with SG_INFO_CHECK and the sense buffer is sent to the 
handler.

This dump lead me to the conclusion that the error was downstream, but 
perhaps I am wrong. 

Should sg.c be modified, so that when SG_INFO_CHECK is set, the 
initialisation of the sense_buffer is done? 

The possibility that there may be a problem somewhere else should not be 
excluded either. I am using a driver which is "fresh from the oven".

Regards

Ze Paulo

PS: Most probably I will not be able to read my email until next Monday. 
Sorry for being so long, but didn't want to forget anything.

--------------Boundary-00=_LGDP714O5XTG771YA91A
Content-Type: application/x-bzip2;
  name="sane_log.bz2"
Content-Transfer-Encoding: base64
Content-Disposition: attachment; filename="sane_log.bz2"

QlpoOTFBWSZTWV7w2meAHnZfgEAwYGX9/gjgCBv/7//wYAk/PoBQAApRQUCgAlQEMNDQAAAaAAAA
AMNDQAAAaAAAAAMNDQAAAaAAAAAMNDQAAAaAAAAAE1KlR6k8pnqT0TTJgQYmmQ0yME0ClJCCAjEa
U2Kamm2oT1A3pGmyTsDa5v08u//GrWZjMNO/VtttLMVzxrNeOt3XktcnKttttP7a0KvdhMwSud+b
k1xnLldd2Nco4EXCotcR0a22G1bcsZhrGYmWYyEzDMJmLZQcZJaDoHY22w525etx4xq2CvzDQO8O
H57azWaZmtWstNNNGsYxjMIOuIaEmYyxjaqj8MhLWzGtYa1rWtVIvrIz2N27uDd0DZBaR0YT9n9J
Gg/Vu8n5MR8064KmO17L/DkH6O8PGR1au59H/IY8mwF6BeYc+kg2wq8ubvo828jk6zlOh4nK77tp
X5ctrXdtxZBZBdV0WpB64VeravGLsymqBb4ki6I4BjQej6yObZ33OuTIF2b2vG+l9MPEd18UF5F6
yPCiPrgVeh+xwHtvddtsGo1aCsuoYuRbBgDAwGB/RgbmwcsqifliRLg4aUcscnzQhxhEzANju/iG
wasqlvikTcN6gXEjTDZt/VBbh86qT+WFCzKWkS7A5SMPrRL9MULMqZiJOYbB9oj74JGZSWZURu7W
afO+V9AfGUprJN4+DJVdMpW5JvkpcB1/F+IY+IYcBsguViC3ovvkoccMc7ZBcs5aY+d+1yuVlzuT
+4vRiMF/+j46kv/BVR/GR/Iko+/TUxryrYj7Xbb0C7m9vclBxkV7d9ZVJe7ATaonRpDjVRNhW0Qt
oosxMyok8uOALrET75Efn5Hc4QXixu9AyhX0yqU4vm773pdIUc9EWySthVbSStgRZgjZvpex4B3H
O0FhC+UgX3cpwT7eKR+ltIXKx+HxD3R3hvO66rSHZSoPHKUzKBmUS2ibRVbAbJKtqqp23RkC7kF7
A1Yzst9US1bW27nI6K+2KcP727h+pq4XF+ts1YT5dbzeD927xDht/B5WyiPLKU8utr4ueaAcYgOx
BfB8EjSTtyosyFtZpFmKh4Bu1qKDzyqqzKjNILa2obUqTsXiFcnSXVdt2bi6h5Sl7sUhyAu7qykL
jChdv1dHVpoOTGg2ILMgO7FVsyQu2xgbeYZ8JGEFzPSoyOrvdgGvYRqTwA5ycWqC4SVtIxxjwNpG
+zwDfvY9F4TkyMD/XwcSRHPCFmTMShyRYY0RxoKXEKpzyJNiSZiVVr2x33S7HhdQ1zc2lFY/Jjrd
tAuRUfvlSl3E4PeutR5DL3h/V4VHmMpl/BGr2iHwylfJ33qjiwNObTsjVtarEcWmrLQaSi8cpKzI
EZhBZkpV6829Ura4iK8sQZkR8Rxe6o77ktpHwQ6O+nqguyz7PQPR3/C8qd4d7d3sqQssygWW6CxU
llmv3M7lmebSC8LgC23FC5h4niThy9joxsgvUM7ejTsCyy5dqzCWMrBJYUP3wiGZAbKFcSS/zFDi
c64zZVKvBgG1FNpKtlUbJSuNkLfFSzKSaqhe8OjpAtw3/OupVKvDlKWyptCVbQUzKKTg+Lk5OYe1
Sj34UexG3AXtxkqHPKi9rA3DsuaA6YqrMh58dAwLnZz+j5PZQR6yFjTm6++RkC1UdGOkjudgW1x+
IeDnz+ljSBZRTwxQZipmJd25Q7sgWZUvOqm1Ob8LweeBcWqB64zErwOw1lRq8CY1T0blDlgV1xF4
NLj2FK9GbIlsNgk++VTtTdN1nHO5uaVOdf+LuSKcKEgveG0zgA==

--------------Boundary-00=_LGDP714O5XTG771YA91A--