cvs commit: src/sys/alpha/alpha mem.c promcons.c src/sys/alpha/tlsb zs_tlsb.c src/sys/amd64/amd64 mem.c src/sys/cam cam_xpt.c src/sys/cam/scsi scsi_ch.c scsi_pass.c scsi_pt.c s

[M. Warner Losh]

In message: <403A22BB.6010409 at freebsd.org>
            Scott Long <scottl at FreeBSD.org> writes:
: I agree with these arguments.  However, I would have liked to have seen
: a small grace period defined, especially since people are now bumping
: into the consequences of this change and not knowing why.  I plan to do
: this with the interrupt API change when it comes.

Especially since the code to do this is on the order of:

	if (devsw->d_version == 0) {
		printf("Warning, older interface ...");
		devsw->d_flags ^= D_NEEDGIANT;
		devsw->d_version = D_VERSION_00;
	}

unless there are other issues not elucidated (I'm only looking at the
new code).  One could also have had two different bits so APIly it
changed, but you could whine if only one of them was set.

Warner