Subject: scn.c and ncr.c enhancements.
To: Matthias Pfaller <leo@dachau.marco.de>
From: Ian Dall <Ian.Dall@dsto.defence.gov.au>
List: port-pc532
Date: 04/14/1997 08:45:55
I started using Phil's scn driver at 115.2 kBaud (with sc26c92
duarts) on my modem and began getting overruns again.
I did some more work on ncr.c. I reduced the read chunk size even
more, but I unrolled the loops so that the relatively expensive
ncr_ready() check is only done at the end of each block. This didn't
solve my problem completely. I then had another look at scn.c. I
realized that I misunderstood the device. You don't need to turn off
the automatic control of RTS to be able to manually de-assert RTS. So,
now we have hardware control of RTS as well as flow control on the
ring buffer --- the best of bosth worlds. This actually simplifies the
code, since I got rid of the scn_soft_rts flag --- there is no longer
a trade off involved. I also enabled a bit of code Phil had put in
(but not tested?) to only interrupt when the fifo is 3/4 full (for the
sc26c92 only) and added some code of my own to completely fill the
transmit buffer. This leads to a dramatic reduction in interrupt
overhead. When receiving at 38.4kB sustained, the CPU spends about 90%
of its time in "system" with the 2681 duarts. With the 26c92 duarts,
and this new code, it drops to under 20%. (It could go even lower I am
sure if we could transfer more than 1 byte at a time up to the line
discipline layers.) On transmit the levels are lower, but the trend is
the same, dropping from about 15% CPU to 2% CPU in system state.
I can now talk at 115.2kBaud to my modem and haven't seen a single
fifo overrun yet in several days of use. (Of course, that is not
115.2kBaud sustained, but with compression I do get little bursts of
115.2kBaud).
BTW I have been using uuencode, as the lowest common denominator for
sending in stuff like this. An obvious alternative is to use a MIME
base64 attachment. Can everyone read mime these days, or is it beter
to stick with uuencode?
Here are the patches:
begin 644 pc532.diffs.gz
M'XL(`````````^U8;7/:2!+^+'Y%;U+G$@892;R;LS<.V`EU?MD"7'$NE5()
M:4"J"$D[(XQ]B>^W7\^,),#@D-T]ZNY#^"!&H^EGNGOZ;=KU)Q/0Z!R.CBJ,
M.A7VR"HV=;Q*[-2K9L4E]Y70H4<.O/2AH&G:+F)EY,WARJ9@ZF":QT;[6#?`
M:+>;A5*I]"*R<A6%<!93,&J@&\>U9D[UY@UHIMDN-Z'$_UKPYDT!7I/0]2<%
MP)$_"5TR@>ONP!H-^_\\M_K7!>TU3ODA69M5#+-5*&W]4C?,[5^LV^O!S>7E
M>0_4U>F*T2BN,9&2COI7YS>W(\70Q4_RWFB7#1.9;^IEPQ#</W$:EMB)[X`?
M)@4-56#%[LRV_%!E3AEBSV:D#*Z=V`$)Y:!8*.U:9M.I)9>"PA(Z=Q)`DGJU
MI5LLFB0.'#*G@]]PSV>TG8*FS"W'PV,[Y%.=0BE_SU"1\BLNNX\"9#P@D'WG
M',$)_-:[.K/.>KW!^7"8;4()\]TRL`X76&&XBL7!V(_48D?JIE4OFW74#?X;
MM:90CJ+X$U"YJ)38[B,*6H23$]"+N#M^5*91$G&M$4KG<<)W4IZ0+<7U.2H.
M9M$]<U4U8Z_(^9."EM?.%BI0DQ1$D)84Y2M_*)1,?8;XL*H1`+`LF\TL2WU%
MS5?($RP5LT:U74$KU(:@7M=726"X&2,_+$1NH)5:2ID+\Q/M)]I/M)]H_T=H
M6<3$\)<'3DR&BDB((NZ+C%$4B49D$!G@_UQ&2'`;@8>1GOG_(M%$Q54RY/]0
MMDC7)G!XL@$@@;43SJ&6`8U?`))2K>2:'`#%^%]G'B75=!E"^1:NZ$W=>K3%
M;&,I@OS_V\MKQ6JV\!/'4T-^;/S=P0($C/JQ>%DQM+]@:2NF)M%K>T6O[A7=
MW"NZL5=T?9_H[7V"M_8)WMPG>&.?X'MUT[UZZ5Z==+\^NB=PO*S:\R"1Z$_/
MTGR>-"MK&6]7!9#D87XS6RY3A4R7I1QM5[Y<KQ[6,V9:-RB\+EC8?F*%48(%
MPN^\/,`:(*\'C@ON[K8'<\+M;0_Q84?;0ZSYPVT/236T$]GV,'DJ-'GG8]GV
M:-9XUP.?#=GT\$,GF+L$7B$M)OTC[]6SR7N;BLF"QC,Z3HA[OT43?O4V.GG3
MP8E"EH!0/"["M>Q3[3,N^=I'$QI]U,L@!T8V,+-!]6DK"GW8!!GT/N8X*V-S
M95S%,0<4TE;-5MFH0ZEJ-LN&E%@4?FZLG2:/,>$UW[!K-KIM,ZW\4ML)9U1'
MH^%/CJ6]1BJ=EUB50S2#>)X`M1,"GC_U@(31'/]8!(T:C'TL>_P9@86-E8D;
M30%M"**0T&C.?H7#"K>S/P52*,'SW^$:<%K4<ME0AO`3?WP^\EE,B`NG>%BF
MKJ=E,.>`D00N^A<WD'CH!5X4N("&TTA9_-Z*240A2CQ"Q4+D(O_.8$'P#.>X
MTK/O"6`1S4&N!H9U<7%[>9D2"%Z59XRBKN';":[5K<'=AQY\2X?]ZY&LOX$$
M&,&^OD1[<`+_5K=1%[/R?=E<2\4+B)/`V\$[F$5H[9SX##4:/!93%H45\(M%
MMIN>LBFU+*@.Q$97-[WSM`%EU*JR.R<'NK0Z7@#[H*%O:*?,L>AX2L2U@I]8
M/N?Y>.!P<(`W$SB%]>EB:J#(>'\BC*H[&`V[HR&LD/H,V-SQ\$#P)?%\Q@\;
M#PA"<H_%O&?',0D9/X6-R:-TMAL0M'^$YIMQ/6A*RN32\XO"AH;=:^OF-^MM
MORLZARF[42Q6R(L)'IA8RHGI@^LS>QP0&4U+.P&`TTD5/%CC('*^$#>-.2)*
MY_KAGH3:S=3?,(7/XT!/6[O+ZTS:PLR$$>U+'8F'-Q>CKAKX(='%SM\E,-8(
MC**($!M*@F_?X!=$?R:!T-TBHE]XYU*H!<,<14J,D-G^$FP;FK&)AJ&)-ZO_
M\:[W%@EY6$`#$HS!W\4.CGODN%:("8+E/B`7+GDI/6,&H[3(M)F`&I[7#J9+
M/X8D35XL/<WOVQ@DN*!(B(F9FWA"_>F4J[^RX&F7"H=,)>5F,^I?]:_?R0.O
MMQO2W^JM9KF=-GP513WD6[+%IX0'>XN_?#X*+,J9`17M+8F+LHFL*&MNN#0V
M5&2(2N0;#G`[:_0>[[?OLQ@J[KA9!YHFR6->4HG+[[,/8INM!JUW,KBM;K;J
M:<.7/$U9<;;<WTBXYFZ[@5(>X^!!S5^?TEFIZY;10)\J&2V=UQ%2U3-JRL!M
M6J,[#$B=O//B3`)[BA&21RJK?W%Y\R%KPTS47S8EG5%#`AD8N`<CV4C8G%R)
M"VG8.UD>T?O^AS.1+U;3Q<9R/8T5K3J:3HW+4ZOR@10($\/"_D+FL7H@;6>!
M,\4L[,@I7J\0SAA*]O9V^+&31P"QE>.-L=S^U'UO#0>?40-#K$WN+"Q-4@6@
MB=_=W4$0V2[,L'#VXX"(LL=V,.HP0"BSX;3-7T&+@W$:AAWD'-.&D[$5S9/?
MY?%O[-H[&_&ZR9&AB6]W+HP!'<L.V0SKB[R894<I?`KBSFV::*<"J7=K]:\&
M'$E=_^K/*!<^G4P>K*PL7WB\6?1#:BA)BU>WBB7:<9I1%(8[IL3^(JSA>X**
9>\=_0X@"(`O'/,/DGO!4^`^SU5M@Z1P``%>W
`
end