netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: David Miller <davem@davemloft.net>
To: tilman@imap.cc
Cc: linux-kernel@vger.kernel.org, acme@redhat.com, netdev@vger.kernel.org
Subject: Re: trying to track a change to drivers/isdn/usb-gigaset.c
Date: Mon, 07 May 2007 03:41:17 -0700 (PDT)	[thread overview]
Message-ID: <20070507.034117.115910272.davem@davemloft.net> (raw)
In-Reply-To: <463EF77A.9020808@imap.cc>

From: Tilman Schmidt <tilman@imap.cc>
Date: Mon, 07 May 2007 11:55:06 +0200

[ netdev@vger.kernel.org is the place to discuss kernel networking
  issues, thanks ]

> But I cannot remember seeing that patch via the maintainer
> addresses for ISDN or the Gigaset driver, and the LKML archive
> doesn't seem to have it either.
> 
> Where was that patch discussed? What is its rationale?

This was a wholesale change done across the entire tree, they were all
straightforward transformations that didn't change what the code did.

It was a change done so that we could hide the skb data buffering
details to the point where we could change the protocol header
pointers into 32-bit offsets which saves 4 bytes per such pointer per
protocol header stored in struct sk_buff.

We're not going to notify the maintainer of each of the hundreds of
source files we had to touch in order to pull this off, sorry.

These changes were posted to netdev@vger.kernel.org, they also
sat in Andrew Morton's -mm tree for quite some time.

       reply	other threads:[~2007-05-07 10:41 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <463EF77A.9020808@imap.cc>
2007-05-07 10:41 ` David Miller [this message]
2007-05-07 16:23   ` trying to track a change to drivers/isdn/usb-gigaset.c Tilman Schmidt

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=20070507.034117.115910272.davem@davemloft.net \
    --to=davem@davemloft.net \
    --cc=acme@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=tilman@imap.cc \
    /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).