Am 10.11.2014 um 22:07 schrieb Linus Torvalds: > On Mon, Nov 10, 2014 at 12:18 PM, Christian Borntraeger > <borntraeger@xxxxxxxxxx> wrote: >> >> Now: I can reproduces belows miscompile on gcc46 and gcc 47 >> gcc 45 seems ok, gcc 48 is fixed. This makes blacklisting >> a bit hard, especially since it is not limited to s390, but >> covers all architectures. >> In essence ACCESS_ONCE will not work reliably on aggregate >> types with gcc 4.6 and gcc 4.7. >> In Linux we seem to use ACCESS_ONCE mostly on scalar types, >> below code is an example were we dont - and break. > > Hmm. I think we should see how painful it would be to make it a rule > that ACCESS_ONCE() only works on scalar types. > > Even in the actual code you show as an example, the "fix" is really to > use the "unsigned long val" member of the union for the ACCESS_ONCE(). > And that seems to be true in many other cases too. Yes, using the val like in - new = old = ACCESS_ONCE(*ic); + new.val = old.val = ACCESS_ONCE(ic->val); does solve the problem as well. In fact, gcc does create the same binary code on my 4.7.2. Are you ok with the patch as is in kvm/next for the time being or shall we revert that and replace it with the .val scheme? We can also do the cleanup later on if we manage to get your initial patch into a shape that works out. Christian -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
