Re: Remove atdevbase?

From: John Baldwin <jhb_at_FreeBSD.org>
Date: Wed, 2 Jun 2004 12:11:05 -0400
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
_at__at_ -71,13 +71,8 _at__at_
 };
 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
_at__at_ -141,7 +141,6 _at__at_
 SYSINIT(cpu, SI_SUB_CPU, SI_ORDER_FIRST, cpu_startup, NULL)
 
 int	_udatasel, _ucodesel, _ucode32sel;
-u_long	atdevbase;
 
 int cold = 1;
 
_at__at_ -1119,8 +1118,6 _at__at_
 	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
_at__at_ -37,7 +37,6 _at__at_
  */
 
 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
_at__at_ -67,7 +67,7 _at__at_
 #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
_at__at_ -50,6 +50,7 _at__at_
 
 #include <vm/vm.h>
 #include <vm/pmap.h>
+#include <vm/vm_param.h>
 #endif /* __i386__ */
 
 #include <sys/kbio.h>
--- //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
_at__at_ -160,7 +160,7 _at__at_
 SYSINIT(cpu, SI_SUB_CPU, SI_ORDER_FIRST, cpu_startup, NULL)
 
 int	_udatasel, _ucodesel;
-u_int	atdevbase, basemem;
+u_int	basemem;
 
 int cold = 1;
 
_at__at_ -1954,7 +1954,6 _at__at_
 	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
_at__at_ -8,7 +8,6 _at__at_
 	_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
_at__at_ -42,7 +42,6 _at__at_
 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
_at__at_ -61,8 +61,8 _at__at_
 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
_at__at_ -38,6 +38,7 _at__at_
 #include <sys/sysctl.h>
 #include <vm/vm.h>
 #include <vm/pmap.h>
+#include <vm/vm_param.h>
 #include <machine/md_var.h>
 #include <dev/pci/pcivar.h>
 #include <dev/pci/pcireg.h>
--- //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
_at__at_ -47,6 +47,7 _at__at_
 
 #include <vm/vm.h>
 #include <vm/pmap.h>
+#include <vm/vm_param.h>
 
 #include <i386/isa/timerreg.h>
 
--- //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
_at__at_ -167,7 +167,7 _at__at_
 #endif
 
 int	_udatasel, _ucodesel;
-u_int	atdevbase, basemem;
+u_int	basemem;
 
 #ifdef PC98
 static int	ispc98 = 1;
_at__at_ -2013,7 +2013,6 _at__at_
 	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 <jhb_at_FreeBSD.org>  <><  http://www.FreeBSD.org/~jhb/
"Power Users Use the Power to Serve"  =  http://www.FreeBSD.org
Received on Wed Jun 02 2004 - 07:10:35 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:37:55 UTC