On Sat, Aug 22, 2020 at 11:51:56AM +0200, Sedat Dilek wrote: > On Sat, Aug 22, 2020 at 11:23 AM Sedat Dilek <sedat.di...@gmail.com> wrote: > > > > On Sat, Aug 22, 2020 at 10:42 AM Segher Boessenkool > > <seg...@kernel.crashing.org> wrote: > > > > > > Hi Arvind, > > > > > > On Fri, Aug 21, 2020 at 11:55:52PM -0400, Arvind Sankar wrote: > > > > Cc Segher. > > > > > > > > Segher, we were looking at gcc PR82602, where IRA could reorder volatile > > > > asm's (reported on ARM). The fix was backported to gcc-6. > > > > > > I know ;-) > > > > > > > Do you know if > > > > there is any reason the problem couldn't occur on x86 on older gcc > > > > without the fix? > > > > > > No, I see no particular reason, at least GCC 5 seems vulnerable. (The > > > GCC 5 release branch was closed at the time this bug report was made, > > > already). There is no reason I see why it would work on x86 but fail > > > elsewhere, either.
> There exist gcc-4.8 and gcc-4.9 for Debian/jessie where EOL was June > 30, 2020 (see [1] and [2]). > > In the latest available version "4.9.2-10+deb8u1" I see no PR82602 was > backported (see [3] and [4]). [ There is GCC 4.9.4, no one should use an older 4.9. ] I mentioned 5 for a reason: the whole function this patch is to did not exist before then! That does not mean the bug existed or did not exist before GCC 5, but it does for example mean that a backport to 4.9 or older isn't trivial at all. > I am asking myself who is using such ancient compilers? Some distros have a GCC 4.8 as system compiler. We allow building GCC itself with a compiler that far back, for various reasons as well (and this is very sharp already, the last mainline GCC 4.8 release is from June 2015, not all that long ago at all). But, one reason this works is because people actually test it. Does anyone actually test the kernel with old compilers? It isn't hard to build a new compiler (because we make sure building a newer compiler works with older compilers, etc. :-) ), and as you say, most distros have newer compilers available nowadays. Segher
