sane-project-website/old-archive/1999-11/0084.html

<!-- received="Thu Nov 11 04:49:19 1999 PST" -->
<!-- sent="Thu, 11 Nov 1999 13:42:16 +0100" -->
<!-- name="Christian Lorenz" -->
<!-- email="Christian.Lorenz@ast.dfs.de" -->
<!-- subject="Re: SANE 1.0.1 under DEC UNIX 4.0" -->
<!-- id="" -->
<!-- inreplyto="SANE 1.0.1 under DEC UNIX 4.0" -->
<title>sane-devel: Re: SANE 1.0.1 under DEC UNIX 4.0</title>
<h1>Re: SANE 1.0.1 under DEC UNIX 4.0</h1>
<b>Christian Lorenz</b> (<a href="mailto:Christian.Lorenz@ast.dfs.de"><i>Christian.Lorenz@ast.dfs.de</i></a>)<br>
<i>Thu, 11 Nov 1999 13:42:16 +0100</i>
<p>
<ul>
<li> <b>Messages sorted by:</b> <a href="date.html#84">[ date ]</a><a href="index.html#84">[ thread ]</a><a href="subject.html#84">[ subject ]</a><a href="author.html#84">[ author ]</a>
<!-- next="start" -->
<li> <b>Next message:</b> <a href="0085.html">William L. Sebok: "next release?"</a>
<li> <b>Previous message:</b> <a href="0083.html">Douglas Gilbert: "Re: Another Microtek Phantom 636 Problem"</a>
<!-- nextthread="start" -->
<!-- reply="end" -->
</ul>
<!-- body="start" -->
Petter Reinholdtsen wrote:<br>
<i>&gt; </i><br>
<i>&gt; [Christian Lorenz]</i><br>
<i>&gt; &gt; I have downloaded sane-1.0.1 and had to make a few patches to get it</i><br>
<i>&gt; &gt; running under DEC Unix 4.0D. You find the patch attached.</i><br>
<i>&gt; </i><br>
<i>&gt; I only wounder on one of you suggested changes:</i><br>
<i>&gt; </i><br>
<i>&gt; diff -ur -C 4 sane-1.0.1/sanei/sanei_scsi.c sane-1.0.1-DU4.0/sanei/sanei_scsi.c</i><br>
<i>&gt; *** sane-1.0.1/sanei/sanei_scsi.c       Sat Apr 17 22:18:18 1999</i><br>
<i>&gt; --- sane-1.0.1-DU4.0/sanei/sanei_scsi.c Wed Nov  3 09:30:42 1999</i><br>
<i>&gt; ***************</i><br>
<i>&gt; *** 1873,1883 ****</i><br>
<i>&gt;     size_t cdb_size;</i><br>
<i>&gt; </i><br>
<i>&gt;     cdb_size = CDB_SIZE (*(u_char *) src);</i><br>
<i>&gt; </i><br>
<i>&gt; -   memset (ccb, 0, sizeof (ccb));</i><br>
<i>&gt; +   memset (&amp;ccb, 0, sizeof (ccb));</i><br>
<i>&gt;     ccb.cam_ch.my_addr = (CCB_HEADER *) &amp; ccb;</i><br>
<i>&gt; -   ccb.cam_ch.cam_ccb_len = sizeof (ccb);</i><br>
<i>&gt; +   ccb.cam_ch.cam_ccb_len = sizeof(CCB_SCSIIO);</i><br>
<i>&gt;     ccb.cam_ch.cam_func_code = XPT_SCSI_IO;</i><br>
<i>&gt;     ccb.cam_ch.cam_path_id = fd_info[fd].bus;</i><br>
<i>&gt;     ccb.cam_ch.cam_target_id = fd_info[fd].target;</i><br>
<i>&gt;     ccb.cam_ch.cam_target_lun = fd_info[fd].lun;</i><br>
<i>&gt; </i><br>
<i>&gt; Why do you want to change 'sizeof(ccb)' to 'sizeof(CCB_SCSIIO)', when</i><br>
<i>&gt; ccb already is of type CCB_SCSIIO?  It seems like duplicating</i><br>
<i>&gt; information, and a complete waste?</i><br>
<p>
You are right! When I was investigating the problem I compared this<br>
code against our CDWriter SW and found the missing line<br>
<p>
    hdr.uagt_ccblen = sizeof(CCB_SCSIIO);<br>
<p>
(around line number 1910, last part of the patch) and inserted the<br>
line from out of our code which uses the type CCB_SCSIIO. So forget<br>
about the other two changes I did them without checking the type.<br>
<p>
See you,<br>
Christian<br>
<p>
<pre>
-- 
Christian Lorenz                     DFS Deutsche Flugsicherung GmbH
fon: +49 6103 594 1650               Paul-Ehrlich-Str. 33
fax: +49 6103 594 1680               63225 Langen
<a href="mailto:Christian.Lorenz@ast.dfs.de">mailto:Christian.Lorenz@ast.dfs.de</a>   Germany
<p>
<pre>
--
Source code, list archive, and docs: <a href="http://www.mostang.com/sane/">http://www.mostang.com/sane/</a>
To unsubscribe: echo unsubscribe sane-devel | mail <a href="mailto:majordomo@mostang.com">majordomo@mostang.com</a>
</pre>
<!-- body="end" -->
<p>
<ul>
<!-- next="start" -->
<li> <b>Next message:</b> <a href="0085.html">William L. Sebok: "next release?"</a>
<li> <b>Previous message:</b> <a href="0083.html">Douglas Gilbert: "Re: Another Microtek Phantom 636 Problem"</a>
<!-- nextthread="start" -->
<!-- reply="end" -->
</ul>