Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 13 May 2000 13:45:21 -0700
From:      Mike Smith <msmith@freebsd.org>
To:        Peter Wemm <peter@netplex.com.au>
Cc:        obrien@FreeBSD.org, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/alpha/alpha mem.c 
Message-ID:  <200005132045.NAA05940@mass.cdrom.com>
In-Reply-To: Your message of "Sat, 13 May 2000 13:41:02 PDT." <20000513204102.7508C1CDF@overcee.netplex.com.au> 

next in thread | previous in thread | raw e-mail | index | archive | help
> "David O'Brien" wrote:
> > On Sat, May 13, 2000 at 12:43:00AM -0700, Peter Wemm wrote:
> > >   Log:
> > >   There are still too many diffs relative to the i386 version.
> > 
> > Couldn't the two be merged?  (perhaps with just a very few #ifdef's)
> 
> More of it maybe, but take (for instance) the physical addresses. They
> are 32bit vs. 64 bit.  /dev/mem and /dev/kmem are different too.

But we have an MI physical address type (vm_offset_t), so the same code 
can compile on different pointer-size systems.

> IMHO, /dev/null, /dev/zero, /dev/[u]random etc have no business being in
> the same driver as /dev/[k]mem.  I think *that* is the basic mistake here.
> Everything but mem/kmem should be in a MI "misc devices" driver.

The only parts of any of this code that I thought should be MD were the 
interrupt/entropy related parts, and most of those are just plain ugly;
I don't think any of them couldn't become MI either.

-- 
\\ Give a man a fish, and you feed him for a day. \\  Mike Smith
\\ Tell him he should learn how to fish himself,  \\  msmith@freebsd.org
\\ and he'll hate you for a lifetime.             \\  msmith@cdrom.com




To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200005132045.NAA05940>