From owner-freebsd-current@FreeBSD.ORG Wed Jun 2 09:10:35 2004 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id BE34A16A4CF for ; Wed, 2 Jun 2004 09:10:35 -0700 (PDT) Received: from mail1.speakeasy.net (mail1.speakeasy.net [216.254.0.201]) by mx1.FreeBSD.org (Postfix) with ESMTP id 86AD843D3F for ; Wed, 2 Jun 2004 09:10:35 -0700 (PDT) (envelope-from jhb@FreeBSD.org) Received: (qmail 30351 invoked from network); 2 Jun 2004 16:10:35 -0000 Received: from dsl027-160-063.atl1.dsl.speakeasy.net (HELO server.baldwin.cx) ([216.27.160.63]) (envelope-sender ) encrypted SMTP for ; 2 Jun 2004 16:10:34 -0000 Received: from 10.50.41.233 (gw1.twc.weather.com [216.133.140.1]) by server.baldwin.cx (8.12.11/8.12.11) with ESMTP id i52GAKN9065507; Wed, 2 Jun 2004 12:10:27 -0400 (EDT) (envelope-from jhb@FreeBSD.org) From: John Baldwin To: Bruce Evans Date: Wed, 2 Jun 2004 12:11:05 -0400 User-Agent: KMail/1.6 References: <200406011558.57696.jhb@FreeBSD.org> <200406011733.03424.jhb@FreeBSD.org> <20040602175149.B23521@gamplex.bde.org> In-Reply-To: <20040602175149.B23521@gamplex.bde.org> MIME-Version: 1.0 Content-Disposition: inline Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Message-Id: <200406021211.05820.jhb@FreeBSD.org> X-Spam-Checker-Version: SpamAssassin 2.63 (2004-01-11) on server.baldwin.cx cc: mdodd@FreeBSD.org cc: peter@FreeBSD.org cc: current@FreeBSD.org Subject: Re: Remove atdevbase? X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 02 Jun 2004 16:10:35 -0000 On Wednesday 02 June 2004 03:54 am, Bruce Evans wrote: > On Tue, 1 Jun 2004, John Baldwin wrote: > > On Tuesday 01 June 2004 05:14 pm, Bruce Evans wrote: > > > On Tue, 1 Jun 2004, John Baldwin wrote: > > > > http://www.FreeBSD.org/~jhb/patches/kill_atdevbase.patch > > > > > > Review of what I can quote easily: this URL appears to be well formed. > > > > Do you want me to mail you a copy? > > I can fetch it easily enough. It just takes a bit longer, and it doesn't > make sense to ask for review of small unseen material, so any review would > have to quote the material to discuss its detail. I was more interested in if people would object to the change in principle (i.e. the notion of removing atdevbase) rather than its implementation as the implementation is rather straightforward. --- //depot/vendor/freebsd/src/sys/alpha/include/pc/bios.h 2001/07/17 00:20:42 +++ //depot/user/jhb/acpipci/alpha/include/pc/bios.h 2004/05/17 13:21:18 @@ -71,13 +71,8 @@ }; extern void bios32(caddr_t func_addr, struct bios32_args *args); -#if 0 -#define BIOS_PADDRTOVADDR(x) (((x) - ISA_HOLE_START) + atdevbase) -#define BIOS_VADDRTOPADDR(x) (((x) - atdevbase) + ISA_HOLE_START) -#else #define BIOS_PADDRTOVADDR(x) (x) #define BIOS_VADDRTOPADDR(x) (x) -#endif /* * System Management BIOS / Desktop Management Interface tables --- //depot/vendor/freebsd/src/sys/amd64/amd64/machdep.c 2004/03/07 16:20:31 +++ //depot/user/jhb/acpipci/amd64/amd64/machdep.c 2004/05/17 13:21:18 @@ -141,7 +141,6 @@ SYSINIT(cpu, SI_SUB_CPU, SI_ORDER_FIRST, cpu_startup, NULL) int _udatasel, _ucodesel, _ucode32sel; -u_long atdevbase; int cold = 1; @@ -1119,8 +1118,6 @@ thread0.td_pcb = (struct pcb *) (thread0.td_kstack + KSTACK_PAGES * PAGE_SIZE) - 1; - atdevbase = ISA_HOLE_START + KERNBASE; - /* * This may be done better later if it gets more high level * components in it. If so just link td->td_proc here. --- //depot/vendor/freebsd/src/sys/amd64/include/md_var.h 2004/04/10 21:30:29 +++ //depot/user/jhb/acpipci/amd64/include/md_var.h 2004/05/17 13:21:18 @@ -37,7 +37,6 @@ */ extern long Maxmem; -extern u_long atdevbase; /* offset in virtual memory of ISA io mem */ extern u_int basemem; extern int busdma_swi_pending; extern u_int cpu_exthigh; --- //depot/vendor/freebsd/src/sys/dev/fb/vga.c 2004/04/03 07:32:31 +++ //depot/user/jhb/acpipci/dev/fb/vga.c 2004/05/17 13:21:18 @@ -67,7 +67,7 @@ #if !defined(__amd64__) #define BIOS_PADDRTOVADDR(x) (x) #else -#define BIOS_PADDRTOVADDR(x) (((x) - ISA_HOLE_START) + atdevbase) +#define BIOS_PADDRTOVADDR(x) ((x) + KERNBASE) #endif #endif --- //depot/vendor/freebsd/src/sys/dev/kbd/atkbd.c 2003/10/07 02:25:29 +++ //depot/user/jhb/acpipci/dev/kbd/atkbd.c 2004/05/27 07:11:24 @@ -50,6 +50,7 @@ #include #include +#include #endif /* __i386__ */ #include --- //depot/vendor/freebsd/src/sys/i386/i386/machdep.c 2004/05/10 12:08:15 +++ //depot/user/jhb/acpipci/i386/i386/machdep.c 2004/05/17 13:21:18 @@ -160,7 +160,7 @@ SYSINIT(cpu, SI_SUB_CPU, SI_ORDER_FIRST, cpu_startup, NULL) int _udatasel, _ucodesel; -u_int atdevbase, basemem; +u_int basemem; int cold = 1; @@ -1954,7 +1954,6 @@ thread0.td_kstack = proc0kstack; thread0.td_pcb = (struct pcb *) (thread0.td_kstack + KSTACK_PAGES * PAGE_SIZE) - 1; - atdevbase = ISA_HOLE_START + KERNBASE; /* * This may be done better later if it gets more high level --- //depot/vendor/freebsd/src/sys/i386/i386/symbols.raw 2001/07/17 00:20:42 +++ //depot/user/jhb/acpipci/i386/i386/symbols.raw 2004/05/17 13:21:18 @@ -8,7 +8,6 @@ _IdlePTD _PTD _panicstr - _atdevbase # _version #dmesg _msgbufp --- //depot/vendor/freebsd/src/sys/i386/include/md_var.h 2004/04/10 21:30:29 +++ //depot/user/jhb/acpipci/i386/include/md_var.h 2004/05/17 13:21:18 @@ -42,7 +42,6 @@ extern int (*copyout_vector)(const void *kaddr, void *udaddr, size_t len); extern long Maxmem; -extern u_int atdevbase; /* offset in virtual memory of ISA io mem */ extern u_int basemem; /* PA of original top of base memory */ extern int busdma_swi_pending; extern u_int cpu_exthigh; --- //depot/vendor/freebsd/src/sys/i386/include/pc/bios.h 2002/09/23 11:55:25 +++ //depot/user/jhb/acpipci/i386/include/pc/bios.h 2004/05/17 13:21:18 @@ -61,8 +61,8 @@ extern u_int32_t bios_sigsearch(u_int32_t start, u_char *sig, int siglen, int paralen, int sigofs); -#define BIOS_PADDRTOVADDR(x) (((x) - ISA_HOLE_START) + atdevbase) -#define BIOS_VADDRTOPADDR(x) (((x) - atdevbase) + ISA_HOLE_START) +#define BIOS_PADDRTOVADDR(x) ((x) + KERNBASE) +#define BIOS_VADDRTOPADDR(x) ((x) - KERNBASE) /* --- //depot/vendor/freebsd/src/sys/i386/pci/pci_pir.c 2004/05/04 14:21:13 +++ //depot/user/jhb/acpipci/i386/pci/pci_pir.c 2004/05/27 07:11:24 @@ -38,6 +38,7 @@ #include #include #include +#include #include #include #include --- //depot/vendor/freebsd/src/sys/isa/syscons_isa.c 2003/10/29 12:50:44 +++ //depot/user/jhb/acpipci/isa/syscons_isa.c 2004/05/27 07:11:24 @@ -47,6 +47,7 @@ #include #include +#include #include --- //depot/vendor/freebsd/src/sys/pc98/i386/machdep.c 2004/03/28 04:05:27 +++ //depot/user/jhb/acpipci/pc98/i386/machdep.c 2004/05/17 13:21:18 @@ -167,7 +167,7 @@ #endif int _udatasel, _ucodesel; -u_int atdevbase, basemem; +u_int basemem; #ifdef PC98 static int ispc98 = 1; @@ -2013,7 +2013,6 @@ thread0.td_kstack = proc0kstack; thread0.td_pcb = (struct pcb *) (thread0.td_kstack + KSTACK_PAGES * PAGE_SIZE) - 1; - atdevbase = ISA_HOLE_START + KERNBASE; /* * This may be done better later if it gets more high level -- John Baldwin <>< http://www.FreeBSD.org/~jhb/ "Power Users Use the Power to Serve" = http://www.FreeBSD.org