All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christophe Leroy <christophe.leroy@csgroup.eu>
To: Masahiro Yamada <masahiroy@kernel.org>,
	linux-kernel@vger.kernel.org,
	Linus Torvalds <torvalds@linux-foundation.org>
Cc: Albert Ou <aou@eecs.berkeley.edu>, Arnd Bergmann <arnd@arndb.de>,
	linux-kbuild@vger.kernel.org,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Jonathan Corbet <corbet@lwn.net>,
	linux-doc@vger.kernel.org, linuxppc-dev@lists.ozlabs.org,
	Palmer Dabbelt <palmer@dabbelt.com>,
	Paul Walmsley <paul.walmsley@sifive.com>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Miguel Ojeda <ojeda@kernel.org>,
	Paul Mackerras <paulus@samba.org>,
	linux-riscv@lists.infradead.org, Will Deacon <will@kernel.org>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH] Raise the minimum GCC version to 5.2
Date: Mon, 3 May 2021 08:16:58 +0200	[thread overview]
Message-ID: <8b5f1d57-1357-affd-565f-f4826f3ecbdf@csgroup.eu> (raw)
In-Reply-To: <20210501151538.145449-1-masahiroy@kernel.org>



Le 01/05/2021 à 17:15, Masahiro Yamada a écrit :
> The current minimum GCC version is 4.9 except ARCH=arm64 requiring
> GCC 5.1.
> 
> When we discussed last time, we agreed to raise the minimum GCC version
> to 5.1 globally. [1]
> 
> I'd like to propose GCC 5.2 to clean up arch/powerpc/Kconfig as well.

One point I missed when I saw your patch first time, but I realised during the discussion:

Up to 4.9, GCC was numbered with 3 digits, we had 4.8.0, 4.8.1, ... 4.8.5, 4.9.0, 4.9.1, .... 4.9.4

Then starting at 5, GCC switched to a 2 digits scheme, with 5.0, 5.1, 5.2, ... 5.5

So, that is not GCC 5.1 or 5.2 that you should target, but only GCC 5.
Then it is up to the user to use the latest available version of GCC 5, which is 5.5 at the time 
begin, just like the user would have selected 4.9.4 when 4.9 was the minimum GCC version.

Christophe

WARNING: multiple messages have this Message-ID (diff)
From: Christophe Leroy <christophe.leroy@csgroup.eu>
To: Masahiro Yamada <masahiroy@kernel.org>,
	linux-kernel@vger.kernel.org,
	Linus Torvalds <torvalds@linux-foundation.org>
Cc: Albert Ou <aou@eecs.berkeley.edu>, Arnd Bergmann <arnd@arndb.de>,
	linux-kbuild@vger.kernel.org,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Jonathan Corbet <corbet@lwn.net>,
	linux-doc@vger.kernel.org, linuxppc-dev@lists.ozlabs.org,
	Palmer Dabbelt <palmer@dabbelt.com>,
	Paul Walmsley <paul.walmsley@sifive.com>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Miguel Ojeda <ojeda@kernel.org>,
	Paul Mackerras <paulus@samba.org>,
	linux-riscv@lists.infradead.org, Will Deacon <will@kernel.org>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH] Raise the minimum GCC version to 5.2
Date: Mon, 3 May 2021 08:16:58 +0200	[thread overview]
Message-ID: <8b5f1d57-1357-affd-565f-f4826f3ecbdf@csgroup.eu> (raw)
In-Reply-To: <20210501151538.145449-1-masahiroy@kernel.org>



Le 01/05/2021 à 17:15, Masahiro Yamada a écrit :
> The current minimum GCC version is 4.9 except ARCH=arm64 requiring
> GCC 5.1.
> 
> When we discussed last time, we agreed to raise the minimum GCC version
> to 5.1 globally. [1]
> 
> I'd like to propose GCC 5.2 to clean up arch/powerpc/Kconfig as well.

One point I missed when I saw your patch first time, but I realised during the discussion:

Up to 4.9, GCC was numbered with 3 digits, we had 4.8.0, 4.8.1, ... 4.8.5, 4.9.0, 4.9.1, .... 4.9.4

Then starting at 5, GCC switched to a 2 digits scheme, with 5.0, 5.1, 5.2, ... 5.5

So, that is not GCC 5.1 or 5.2 that you should target, but only GCC 5.
Then it is up to the user to use the latest available version of GCC 5, which is 5.5 at the time 
begin, just like the user would have selected 4.9.4 when 4.9 was the minimum GCC version.

Christophe

_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

