Date: Thu, 7 Mar 2002 22:28:02 -0500 From: Garance A Drosihn <drosih@rpi.edu> To: Matthew Dillon <dillon@apollo.backplane.com>, Jake Burkholder <jake@locore.ca> Cc: Robert Watson <rwatson@FreeBSD.ORG>, FreeBSD current users <current@FreeBSD.ORG> Subject: Re: Patch for critical_enter()/critical_exit() & interrupt assem Message-ID: <p0510151eb8adde06e3ac@[128.113.24.47]> In-Reply-To: <200203080304.g2834n571759@apollo.backplane.com> References: <200203072143.g27LhaL97112@harmony.village.org> <Pine.NEB.3.96L.1020307171254.23264D-100000@fledge.watson.org> <20020307205844.C12044@locore.ca> <200203080304.g2834n571759@apollo.backplane.com>
next in thread | previous in thread | raw e-mail | index | archive | help
At 7:04 PM -0800 3/7/02, Matthew Dillon wrote: >:Bruce also had some comments which were shrugged off, I thought they >:were important. Specifically, please do not make unnecessary changes >:to the assembler code. Macros do not need to be defined before they >:are used, I believe this was the justification for some of the >:reordering in apic_vector.s which makes the patch confusing. Please >:do not tab out the "; \" at the end of the lines in the INTR and >:FAST_INTR macros in icu_vector.s. This just makes unnecessary diffs. >: >:Jake > > Actually all I did there was square up icu_vector.s so it looked > almost the same as apic_vector.s. I would consider that an > improvement. Perhaps that part of the update could be considered "cosmetic", and thus be done as a separate update -- just so people can tell which lines are cosmetic changes and which ones are substantive. That is more work for you, but it makes it easier on reviewers, and it is certainly consistent with what developers are asked to do on other updates they make. You'll still probably have a debate on the "wonderfulness" of that cosmetic change, but at least it makes it easier for the major changes to looked at and commented on separately. -- Garance Alistair Drosehn = gad@eclipse.acs.rpi.edu Senior Systems Programmer or gad@freebsd.org Rensselaer Polytechnic Institute or drosih@rpi.edu To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?p0510151eb8adde06e3ac>