netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Stephen Hemminger <shemminger@osdl.org>
To: David Miller <davem@davemloft.net>
Cc: gerrit@erg.abdn.ac.uk, netdev@vger.kernel.org
Subject: Re: [PATCH 1/3] [NET]: Supporting UDP-Lite (RFC 3828) in Linux
Date: Fri, 10 Nov 2006 14:48:49 -0800	[thread overview]
Message-ID: <20061110144849.207e814c@freekitty> (raw)
In-Reply-To: <20061110.143846.93212026.davem@davemloft.net>

On Fri, 10 Nov 2006 14:38:46 -0800 (PST)
David Miller <davem@davemloft.net> wrote:

> From: Stephen Hemminger <shemminger@osdl.org>
> Date: Fri, 10 Nov 2006 09:37:15 -0800
> 
> > On Fri, 10 Nov 2006 16:09:20 +0000
> > Gerrit Renker <gerrit@erg.abdn.ac.uk> wrote:
> > 
> > > [NET/IPv4]: Support for the UDP-Lite protocol (RFC 3828)
> > > 
> > > This adds support for UDP-Lite to the IPv4 stack, provided as an extension
> > > to the existing UDPv4 code:
> > > 	* generic routines are all located in net/ipv4/udp.c
> > > 	* UDP-Lite specific routines are in net/ipv4/udplite.c
> > > 	* MIB/statistics support in /proc/net/snmp and /proc/net/udplite
> > > 	* shared API with extensions for partial checksum coverage
> > > 	* consolidation of shared code
> > > 
> > > 
> > 
> > This code is does too much inlining (like existing network code).
> > Should it be made configurable?
> 
> It doesn't get built at all if you check his patches :-)
> 
> But once that's fixed up, I think we can fix that with a followon
> patch.  We've put Gerrit through the ringer enough on this stuff.

Agreed.  But we can do some code rearranging/deinlining early in 2.6.20

-- 
Stephen Hemminger <shemminger@osdl.org>

  reply	other threads:[~2006-11-10 22:49 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-11-10 16:09 [PATCH 1/3] [NET]: Supporting UDP-Lite (RFC 3828) in Linux Gerrit Renker
2006-11-10 17:37 ` Stephen Hemminger
2006-11-10 22:38   ` David Miller
2006-11-10 22:48     ` Stephen Hemminger [this message]
2006-11-13  7:43     ` Gerrit Renker
2006-11-13  7:56   ` Gerrit Renker
2006-11-10 22:38 ` David Miller
2006-11-13  7:34   ` Gerrit Renker
2006-11-13  7:46     ` David Miller
     [not found]     ` <200611130809.34613@strip-the-willow>
     [not found]       ` <20061113.001615.74565996.davem@davemloft.net>
2006-11-13  8:32         ` Gerrit Renker
2006-11-13  9:28           ` David Miller

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20061110144849.207e814c@freekitty \
    --to=shemminger@osdl.org \
    --cc=davem@davemloft.net \
    --cc=gerrit@erg.abdn.ac.uk \
    --cc=netdev@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).