WARNING: multiple messages have this Message-ID (diff)
From: Christophe Leroy <christophe.leroy@csgroup.eu>
To: Masahiro Yamada <masahiroy@kernel.org>,
	linux-kernel@vger.kernel.org,
	Linus Torvalds <torvalds@linux-foundation.org>
Cc: Albert Ou <aou@eecs.berkeley.edu>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Arnd Bergmann <arnd@arndb.de>, Jonathan Corbet <corbet@lwn.net>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	linux-doc@vger.kernel.org, Will Deacon <will@kernel.org>,
	Palmer Dabbelt <palmer@dabbelt.com>,
	linux-arm-kernel@lists.infradead.org,
	Paul Walmsley <paul.walmsley@sifive.com>,
	Miguel Ojeda <ojeda@kernel.org>,
	Paul Mackerras <paulus@samba.org>,
	linux-riscv@lists.infradead.org, linuxppc-dev@lists.ozlabs.org,
	linux-kbuild@vger.kernel.org
Subject: Re: [PATCH] Raise the minimum GCC version to 5.2
Date: Mon, 3 May 2021 08:16:58 +0200	[thread overview]
Message-ID: <8b5f1d57-1357-affd-565f-f4826f3ecbdf@csgroup.eu> (raw)
In-Reply-To: <20210501151538.145449-1-masahiroy@kernel.org>



Le 01/05/2021 à 17:15, Masahiro Yamada a écrit :
> The current minimum GCC version is 4.9 except ARCH=arm64 requiring
> GCC 5.1.
> 
> When we discussed last time, we agreed to raise the minimum GCC version
> to 5.1 globally. [1]
> 
> I'd like to propose GCC 5.2 to clean up arch/powerpc/Kconfig as well.

One point I missed when I saw your patch first time, but I realised during the discussion:

Up to 4.9, GCC was numbered with 3 digits, we had 4.8.0, 4.8.1, ... 4.8.5, 4.9.0, 4.9.1, .... 4.9.4

Then starting at 5, GCC switched to a 2 digits scheme, with 5.0, 5.1, 5.2, ... 5.5

So, that is not GCC 5.1 or 5.2 that you should target, but only GCC 5.
Then it is up to the user to use the latest available version of GCC 5, which is 5.5 at the time 
begin, just like the user would have selected 4.9.4 when 4.9 was the minimum GCC version.

Christophe

WARNING: multiple messages have this Message-ID (diff)
From: Christophe Leroy <christophe.leroy@csgroup.eu>
To: Masahiro Yamada <masahiroy@kernel.org>,
	linux-kernel@vger.kernel.org,
	Linus Torvalds <torvalds@linux-foundation.org>
Cc: Albert Ou <aou@eecs.berkeley.edu>, Arnd Bergmann <arnd@arndb.de>,
	linux-kbuild@vger.kernel.org,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Jonathan Corbet <corbet@lwn.net>,
	linux-doc@vger.kernel.org, linuxppc-dev@lists.ozlabs.org,
	Palmer Dabbelt <palmer@dabbelt.com>,
	Paul Walmsley <paul.walmsley@sifive.com>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Miguel Ojeda <ojeda@kernel.org>,
	Paul Mackerras <paulus@samba.org>,
	linux-riscv@lists.infradead.org, Will Deacon <will@kernel.org>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH] Raise the minimum GCC version to 5.2
Date: Mon, 3 May 2021 08:16:58 +0200	[thread overview]
Message-ID: <8b5f1d57-1357-affd-565f-f4826f3ecbdf@csgroup.eu> (raw)
In-Reply-To: <20210501151538.145449-1-masahiroy@kernel.org>



Le 01/05/2021 à 17:15, Masahiro Yamada a écrit :
> The current minimum GCC version is 4.9 except ARCH=arm64 requiring
> GCC 5.1.
> 
> When we discussed last time, we agreed to raise the minimum GCC version
> to 5.1 globally. [1]
> 
> I'd like to propose GCC 5.2 to clean up arch/powerpc/Kconfig as well.

One point I missed when I saw your patch first time, but I realised during the discussion:

Up to 4.9, GCC was numbered with 3 digits, we had 4.8.0, 4.8.1, ... 4.8.5, 4.9.0, 4.9.1, .... 4.9.4

Then starting at 5, GCC switched to a 2 digits scheme, with 5.0, 5.1, 5.2, ... 5.5

So, that is not GCC 5.1 or 5.2 that you should target, but only GCC 5.
Then it is up to the user to use the latest available version of GCC 5, which is 5.5 at the time 
begin, just like the user would have selected 4.9.4 when 4.9 was the minimum GCC version.

