On Sat, Aug 15, 2020 at 2:27 AM Nick Desaulniers wrote: > > On Fri, Aug 14, 2020 at 3:57 PM Nick Desaulniers > wrote: > > > > On Fri, Aug 14, 2020 at 2:19 PM Sedat Dilek wrote: > > > > > > On Fri, Aug 14, 2020 at 7:29 PM Sedat Dilek wrote: > > > > > > > > Thanks for the proposal. > > > > > > > > I have adapted it to fit my patchset against Linux v5.8. > > > > > > > > Both Debian's GCC-10 and a snapshot version of LLVM toolchain > > > > v11.0.0-rc1+ seems to be OK. > > > > > > > > > > Yupp, OK. > > > > > > I was able to boot FreeDOS 1.2 VM in VirtualBox GUI. > > > > Hi Sedat, > > Apologies, but it's not clear to me precisely which patch you tested. > > Can you please confirm whether you tested: > > 1. Arnd's patch that started this thread. > > 2. My proposed diff adding -fno-addrsig to CFLAGS_powernow-k6.o. > > 3. My proposed diff removing __force_order from the kernel. > > > > I'm hoping you were referring to testing 3., but it's not clear to me. > > Ah, sorry, I missed your comment on github: > https://github.com/ClangBuiltLinux/linux/issues/1120#issuecomment-674282107 > > Ok, I will look at more disassembly next week and hopefully have a > patch ready, with your tested by tag. > Sorry for not being precise - I tested with solution (3.). Later I added the diff I used as mentioned in your above comment. See [1]: > In a 2nd run building with a selfmade clang-11 and LLVM "bin"utils is fine, too. I cannot say much to older versions of GCC and/or LLVM/Clang if removing "__force_order" works fine. Another (4.) solution: Sami tried successfully by adding "__weak" declaration with CONFIG_LKDTM=m (see [2]). I am OK if this works, too. Please, see my attachments. - Sedat - [1] https://github.com/ClangBuiltLinux/linux/issues/1120#issuecomment-674340760 [2] https://github.com/ClangBuiltLinux/linux/issues/1120#issuecomment-674182703