linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Willy Tarreau <wtarreau@free.fr>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: Miquel van Smoorenburg <miquels@cistron.nl>,
	linux-kernel@vger.kernel.org
Subject: Re: Linux 2.2.18pre25
Date: Fri, 08 Dec 2000 10:47:46 +0100 (MET)	[thread overview]
Message-ID: <976268866.3a30ae4296b71@imp.free.fr> (raw)
In-Reply-To: <E144AfG-0003B2-00@the-village.bc.nu>
In-Reply-To: <E144AfG-0003B2-00@the-village.bc.nu>

> I asked people to explain why it was needed. I am still waiting. It is a
> patch that does nothing. I will not put random deep magic into the
> kernel.

Alan, I replied to you a few weeks ago (pre20 times) when you asked me why
I was sending you this patch. (perhaps you didn't receive my email). What I 
observed was that my netraid card had a 0xXXXX8 base address and the patch
aligned that address to 16 bytes :

|Bus  0, device   2, function  1:
|  Unknown class: Intel OEM MegaRAID Controller (rev 5).
|    Medium devsel.  Fast back-to-back capable.  BIST capable.  IRQ 10.  Master
Capable.  Latency=64.  
|    Prefetchable 32 bit memory at 0xf0000000 [0xf0000008].

as you see, the board is found at 0xf0000008, but used aligned to 0xf0000000.

my server currently works with that patch, but I'm sure it won't boot anymore
if I apply this 2.2.18pre25 alone. 

just in case, here it is again.

Cheers,
Willy

--- 18pre/drivers/scsi/megaraid.c       Wed Nov  8 16:02:45 2000
+++ 18pre+megaraid/drivers/scsi/megaraid.c      Fri Nov 10 12:03:05 2000
@@ -1920,10 +1920,14 @@
 
     pciIdx++;
 
-    if (flag & BOARD_QUARTZ)
+    if (flag & BOARD_QUARTZ) {
+       megaBase &= PCI_BASE_ADDRESS_IO_MASK;
        megaBase = (long) ioremap (megaBase, 128);
-    else
+    }
+    else {
+       megaBase &= PCI_BASE_ADDRESS_MEM_MASK;
        megaBase += 0x10;
+    }
 
     /* Initialize SCSI Host structure */
     host = scsi_register (pHostTmpl, sizeof (mega_host_config));

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/

  reply	other threads:[~2000-12-08 10:18 UTC|newest]

Thread overview: 72+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2000-12-07 20:03 Linux 2.2.18pre25 Alan Cox
2000-12-07 23:23 ` Miquel van Smoorenburg
2000-12-07 23:41   ` Alan Cox
2000-12-08  9:47     ` Willy Tarreau [this message]
2000-12-08 14:08       ` Alan Cox
2000-12-08 16:07         ` Miquel van Smoorenburg
2000-12-08 17:08           ` Alan Cox
2000-12-08 18:12       ` Philipp Rumpf
2000-12-08  0:20 ` Andrea Arcangeli
2000-12-08  0:27   ` Alan Cox
2000-12-08  0:41     ` Andrea Arcangeli
2000-12-08  0:47       ` Alan Cox
2000-12-08  1:27         ` Linus Torvalds
2000-12-08  0:44     ` Signal 11 Rainer Mager
2000-12-08  1:05       ` Jeff V. Merkey
2000-12-08  1:09       ` Michel LESPINASSE
2000-12-08  2:14         ` Rainer Mager
2000-12-08  1:20       ` Andi Kleen
2000-12-08  1:24         ` Jeff V. Merkey
2000-12-08  1:40           ` Andi Kleen
2000-12-08  1:43             ` Jeff V. Merkey
2000-12-08  1:55               ` Jeff V. Merkey
2000-12-08 19:20               ` Dr. Kelsey Hudson
2000-12-08  2:28           ` davej
2000-12-08  3:13             ` Jeff V. Merkey
2000-12-08  3:25               ` davej
2000-12-08 16:44                 ` Matthew Vanecek
2000-12-08 19:43                 ` Dr. Kelsey Hudson
2000-12-08 13:52             ` Alan Cox
2000-12-15  0:11             ` lamont
2000-12-08  1:58       ` Richard B. Johnson
2000-12-08  2:04         ` Peter Samuelson
2000-12-08 16:36           ` Matthew Vanecek
2000-12-08 16:49             ` Richard B. Johnson
2000-12-08 17:40               ` Peter Samuelson
2000-12-08 19:36           ` Dr. Kelsey Hudson
2000-12-08  9:46       ` David Woodhouse
2000-12-08 14:06         ` Alan Cox
2000-12-09 19:01           ` Matthew Vanecek
2000-12-09 19:20             ` davej
2000-12-09 23:31               ` Matthew Vanecek
2000-12-11  1:31                 ` OOPS when using 4GB memory setting Rainer Mager
2000-12-11  0:58           ` Signal 11 Rainer Mager
2000-12-11  9:05             ` Rainer Mager
2000-12-11 13:33               ` Mike Galbraith
2000-12-11 23:24                 ` Rainer Mager
2000-12-13  0:22                   ` Signal 11 - the continuing saga Rainer Mager
2000-12-13  2:17                     ` Jeff V. Merkey
2000-12-13  1:45                       ` Rainer Mager
2000-12-13  4:29                         ` Mike Galbraith
2000-12-13  9:34                           ` Rainer Mager
2000-12-13 15:40                             ` Mike Galbraith
2000-12-13  3:17                       ` Linus Torvalds
2000-12-13  9:34                         ` Rainer Mager
2000-12-13 17:43                         ` Jeff V. Merkey
2000-12-13 12:10                     ` R: " CMA
2000-12-11 14:14               ` Signal 11 davej
2000-12-08 16:21         ` Horst von Brand
2000-12-08 19:34         ` Mark Vojkovich
2000-12-08 23:16           ` Jeff V. Merkey
2000-12-08 22:24             ` David Woodhouse
2000-12-09  0:56               ` Jeff V. Merkey
2000-12-08 17:02   ` Linux 2.2.18pre25 Martin Kacer
2000-12-08 17:20     ` Alan Cox
2000-12-08 17:36       ` Martin Kacer
2000-12-08 18:08     ` Andrea Arcangeli
2000-12-08 18:30       ` Martin Kacer
2000-12-08 23:55         ` Alan Cox
2000-12-08 15:16 willy tarreau
2000-12-08 17:06 ` Alan Cox
2000-12-08 20:03 willy tarreau
2000-12-08 20:12 willy tarreau

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=976268866.3a30ae4296b71@imp.free.fr \
    --to=wtarreau@free.fr \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=linux-kernel@vger.kernel.org \
    --cc=miquels@cistron.nl \
    /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).