Re: Reorganization of spinlock defines

Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> Prompted by confusion over Itanium/Opterion, I have written a patch to
> improve the way we define spinlocks for platforms and cpu's.

The main.c part of the patch strikes me as irrelevant to the claimed
purpose and unlikely to accomplish anything except breaking things.
Do you have a system the main.c "__alpha" code is relevant to, on which
to test that you did not break it?

Other than that, it looks like basically a good idea. But...

> I plan to apply this to 7.4.

This seems like living dangerously. You do realize that this patch will
invalidate our trust that the code works on every single supported
platform? I think beta3 may be a bit late in the game for this sort of
thing, because we've already gotten a good bit of the testing we can
expect to get for lesser-used platforms during this beta cycle.

At the very least I'd like to see the decision discussed on -hackers
and not buried in -patches.

regards, tom lane