netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "David S. Miller" <davem@redhat.com>
To: sim@netnation.com
Cc: ralph+d@istop.com, hadi@shell.cyberus.ca, xerox@foonet.net,
	fw@deneb.enyo.de, netdev@oss.sgi.com, linux-net@vger.kernel.org
Subject: Re: Route cache performance tests
Date: Mon, 16 Jun 2003 16:08:56 -0700 (PDT)	[thread overview]
Message-ID: <20030616.160856.35828947.davem@redhat.com> (raw)
In-Reply-To: <20030616230922.GC18484@netnation.com>

   From: Simon Kirby <sim@netnation.com>
   Date: Mon, 16 Jun 2003 16:09:22 -0700

   On Mon, Jun 16, 2003 at 03:44:01PM -0700, David S. Miller wrote:
   
   > I pushed all of our current work to Linus's tree.
   > But for your convenience here are the routing diffs
   > against plain 2.5.71
   
   Trying to apply against 2.5.71:
   
   patching file net/ipv4/route.c
   Hunk #2 succeeded at 454 (offset -2 lines).
   Hunk #3 succeeded at 738 (offset -2 lines).
   Hunk #4 succeeded at 775 (offset -2 lines).
   patching file net/ipv4/route.c
   Hunk #1 FAILED at 739.
   1 out of 1 hunk FAILED -- saving rejects to file net/ipv4/route.c.rej
   
   Trying to apply against 2.5.71-bk2:
   
   patching file net/ipv4/route.c
   patching file net/ipv4/route.c
   Hunk #1 FAILED at 739.
   1 out of 1 hunk FAILED -- saving rejects to file net/ipv4/route.c.rej
   
   Missing something between?
   
   Code from bk2:
   
   static int rt_intern_hash(unsigned hash, struct rtable *rt, struct rtable **rp)
   {
           struct rtable   *rth, **rthp;
           unsigned long   now = jiffies;
           int attempts = !in_softirq();
   
   Patch:
   
It depends upon the first patch that I enclosed.
What I gave you was a 2-part patch, the first one
did:

@@ -721,6 +740,9 @@
 {
 	struct rtable	*rth, **rthp;
 	unsigned long	now = jiffies;
+	struct rtable *cand = NULL, **candp = NULL;
+	u32 		min_score = ~(u32)0;
+	int		chain_length = 0;
 	int attempts = !in_softirq();
 
 restart:

The second one did:

@@ -739,13 +739,19 @@
 static int rt_intern_hash(unsigned hash, struct rtable *rt, struct rtable **rp)
 {
 	struct rtable	*rth, **rthp;
-	unsigned long	now = jiffies;
-	struct rtable *cand = NULL, **candp = NULL;
-	u32 		min_score = ~(u32)0;
-	int		chain_length = 0;
+	unsigned long	now;
+	struct rtable *cand, **candp;
+	u32 		min_score;
+	int		chain_length;
 	int attempts = !in_softirq();
 
...

I have no idea why it doesn't apply.
Nothing else has happened in these bits of code for a while.

  reply	other threads:[~2003-06-16 23:08 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-06-10  7:57 Route cache performance tests Simon Kirby
2003-06-10 11:23 ` Jamal Hadi
2003-06-10 20:36   ` CIT/Paul
2003-06-10 13:34 ` Ralph Doncaster
2003-06-10 13:39   ` Jamal Hadi
2003-06-13  6:20 ` David S. Miller
2003-06-16 22:37   ` Simon Kirby
2003-06-16 22:44     ` David S. Miller
2003-06-16 23:09       ` Simon Kirby
2003-06-16 23:08         ` David S. Miller [this message]
2003-06-16 23:27           ` Simon Kirby
2003-06-16 23:49             ` Simon Kirby
2003-06-17 15:59               ` David S. Miller
2003-06-17 16:50                 ` Robert Olsson
2003-06-17 16:50                   ` David S. Miller
2003-06-17 17:29                     ` Robert Olsson
2003-06-17 19:06                       ` Mr. James W. Laferriere
2003-06-17 20:12                         ` Robert Olsson
2003-06-17 20:07                   ` Simon Kirby
2003-06-17 20:17                     ` Martin Josefsson
2003-06-17 20:37                       ` Simon Kirby
2003-06-17 20:36                         ` David S. Miller
2003-06-17 20:51                           ` Simon Kirby
2003-06-17 20:49                             ` David S. Miller
2003-06-18  5:50                             ` Pekka Savola
2003-06-17 20:49                     ` Robert Olsson
2003-06-17 21:07                     ` Simon Kirby
2003-06-17 22:50                       ` Simon Kirby
2003-06-17 23:07                         ` David S. Miller
2003-06-17 22:11                     ` Ralph Doncaster
2003-06-17 22:08                       ` David S. 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=20030616.160856.35828947.davem@redhat.com \
    --to=davem@redhat.com \
    --cc=fw@deneb.enyo.de \
    --cc=hadi@shell.cyberus.ca \
    --cc=linux-net@vger.kernel.org \
    --cc=netdev@oss.sgi.com \
    --cc=ralph+d@istop.com \
    --cc=sim@netnation.com \
    --cc=xerox@foonet.net \
    /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).