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
In-Reply-To: <01000163f00c1dd6-11e49be4-fe96-49cf-bbf6-8d951874faa1-000000@email.amazonses.com>
References: <01000163ef24c875-daa2c00e-611e-4383-99a6-c8c72ef04a14-000000@email.amazonses.com>
 <20180611141742.GP12198@hirez.programming.kicks-ass.net> <20180611143801.GH12235@hirez.programming.kicks-ass.net>
 <20180611153042.GI12235@hirez.programming.kicks-ass.net> <01000163effdcb29-f886b68d-bdb4-44c0-a68e-f749e10280fd-000000@email.amazonses.com>
 <01000163f00c1dd6-11e49be4-fe96-49cf-bbf6-8d951874faa1-000000@email.amazonses.com>
From:   Diego Viola <diego.viola@gmail.com>
Date:   Mon, 11 Jun 2018 16:23:50 -0300
Message-ID: <CA+ToGPHtL0Fd3TgqakqmEtKi7z6jszpoEdJcO+EyD9yh8pGTyQ@mail.gmail.com>
Subject: Re: Regression: x86/tsc: Fix mark_tsc_unstable()
To:     Jeremy Cline <jeremy@jcline.org>
Cc:     Peter Zijlstra <peterz@infradead.org>,
        Thomas Gleixner <tglx@linutronix.de>,
        "Rafael J. Wysocki" <rafael.j.wysocki@intel.com>,
        rui.zhang@intel.com, len.brown@intel.com, stable@vger.kernel.org,
        linux-kernel@vger.kernel.org, regressions@leemhuis.info,
        mmarget@physik.tu-berlin.de
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Mon, Jun 11, 2018 at 3:11 PM, Jeremy Cline <jeremy@jcline.org> wrote:
> On 06/11/2018 01:56 PM, Jeremy Cline wrote:
>> On 06/11/2018 11:30 AM, Peter Zijlstra wrote:
>>> On Mon, Jun 11, 2018 at 04:38:01PM +0200, Peter Zijlstra wrote:
>>>> On Mon, Jun 11, 2018 at 04:17:42PM +0200, Peter Zijlstra wrote:
>>>>> On Mon, Jun 11, 2018 at 01:59:15PM +0000, Jeremy Cline wrote:
>>>>>> A user has bisected the problem to the v4.16 commit 1ab4ca7c59d4
>>>>>> ("x86/tsc: Fix mark_tsc_unstable()"). According to the reporter,
>>>>>> explicitly setting "tsc=" on the kernel command line causes the boot to
>>>>>> always succeed. All the users have Thinkpad T500s or T400s (Core 2 Duos)
>>>>>
>>>>> Weird. So Core2 typically triggers mark_tsc_unstable() in either
>>>>> intel_idle or processor_idle. ISTR testing that when I did the patches.
>>>>>
>>>>> When I make that mark_tsc_unstable() in the idle drivers unconditional
>>>>> and boot my ivb with that, it doesn't want to fail. I've booted the
>>>>> machine 5 consequctive times without issue.
>>>>>
>>>>> Let me try and checkout -stable, maybe something's up with that.
>>>>
>>>> Nope -stable seems to be working as well on the IVB (with modification).
>>>> I just dug up my T500 and that's actually still running the test kernel.
>>>> Let me try and build the -stable kernel for that.
>>>
>>> 4.16.8 works without issue on my T500 with a debian/ubuntu like distro
>>> config.
>>>
>>
>> Adding mmarget (who bisected the problem) to the CC.
>>
>> It might well be something Fedora-specific, then. I just noticed mmarget
>> commented over the weekend noting that they couldn't reproduce the
>> problem without using the initramfs generated during the RPM install of
>> the kernel. mmarget's theory was that it's a race condition that doesn't
>> occur when the initramfs takes long enough to unpack, but I don't know
>> enough about the early boot process *or* how Fedora's generating the
>> initramfs for RPM installs vs "make install" yet to know how likely that
>> is. I'm going to have to do some research.
>>
>> Thanks for looking into this so quickly and also sorry if this turns out
>> to be a Fedora problem :(
>
> Attached is the Fedora configuration for 4.16.8, as well, in case you'd
> like to test it with that.
>
> Thanks,
> Jeremy

Hi Jeremy,

I've compiled 4.16.8 with your config and booted my machine about 10
times with this kernel, and I'm unable to reproduce the issue.

Maybe it's an issue with the Fedora initramfs?

Diego