Christophe

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  parent reply	other threads:[~2021-05-03  6:17 UTC|newest]

Thread overview: 146+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-05-01 15:15 [PATCH] Raise the minimum GCC version to 5.2 Masahiro Yamada
2021-05-01 15:15 ` Masahiro Yamada
2021-05-01 15:15 ` Masahiro Yamada
2021-05-01 15:15 ` Masahiro Yamada
2021-05-01 15:52 ` Miguel Ojeda
2021-05-01 15:52   ` Miguel Ojeda
2021-05-01 15:52   ` Miguel Ojeda
2021-05-01 15:52   ` Miguel Ojeda
2021-05-01 17:38   ` Christophe Leroy
2021-05-01 17:38     ` Christophe Leroy
2021-05-01 17:38     ` Christophe Leroy
2021-05-01 17:38     ` Christophe Leroy
2021-05-02  2:41   ` Joe Perches
2021-05-02  2:41     ` Joe Perches
2021-05-02  2:41     ` Joe Perches
2021-05-02  2:41     ` Joe Perches
2021-05-02 18:30     ` Segher Boessenkool
2021-05-02 18:30       ` Segher Boessenkool
2021-05-02 18:30       ` Segher Boessenkool
2021-05-02 18:30       ` Segher Boessenkool
2021-05-02 20:00       ` Joe Perches
2021-05-02 20:00         ` Joe Perches
2021-05-02 20:00         ` Joe Perches
2021-05-02 20:00         ` Joe Perches
2021-05-02 20:32         ` Segher Boessenkool
2021-05-02 20:32           ` Segher Boessenkool
2021-05-02 20:32           ` Segher Boessenkool
2021-05-02 20:32           ` Segher Boessenkool
2021-05-02 21:05           ` Ali Kaasinen
2021-05-02 21:05             ` Ali Kaasinen
2021-05-02 21:05             ` Ali Kaasinen
2021-05-02 21:05             ` Ali Kaasinen
2021-05-02 21:08           ` Linus Torvalds
2021-05-02 21:08             ` Linus Torvalds
2021-05-02 21:08             ` Linus Torvalds
2021-05-02 21:08             ` Linus Torvalds
2021-05-02 22:30             ` Matthew Wilcox
2021-05-02 22:30               ` Matthew Wilcox
2021-05-02 22:30               ` Matthew Wilcox
2021-05-02 22:30               ` Matthew Wilcox
2021-05-03  7:34               ` Alexander Dahl
2021-05-03  7:34                 ` Alexander Dahl
2021-05-03  7:34                 ` Alexander Dahl
2021-05-03  7:34                 ` Alexander Dahl
2021-05-03  8:54                 ` Joe Perches
2021-05-03  8:54                   ` Joe Perches
2021-05-03  8:54                   ` Joe Perches
2021-05-03  8:54                   ` Joe Perches
2021-05-03  9:25                 ` Arnd Bergmann
2021-05-03  9:25                   ` Arnd Bergmann
2021-05-03  9:25                   ` Arnd Bergmann
2021-05-03  9:25                   ` Arnd Bergmann
2021-05-03 12:20                   ` David Laight
2021-05-03 12:20                     ` David Laight
2021-05-03 12:20                     ` David Laight
2021-05-03 13:10                     ` Miguel Ojeda
2021-05-03 13:10                       ` Miguel Ojeda
2021-05-03 13:10                       ` Miguel Ojeda
2021-05-03 13:10                       ` Miguel Ojeda
2021-05-04  5:30                   ` Alexander Dahl
2021-05-04  5:30                     ` Alexander Dahl
2021-05-04  5:30                     ` Alexander Dahl
2021-05-04  5:30                     ` Alexander Dahl
2021-05-04  6:33                     ` Christophe Leroy
2021-05-04  6:33                       ` Christophe Leroy
2021-05-04  6:33                       ` Christophe Leroy
2021-05-04 12:07                     ` Arnd Bergmann
2021-05-04 12:07                       ` Arnd Bergmann
2021-05-04 12:07                       ` Arnd Bergmann
2021-05-04 12:07                       ` Arnd Bergmann
2021-05-04 13:25                   ` Andy Shevchenko
2021-05-04 13:25                     ` Andy Shevchenko
2021-05-04 13:25                     ` Andy Shevchenko
2021-05-04 13:25                     ` Andy Shevchenko
2021-05-03 11:35               ` Arnd Bergmann
2021-05-03 11:35                 ` Arnd Bergmann
2021-05-03 11:35                 ` Arnd Bergmann
2021-05-03 11:35                 ` Arnd Bergmann
2021-05-03  9:49             ` Kirill A. Shutemov
2021-05-03  9:49               ` Kirill A. Shutemov
2021-05-03  9:49               ` Kirill A. Shutemov
2021-05-03  9:49               ` Kirill A. Shutemov
2021-05-02 21:23           ` Joe Perches
2021-05-02 21:23             ` Joe Perches
2021-05-02 21:23             ` Joe Perches
2021-05-02 21:23             ` Joe Perches
2021-05-03  0:44             ` Segher Boessenkool
2021-05-03  0:44               ` Segher Boessenkool
2021-05-03  0:44               ` Segher Boessenkool
2021-05-03  0:44               ` Segher Boessenkool
2021-05-03  9:57               ` Arnd Bergmann
2021-05-03  9:57                 ` Arnd Bergmann
2021-05-03  9:57                 ` Arnd Bergmann
2021-05-03  9:57                 ` Arnd Bergmann
2021-05-04  7:56     ` Ben Dooks
2021-05-04  7:56       ` Ben Dooks
2021-05-04  7:56       ` Ben Dooks
2021-05-04  7:56       ` Ben Dooks
2021-05-04  8:38       ` Miguel Ojeda
2021-05-04  8:38         ` Miguel Ojeda
2021-05-04  8:38         ` Miguel Ojeda
2021-05-04  8:38         ` Miguel Ojeda
2021-05-04  9:22         ` Michal Suchánek
2021-05-04  9:22           ` Michal Suchánek
2021-05-04  9:22           ` Michal Suchánek
2021-05-04  9:22           ` Michal Suchánek
2021-05-04 12:09           ` Miguel Ojeda
2021-05-04 12:09             ` Miguel Ojeda
2021-05-04 12:09             ` Miguel Ojeda
2021-05-04 12:09             ` Miguel Ojeda
2021-05-04 12:17             ` Michal Suchánek
2021-05-04 12:17               ` Michal Suchánek
2021-05-04 12:17               ` Michal Suchánek
2021-05-04 12:17               ` Michal Suchánek
2021-05-04 12:26               ` Christophe Leroy
2021-05-04 12:26                 ` Christophe Leroy
2021-05-04 12:26                 ` Christophe Leroy
2021-05-04 12:26                 ` Christophe Leroy
2021-05-02 18:37 ` Segher Boessenkool
2021-05-02 18:37   ` Segher Boessenkool
2021-05-02 18:37   ` Segher Boessenkool
2021-05-02 18:37   ` Segher Boessenkool
2021-05-03  6:16 ` Christophe Leroy [this message]
2021-05-03  6:16   ` Christophe Leroy
2021-05-03  6:16   ` Christophe Leroy
2021-05-03  6:16   ` Christophe Leroy
2021-05-04  2:13   ` Masahiro Yamada
2021-05-04  2:13     ` Masahiro Yamada
2021-05-04  2:13     ` Masahiro Yamada
2021-05-04  2:13     ` Masahiro Yamada
2021-05-04 13:18   ` Andy Shevchenko
2021-05-04 13:18     ` Andy Shevchenko
2021-05-04 13:18     ` Andy Shevchenko
2021-05-04 13:18     ` Andy Shevchenko
2021-05-03 12:27 ` David Sterba
2021-05-03 12:27   ` David Sterba
2021-05-03 12:27   ` David Sterba
2021-05-03 12:27   ` David Sterba
2021-05-15  7:14 ` Pavel Machek
2021-05-15  7:14   ` Pavel Machek
2021-05-15  7:14   ` Pavel Machek
2021-05-15  7:14   ` Pavel Machek
2021-05-15  7:27   ` Joe Perches
2021-05-15  7:27     ` Joe Perches
2021-05-15  7:27     ` Joe Perches
2021-05-15  7:27     ` Joe Perches

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=8b5f1d57-1357-affd-565f-f4826f3ecbdf@csgroup.eu \
    --to=christophe.leroy@csgroup.eu \
    --cc=aou@eecs.berkeley.edu \
    --cc=arnd@arndb.de \
    --cc=catalin.marinas@arm.com \
    --cc=corbet@lwn.net \
    --cc=gregkh@linuxfoundation.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kbuild@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-riscv@lists.infradead.org \
    --cc=linuxppc-dev@lists.ozlabs.org \
    --cc=masahiroy@kernel.org \
    --cc=ojeda@kernel.org \
    --cc=palmer@dabbelt.com \
    --cc=paul.walmsley@sifive.com \
    --cc=paulus@samba.org \
    --cc=torvalds@linux-foundation.org \
    --cc=will@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.