From 08e1a93c4a0c6318f4a3db586051535c5d26f4c1 Mon Sep 17 00:00:00 2001 From: Paul Menzel Date: Fri, 18 Mar 2022 15:33:19 +0100 Subject: [PATCH] send: Make `byteCount` unsigned long Currently, negative benchmark values are shown: @ira$ send okeanos 34141 start : 1647613816.868339 close : 1647613826.868524 done : 1647613826.874427 -1209148824 bytes in 10.006088 seconds -120841314.207910 bytes / seconds -966730513.663282 bits / seconds -966730.513663 kBit/s / seconds -966.730514 MBit/s / seconds With 10 Gbit/s, in 10 s seconds theoretically up to 12.5 GB could be transferred. But a 32-bit (signed) integer can at maximum store the number (2147483648 - 1), and the unsigned integer around 4.3 million, still too small. Therefore, use unsigned long to be able to store the number bytes without overflow for the foreseeable future. Now we have: @ira$ ./send okeanos 34141 start : 1647626064.313421 close : 1647626074.313503 done : 1647626074.317791 10534190976 bytes in 10.004370 seconds 1052958954.536867 bytes / seconds 8423671636.294939 bits / seconds 8423671.636295 kBit/s / seconds 8423.671636 MBit/s / seconds Resolves: https://github.molgen.mpg.de/mariux64/mariux64-issues/issues/23 --- send/send | Bin 35552 -> 35552 bytes send/send.c | 4 ++-- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/send/send b/send/send index b337babeb5c34a51bedc33dcd802563024120cce..da1fe50ea76ca26b6e75fbd9406b474f627fb8d6 100755 GIT binary patch delta 1425 zcmb7^T}T{P6vyu!cdMh-8Fh6ig*vg=eECsJKVQmh>y(=&QcPlEg$R;h2q{S$qQuH( zZKfMK47Q86(ma$lg+N+DK|2qz65L%AVhgqyDkQ!cLc$~~G%Ut$Y`4>M#?90;5a__p zoVoX$-~WHk%+A7yurMNw7Ev)*NW3Fwm(y-pysK!0O{WGGd!T$jrAz!qMICjxSotiy z#TzP>w;g6qJ4ViRnW3 zuiui+Ct;%lHelrk6tJ`q+dJz?6^GR7IS;|iUm=v?00#OV1;6TF-hC zL;7i7>yJMlAUgH3E8r z9(Klv%#dhQb>4tmix}C!(T#}%s6l-V#Y4=nQ;h0aEBMo9LSo6@VEVgwn#N%^ao=j| zqgE$iY1Qg1L}2w4SdHTo1I*+h&uTRPxDKNSZ|W~S1d5=aBR=R81g(|5y7I_`TKw!z z%>Gw4y$NKEwG+C(!mn4=N7KcGWTdXCjMS0JNacDGe8Nx9GIQjZ>$(|9j^UM=mlMNK zKkmI(bs+5}#gfMgGfdi)Ojj{;ie0&5EKnv+;SwT~fYy_VbAI9C{f7&qkExVPcYEgE z1!G7__>{B%p*$8+;X~xovej<$z~59Ynr~>={~@MCoEh4ZOTYKxt~~L!Z!ri!@89_I z#hyl~=3{&jiBzMJ@b2)gE#}p5dwA!zp*r=@bA+_Bp$5F}2%EVr=JK|UH<__m#8SBe z=N#~Xl-l}-K zPc4Di)5*bFx7fht69Id%uSKADd2|0~&{?Yg3yPf1f#dXyJwLEsq{E!Xn#2b*Xg+o} F_z%_-MauvH delta 1287 zcmZWpTSyd97(Qp*O~>1g?z-CrbTzz`saa9^P|TuJW9fsIWkI53WrST#6j_&A6o$!K zJV-uN*i%sv);wq;yIPS35fws7LPi7|l`cz#)!Ou*aT}EeX3m-Of4Be4*`4fSCcBv4 zD9qMJBJHYI5^xR2R@-}Wta&A#A~|N;j({kQ_}#OrDoVnn z=ypz`vzel7tjR%0lq&+>HFgk?M*69!{}|wWn9iZQuU-Z+k=?T}9FJw$rP@)~(xu>nyNE znS4sha!p4g6f{*m44Nv3gQf&w6ta%WLP(Ulnp9N{nz%MPN>6;9pwO?Ks%)SKO{;|v z5QS0SExnL9U*O`dYRdaSm4duF=<83N&BeKzbQBlYFqb9Fc%?H27c#alAgo_R@aqH_ z2-;8K=_6ALj~sg)dy1vkjta-_l8*F083_sMVMeA+f+_cv8#l$U9FF~q|dI&(MUTBRU&Qv~Rq%Fkm1RvxejohIiP8yU*C z<2vPi{&)%#u#&;D@fGxcfnJRELir4=VD<1LviW(aP}WB;i^^KCbb Fe*yQP11