Date: Thu, 7 Sep 2000 14:20:18 +0300 From: Ruslan Ermilov <ru@sunbay.com> To: Luigi Rizzo <luigi@info.iet.unipi.it> Cc: Garrett Wollman <wollman@khavrinen.lcs.mit.edu>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/netinet ip_divert.c ip_icmp.c ip_input.c ip_mroute.c ip_output.c raw_ip.c udp_usrreq.c Message-ID: <20000907142018.H46475@sunbay.com> In-Reply-To: <200009070728.JAA27581@info.iet.unipi.it>; from luigi@info.iet.unipi.it on Thu, Sep 07, 2000 at 09:28:48AM %2B0200 References: <20000907092610.A46475@sunbay.com> <200009070728.JAA27581@info.iet.unipi.it>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Sep 07, 2000 at 09:28:48AM +0200, Luigi Rizzo wrote: > > > According to Mike Karels (in a message to me dated 2 June 1997), > > > BSD/OS leaves the ip_id field in network order all the time, but swaps > > > in the raw IP code to preserve the old interface. There is never any > > > arithmetic performed on the IP ID, only comparisons for equality, so > > > leaving out the swap should save a cycle or three. > > > > > The same does NetBSD, but I feel more logical is to have all these fields > > in host byte order right after ip_input() processing. > > The problem with swapping is that such fields can be part of an mbuf cluster, > which is potentially shared by other flows of control and thus not > expected to be modified. > Yes, my patch has fixed in particular PR 16240 which is exactly this problem. Again, I think it is logical to have all three fields (ip_len, ip_off and ip_id) in the ``same'' (host or network) byte order throughout the code, rather than handle `ip_id' differently (did you look over the changes?) -- Ruslan Ermilov Oracle Developer/DBA, ru@sunbay.com Sunbay Software AG, ru@FreeBSD.org FreeBSD committer, +380.652.512.251 Simferopol, Ukraine http://www.FreeBSD.org The Power To Serve http://www.oracle.com Enabling The Information Age To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20000907142018.H46475>