Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
References: <20180803170550.164688-1-ndesaulniers@google.com> <tip-208cbb32558907f68b3b2a081ca2337ac3744794@git.kernel.org>
In-Reply-To: <tip-208cbb32558907f68b3b2a081ca2337ac3744794@git.kernel.org>
From:   Nick Desaulniers <ndesaulniers@google.com>
Date:   Mon, 6 Aug 2018 14:33:41 -0700
Message-ID: <CAKwvOdmxaE7p-fgaaeCE17bbTf0iYmHQgWT2D6FM7U5fwUoN1Q@mail.gmail.com>
Subject: Re: [tip:x86/urgent] x86/irqflags: Provide a declaration for native_save_fl
To:     hpa@zytor.com, Thomas Gleixner <tglx@linutronix.de>,
        david.laight@aculab.com, LKML <linux-kernel@vger.kernel.org>,
        jdelvare@suse.de, Ingo Molnar <mingo@kernel.org>
Cc:     linux-tip-commits@vger.kernel.org, sedat.dilek@gmail.com
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Sun, Aug 5, 2018 at 1:33 PM tip-bot for Nick Desaulniers
<tipbot@zytor.com> wrote:
>
> Commit-ID:  208cbb32558907f68b3b2a081ca2337ac3744794
> Gitweb:     https://git.kernel.org/tip/208cbb32558907f68b3b2a081ca2337ac3744794
> Author:     Nick Desaulniers <ndesaulniers@google.com>
> AuthorDate: Fri, 3 Aug 2018 10:05:50 -0700
> Committer:  Thomas Gleixner <tglx@linutronix.de>
> CommitDate: Sun, 5 Aug 2018 22:30:37 +0200
>
> x86/irqflags: Provide a declaration for native_save_fl
>
> It was reported that the commit d0a8d9378d16 is causing users of gcc < 4.9
> to observe -Werror=missing-prototypes errors.
>
> Indeed, it seems that:
> extern inline unsigned long native_save_fl(void) { return 0; }
>
> compiled with -Werror=missing-prototypes produces this warning in gcc <
> 4.9, but not gcc >= 4.9.
>
> Fixes: d0a8d9378d16 ("x86/paravirt: Make native_save_fl() extern inline").
> Reported-by: David Laight <david.laight@aculab.com>
> Reported-by: Jean Delvare <jdelvare@suse.de>
> Signed-off-by: Nick Desaulniers <ndesaulniers@google.com>
> Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
> Cc: hpa@zytor.com
> Cc: jgross@suse.com
> Cc: kstewart@linuxfoundation.org
> Cc: gregkh@linuxfoundation.org
> Cc: boris.ostrovsky@oracle.com
> Cc: astrachan@google.com
> Cc: mka@chromium.org
> Cc: arnd@arndb.de
> Cc: tstellar@redhat.com
> Cc: sedat.dilek@gmail.com
> Cc: David.Laight@aculab.com
> Cc: stable@vger.kernel.org

Not sure if this was going to be cleaned up in an automated way, but
looks like this commit message drops the comment to stable as to how
far back it should go:

# 4.17, 4.14, 4.9, 4.4

also, there were tested by's reported:

Tested-by: David Laight <david.laight@acuab.com>
Tested-by: Sedat Dilek <sedat.dilek@gmail.com>

--
Thanks,
~Nick Desaulniers