cvs commit: src/sys/pci if_vr.c

[Coleman Kane]

On 4/1/07, Dag-Erling Smørgrav <des at des.no> wrote:
>
> Colin Percival <cperciva at freebsd.org> writes:
> > Judging by what I see at http://www.freebsd.org/cgi/cvsweb.cgi/src/sys/,
> > it looks like everything has been reverted now, so hopefully Kris can go
> > ahead and redo his original commit later today without any of the
> changes
> > which slipped in by mistake.
>
> Great!  I have a megapatch lined up as well.  In the spirit of
> sustainable kernel development, atomic operations are being renamed to
> renewable operations.
>
> DES
> --
> Dag-Erling Smørgrav - des at des.no
> _______________________________________________
> cvs-src at freebsd.org mailing list
> http://lists.freebsd.org/mailman/listinfo/cvs-src
> To unsubscribe, send any mail to "cvs-src-unsubscribe at freebsd.org"
>

I think that I may just have to start a thread to protest your renewable
operations patch. The naming conventions used in the new macros
unnecessarily stick out under review of the source code where they are
implemented, thus overshadowing and blocking out my abilities to inspect
less prominent features in the code where they are used.

I think that these aesthetic concerns in this situation override any
possible gains through reduction in operation waste, and the changes should
be reconsidered under these grounds. In addition, I (and others) fear that
these renewable operations will render the development of devel/liboil
obsolete, and its developers unemployed.

--
Coleman Kane