From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-lj1-f196.google.com ([209.85.208.196]:45890 "EHLO mail-lj1-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726820AbfJQAQS (ORCPT ); Wed, 16 Oct 2019 20:16:18 -0400 Received: by mail-lj1-f196.google.com with SMTP id q64so536544ljb.12 for ; Wed, 16 Oct 2019 17:16:17 -0700 (PDT) Date: Thu, 17 Oct 2019 03:16:13 +0300 From: "Kirill A. Shutemov" Subject: Re: [patch 014/102] llist: introduce llist_entry_safe() Message-ID: <20191017001613.watsu7vhqujufjxv@box.shutemov.name> References: <57fd50dd./gNBvRBYvu+kYV+l%akpm@linux-foundation.org> <20191016231116.inv5stimz6fg7gof@box.shutemov.name> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: linux-kbuild-owner@vger.kernel.org List-ID: To: Linus Torvalds Cc: Andrew Morton , Sasha Levin , Andrew Pinski , Arnd Bergmann , Masahiro Yamada , Michal Marek , mm-commits@vger.kernel.org, Alexander Potapenko , Dmitry Vyukov , Eric Dumazet , Kostya Serebryany , Ingo Molnar , linux-arch , Linux Kbuild mailing list On Wed, Oct 16, 2019 at 04:29:54PM -0700, Linus Torvalds wrote: > On Wed, Oct 16, 2019 at 4:11 PM Kirill A. Shutemov wrote: > > > > Looks like it was fixed soon after the complain: > > > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63567 > > Ahh, so there are gcc versions which essentially do this wrong, and > I'm not seeing it because it was fixed. > > Ho humm. Considering that this was fixed in gcc five years ago, and we > already require gc-4.6, and did that two years ago, maybe we can just > raise the requirement a bit further. > > BUT. > > It's not clear which versions are ok with this. In your next email you said: > > > It would mean bumping GCC version requirements to 4.7. > > which I think would be reasonable, but is it actually ok in 4.7? I think, not. I don't have 4.7 around, but 4.9.3 has the issue if -std=gnu99 is used. > The bugzilla entry says "Target Milestone: 5.0", and I'm not sure how > to check what that "revision=216440" ends up actually meaning. > > I have a git tree of gcc, and in that one 216440 is commit > d303aeafa9b, but that seems to imply it only made it into 5.1: > > [torvalds@i7 gcc]$ git name-rev --tags > d303aeafa9b46e06cd853696acb6345dff51a6b9 > d303aeafa9b46e06cd853696acb6345dff51a6b9 tags/gcc-5_1_0-release~3943 > > so we'd have to jump forward a _lot_. > > That's a bit sad and annoying. I'd be ok with jumping to 4.7, but I'm > not sure we can jump to 5.1. > > Although maybe we should be a _lot_ more aggressive about gcc > versions, I'm on gcc-9.2.1 right now, and gcc-5.1 is from April 22, > 2015. 5.4.1 builds kernel fine for me with allmodconfig (minus retpoline which requires compiler support). Both -std=gnu99 and -std=gnu11. Note that GCC has changed their version scheme. 5.4.1 is bug-fix release of GCC-5. -- Kirill A. Shutemov