From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-lj1-f196.google.com ([209.85.208.196]:44955 "EHLO mail-lj1-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2391320AbfJPXaQ (ORCPT ); Wed, 16 Oct 2019 19:30:16 -0400 Received: by mail-lj1-f196.google.com with SMTP id m13so465414ljj.11 for ; Wed, 16 Oct 2019 16:30:15 -0700 (PDT) Received: from mail-lf1-f45.google.com (mail-lf1-f45.google.com. [209.85.167.45]) by smtp.gmail.com with ESMTPSA id x14sm115278lfe.3.2019.10.16.16.30.11 for (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 16 Oct 2019 16:30:11 -0700 (PDT) Received: by mail-lf1-f45.google.com with SMTP id w67so308593lff.4 for ; Wed, 16 Oct 2019 16:30:11 -0700 (PDT) MIME-Version: 1.0 References: <57fd50dd./gNBvRBYvu+kYV+l%akpm@linux-foundation.org> <20191016231116.inv5stimz6fg7gof@box.shutemov.name> In-Reply-To: <20191016231116.inv5stimz6fg7gof@box.shutemov.name> From: Linus Torvalds Date: Wed, 16 Oct 2019 16:29:54 -0700 Message-ID: Subject: Re: [patch 014/102] llist: introduce llist_entry_safe() Content-Type: text/plain; charset="UTF-8" Sender: linux-kbuild-owner@vger.kernel.org List-ID: To: "Kirill A. Shutemov" 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 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? 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. Linus