lkml.org 
[lkml]   [2007]   [Aug]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    SubjectRe: [PATCH 1/24] make atomic_read() behave consistently on alpha
    Date
    >>>> So, why not use the well-defined alternative?
    >>> Because we don't need to, and it hurts performance.
    >> It hurts performance by implementing 32-bit atomic reads in assembler?
    >
    > No, I misunderstood the question. Implementing 32-bit atomic reads in
    > assembler is redundant, because any sane compiler, *particularly* and
    > optimizing compiler (and we're only in this mess because of optimizing
    > compilers)

    Oh please, don't tell me you don't want an optimising compiler.
    And if you _do_ want one, well you're in this mess because you
    chose C as implementation language and C has some pretty strange
    rules. Trying to use not-all-that-well-defined-and-completely-
    misunderstood features of the language doesn't make things easier;
    trying to use something that isn't even part of the language and
    that your particular compiler originally supported by accident,
    and that isn't yet an officially supported feature, and that on
    top of it all has a track record of problems -- well it makes me
    wonder if you're in this game for fun or what.

    > will give us that automatically without the assembler.

    No, it does *not* give it to you automatically; you have to do
    either the asm() thing, or the not-defined-at-all *(volatile *)&
    thing.

    > Yes, it is legal for a compiler to violate this assumption. It is
    > also legal for us to refuse to maintain compatibility with compilers
    > that suck this badly.

    So that's rm include/linux/compiler-gcc*.h then. Good luck with
    the intel compiler, maybe it works more to your liking.


    Segher

    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2007-08-10 01:05    [W:2.515 / U:0.272 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site