Date: Tue, 19 Nov 2002 21:44:16 -0800 (PST) From: Nate Lawson <nate@root.org> To: Kris Kennaway <kris@obsecurity.org> Cc: cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG Subject: Re: cvs commit: src/sys/cam/scsi scsi_cd.c scsi_cd.h src/sys/dev/ata atapi-cd.c src/sys/sys cdrio.h src/usr.sbin/burncd burncd.8 burncd.c src/usr.sbin/cdcontrol cdcontrol.1 cdcontrol.c Message-ID: <Pine.BSF.4.21.0211192108150.62331-100000@root.org> In-Reply-To: <20021120022910.GA7969@rot13.obsecurity.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, 19 Nov 2002, Kris Kennaway wrote: > On Tue, Nov 19, 2002 at 06:18:59PM -0800, Nate Lawson wrote: > > > See the HEADSUP on stable@ and ports@. Only people who were using the > > FreeBSD-only ioctls need to change, simply by multiplying by 177 (the > > kernel was doing this conversion before). API is preserved although units > > have changed. > > Also, what will happen when someone uses an old 4.x binary that calls > these ioctls on 5.0? > > Kris I will commit a one-line addition that if the value is less than 177, scale it appropriately. This change won't hurt the new API and maintaining ABI is nice. However, I'm a little dismayed at the review process here. The change has been in -current for a month with a message saying it would be MFCd in a month. Not one of the negative comments on the MFC has indicated a single application outside of our tree that uses our _FreeBSD_only_ ioctl API. Neither of the two maintainers who reviewed the patch had a problem with it. Why didn't you express your concerns earlier? -Nate 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?Pine.BSF.4.21.0211192108150.62331-100000>