Date: Sat, 18 Jan 2003 10:48:28 +1100 (EST) From: Bruce Evans <bde@zeta.org.au> To: phk@freebsd.org Cc: cvs-committers@freebsd.org, <cvs-all@freebsd.org> Subject: Re: cvs commit: src/sys/conf files.ia64 Message-ID: <20030118103138.M9539-100000@gamplex.bde.org> In-Reply-To: <14438.1042844975@critter.freebsd.dk>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, 18 Jan 2003 phk@freebsd.org wrote: > In message <20030118100149.T9442-100000@gamplex.bde.org>, Bruce Evans writes: > >On Fri, 17 Jan 2003, Poul-Henning Kamp wrote: > > > >> phk 2003/01/17 14:09:12 PST > >> > >> Modified files: > >> sys/conf files.ia64 > >> Log: > >> ia64 didn't need subr_diskslice.c and subr_diskmbr.c either. > >> > >> Revision Changes Path > >> 1.44 +0 -2 src/sys/conf/files.ia64 > > > >These files are not machine-dependent. If I had an ia64, then they > >would be needed for the fd and cd drivers, since my versions of the > >fd driver uses subr_diskslice.c directly and my version of the cd driver > >uses the disk mini-layer which uses subr_diskmbr.c in the NOGEOM case. > > This time I'm absolutely sure I only committed to FreeBSD. You are removing infrastructure that is in use and would be needed for future commits. I'm waiting for ken to commit my patches to unbreak the cd driver. My version of the fd driver has used the slice layer since 1995. I don't expect you to supply patches for it. Just don't remove its infrastructure. The fd driver and cd drivers generally are especially interesting since they don't fit well into either GEOM or the disk mini- layer. For the fd driver, this is mainly because it has multiple densities and wants multiple device names for the top-level device. It is unreasonable for a general layer to handle these complications. > BTW: you read arch@FreeBSD.org, right ? Yes. I misread the point that you would actively remove non-GEOM bits in -current after the branchpoint, leaving only 5.0 and previous versions usable for me. Bruce 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?20030118103138.M9539-100000>