Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: edfiS9fqPExSrjyQRJZYd5TDT8EAXcDp4PGIyCrQ3IqhzcOAZf+gj+zgbVultwgTiZsFh0e9OQ
 trwkuVldta5g==
IronPort-SDR: /HLTSyqcCtlfvHUFj3U7R4dejlUUolV2MFsbJsGIJRfXxhBwKpNZCeRz9tzola9ulGY624IieT
 Blo7cZgL6B6Q==
Date:   Sun, 25 Apr 2021 08:56:17 +0800
From:   Oliver Sang <oliver.sang@intel.com>
To:     Lyude Paul <lyude@redhat.com>
Cc:     "Sarvela, Tomi P" <tomi.p.sarvela@intel.com>,
        Jani Nikula <jani.nikula@intel.com>,
        Vasily Khoruzhick <anarsoul@gmail.com>,
        LKML <linux-kernel@vger.kernel.org>,
        Linux Memory Management List <linux-mm@kvack.org>,
        lkp@lists.01.org, lkp@intel.com, rui.zhang@intel.com,
        yu.c.chen@intel.com
Subject: Re: [drm/i915/dp]  4a8d79901d:
 WARNING:at_drivers/gpu/drm/i915/display/intel_display_power.c:#assert_can_disable_lcpll[i915]
Message-ID: <20210425005610.GA5251@xsang-OptiPlex-9020>
References: <20210204090229.GH17757@xsang-OptiPlex-9020>
 <b94a605ba4a44e7b45f1ce3d6071dcc7a449e7ac.camel@redhat.com>
 <20210318021320.GA10304@xsang-OptiPlex-9020>
 <90e58cf788299c76b80ec72735ebfd33eef7fffd.camel@redhat.com>
 <e0d2addfb1042baf9cc34b35b1a563656147353f.camel@redhat.com>
 <20210325135342.GA17080@xsang-OptiPlex-9020>
 <3f97d13aab87ca8023f6607b419e3c0f3249871a.camel@redhat.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <3f97d13aab87ca8023f6607b419e3c0f3249871a.camel@redhat.com>
User-Agent: Mutt/1.9.4 (2018-02-28)
Precedence: bulk

hi, Lyude Paul,

On Fri, Apr 09, 2021 at 04:30:55PM -0400, Lyude Paul wrote:
> Could you retest this against the latest drm-tip? I pushed something that I
> think might fix this

sorry for late.
we tested on the tip for drm-tip several days ago, confirmed the issue was gone.

the test commit is:
commit 0d733c1e940857595e3f647dd59bf98c126806c4
Author:     Ville Syrj<C3><A4>l<C3><A4> <ville.syrjala@linux.intel.com>
AuthorDate: Wed Apr 21 02:02:02 2021 +0300
Commit:     Ville Syrj<C3><A4>l<C3><A4> <ville.syrjala@linux.intel.com>
CommitDate: Wed Apr 21 02:02:02 2021 +0300

    drm-tip: 2021y-04m-20d-23h-01m-26s UTC integration manifest

on repo:
git://anongit.freedesktop.org/drm/drm-tip
branch: drm-tip

> 
> On Thu, 2021-03-25 at 21:53 +0800, Oliver Sang wrote:
> > Hi, Lyude,
> > 
> > On Tue, Mar 23, 2021 at 01:27:29PM -0400, Lyude Paul wrote:
> > > BTW Oliver - I'm still happy to look at this issue, I just need the extra
> > > logs I
> > > mentioned below in order to figure out what's going on here
> > 
> > I enabled "drm.debug=0x16 log_buf_len=5M" and got attached kmsg.xz for
> > 4a8d79901d.
> > hope it's useful.
> > 
> > > 
> > > On Thu, 2021-03-18 at 13:16 -0400, Lyude Paul wrote:
> > > > Well this is surprising, because I had been pretty sure we had asked
> > > > someone
> > > > at
> > > > Intel to test that this patch worked on samus and they had said that it
> > > > did. 
> > > > 
> > > > Is there any way you can get me the kernel messages from this failure with
> > > > drm.debug=0x16 log_buf_len=5M added to the kernel commandline? It's hard
> > > > to
> > > > see
> > > > what's actually going on here that's causing this issue to still reoccur.
> > > > 
> > > > Also adding Tomi P Sarvela onto this thread, since they were the ones who
> > > > originally tested this
> > > > 
> > > > On Thu, 2021-03-18 at 10:13 +0800, Oliver Sang wrote:
> > > > > Hi Lyude, sorry for later.
> > > > > 
> > > > > before we send out the report, we also tested fe7d52bccab6, the issue
> > > > > still
> > > > > exists on it.
> > > > > attached one kmsg-fe7d52bccab6.xz FYI.
> > > > > 
> > > > > we also tested on latest v5.12-rc3, also exists.
> > > > > attached kmsg-v5.12-rc3.xz FYI.
> > > > > 
> > > > > 
> > > > > 
> > > > > On Fri, Feb 05, 2021 at 02:53:11PM -0500, Lyude Paul wrote:
> > > > > > Am I right in assuming this is likely a very delayed test result from
> > > > > > before:
> > > > > > 
> > > > > > fe7d52bccab6 ("drm/i915/dp: Don't use DPCD backlights that need PWM
> > > > > > enable/disable")
> > > > > > 
> > > > > > Made it into the kernel? I see that there's a PWM being left on, which
> > > > > > was
> > > > > > the
> > > > > > same issue we were seeing on fi-bdw-samus
> > > > > > 
> > > > > > On Thu, 2021-02-04 at 17:02 +0800, kernel test robot wrote:
> > > > > > > 
> > > > > > > Greeting,
> > > > > > > 
> > > > > > > FYI, we noticed the following commit (built with gcc-9):
> > > > > > > 
> > > > > > > commit: 4a8d79901d5bed0812d272c372aa40282937b50f ("drm/i915/dp:
> > > > > > > Enable
> > > > > > > Intel's
> > > > > > > HDR backlight interface (only SDR for now)")
> > > > > > > https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git?mas
> > > > > > > ter
> > > > > > > 
> > > > > > > 
> > > > > > > in testcase: suspend-stress
> > > > > > > version: 
> > > > > > > with following parameters:
> > > > > > > 
> > > > > > > ????????mode: freeze
> > > > > > > ????????iterations: 10
> > > > > > > 
> > > > > > > 
> > > > > > > 
> > > > > > > on test machine: 4 threads Broadwell-Y with 8G memory
> > > > > > > 
> > > > > > > caused below changes (please refer to attached dmesg/kmsg for entire
> > > > > > > log/backtrace):
> > > > > > > 
> > > > > > > 
> > > > > > > 
> > > > > > > If you fix the issue, kindly add following tag
> > > > > > > Reported-by: kernel test robot <oliver.sang@intel.com>
> > > > > > > 
> > > > > > > 
> > > > > > > kern? :warn? : [?? 21.137894] ------------[ cut here ]------------
> > > > > > > kern? :warn? : [?? 21.137897] CPU PWM1 enabled
> > > > > > > kern? :warn? : [?? 21.137930] WARNING: CPU: 3 PID: 575 at
> > > > > > > drivers/gpu/drm/i915/display/intel_display_power.c:4902
> > > > > > > assert_can_disable_lcpll+0x335/0x3c0 [i915]
> > > > > > > kern? :warn? : [?? 21.138084] Modules linked in: sd_mod t10_pi sg
> > > > > > > x86_pkg_temp_thermal intel_powerclamp i915 hid_rmi rmi_core
> > > > > > > hid_multitouch
> > > > > > > coretemp intel_ra
> > > > > > > pl_msr crct10dif_pclmul wmi_bmof crc32_pclmul intel_gtt crc32c_intel
> > > > > > > ghash_clmulni_intel rapl drm_kms_helper intel_cstate ahci libahci
> > > > > > > serio_raw
> > > > > > > mei_me proces
> > > > > > > sor_thermal_device intel_uncore processor_thermal_rfim syscopyarea
> > > > > > > sysfillrect
> > > > > > > intel_soc_dts_iosf sysimgblt intel_pch_thermal hid_sensor_custom
> > > > > > > fb_sys_fops
> > > > > > > li
> > > > > > > bata bcma processor_thermal_mbox joydev mei drm
> > > > > > > processor_thermal_rapl
> > > > > > > intel_rapl_common int340x_thermal_zone i2c_hid ideapad_laptop
> > > > > > > sparse_keymap
> > > > > > > wmi rfkill
> > > > > > > video int3400_thermal acpi_thermal_rel i2c_designware_platform
> > > > > > > dw_dmac
> > > > > > > i2c_designware_core acpi_pad ip_tables
> > > > > > > kern? :warn? : [?? 21.138148] CPU: 3 PID: 575 Comm: kworker/u8:6
> > > > > > > Tainted:
> > > > > > > G??????? W I?????? 5.11.0-rc2-00745-g4a8d79901d5b #1
> > > > > > > kern? :warn? : [?? 21.138153] Hardware name: LENOVO 80HE/VIUU4, BIOS
> > > > > > > A6CN38WW
> > > > > > > 09/30/2014
> > > > > > > kern? :warn? : [?? 21.138155] Workqueue: events_unbound
> > > > > > > async_run_entry_fn
> > > > > > > kern? :warn? : [?? 21.138161] RIP:
> > > > > > > 0010:assert_can_disable_lcpll+0x335/0x3c0
> > > > > > > [i915]
> > > > > > > kern? :info? : [?? 21.138263] ahci 0000:00:1f.2:
> > > > > > > pci_pm_suspend_late+0x0/0x40
> > > > > > > returned 0 after 0 usecs
> > > > > > > kern? :warn? : [?? 21.138296] Code: c0 75 22 e8 8d 61 cb ff e9 f3 fd
> > > > > > > ff
> > > > > > > ff
> > > > > > > e8
> > > > > > > fd bc 6a c1 0f 0b e9 0c fd ff ff e8 f1 bc 6a c1 0f 0b e9 05 fe ff ff
> > > > > > > e8
> > > > > > > e5
> > > > > > > bc
> > > > > > > 6a c1 <0f> 0b e9 cf fd ff ff e8 d9 bc 6a c1 0f 0b e9 a2 fd ff ff e8
> > > > > > > cd
> > > > > > > bc
> > > > > > > kern? :warn? : [?? 21.138300] RSP: 0018:ffffc900006f3d58 EFLAGS:
> > > > > > > 00010282
> > > > > > > kern? :warn? : [?? 21.138304] RAX: 0000000000000000 RBX:
> > > > > > > ffff8881451002e8
> > > > > > > RCX:
> > > > > > > 0000000000000027
> > > > > > > kern? :warn? : [?? 21.138307] RDX: 0000000000000027 RSI:
> > > > > > > 0000000000000002
> > > > > > > RDI:
> > > > > > > ffff888249397cf8
> > > > > > > kern? :warn? : [?? 21.138310] RBP: ffff888145100000 R08:
> > > > > > > ffff888249397cf0
> > > > > > > R09:
> > > > > > > ffffc900006f3cf0
> > > > > > > kern? :warn? : [?? 21.138313] R10: 0000000000000001 R11:
> > > > > > > 3030302035313969
> > > > > > > R12:
> > > > > > > ffff8881451007f0
> > > > > > > kern? :warn? : [?? 21.138316] R13: ffff888145106c60 R14:
> > > > > > > 0000000000000002
> > > > > > > R15:
> > > > > > > 0000000000000000
> > > > > > > kern? :warn? : [?? 21.138319] FS:? 0000000000000000(0000)
> > > > > > > GS:ffff888249380000(0000) knlGS:0000000000000000
> > > > > > > kern? :warn? : [?? 21.138322] CS:? 0010 DS: 0000 ES: 0000 CR0:
> > > > > > > 0000000080050033
> > > > > > > kern? :warn? : [?? 21.138325] CR2: 0000557c22ec68a0 CR3:
> > > > > > > 0000000004c0a003
> > > > > > > CR4:
> > > > > > > 00000000003706e0
> > > > > > > kern? :warn? : [?? 21.138329] Call Trace:
> > > > > > > kern? :warn? : [?? 21.138334]? hsw_disable_lcpll+0x22/0x280 [i915]
> > > > > > > kern? :warn? : [?? 21.138479]? i915_drm_suspend_late+0x5e/0x100
> > > > > > > [i915]
> > > > > > > kern? :warn? : [?? 21.138578]? ? pci_pm_poweroff_late+0x40/0x40
> > > > > > > kern? :warn? : [?? 21.138582]? dpm_run_callback+0x4c/0x140
> > > > > > > kern? :warn? : [?? 21.138590]? __device_suspend_late+0x98/0x1a0
> > > > > > > kern? :warn? : [?? 21.138596]? async_suspend_late+0x1b/0xa0
> > > > > > > kern? :warn? : [?? 21.138602]? async_run_entry_fn+0x39/0x160
> > > > > > > kern? :warn? : [?? 21.138608]? process_one_work+0x1ed/0x3c0
> > > > > > > kern? :warn? : [?? 21.138612]? worker_thread+0x50/0x3c0
> > > > > > > kern? :warn? : [?? 21.138617]? ? process_one_work+0x3c0/0x3c0
> > > > > > > kern? :warn? : [?? 21.138620]? kthread+0x116/0x160
> > > > > > > kern? :warn? : [?? 21.138626]? ? kthread_park+0xa0/0xa0
> > > > > > > kern? :warn? : [?? 21.138631]? ret_from_fork+0x22/0x30
> > > > > > > kern? :warn? : [?? 21.138640] ---[ end trace a93fe4d40a5a37bb ]---
> > > > > > > 
> > > > > > > 
> > > > > > > 
> > > > > > > To reproduce:
> > > > > > > 
> > > > > > > ??????? git clone https://github.com/intel/lkp-tests.git
> > > > > > > ??????? cd lkp-tests
> > > > > > > ??????? bin/lkp install job.yaml? # job file is attached in this
> > > > > > > email
> > > > > > > ??????? bin/lkp run???? job.yaml
> > > > > > > 
> > > > > > > 
> > > > > > > 
> > > > > > > Thanks,
> > > > > > > Oliver Sang
> > > > > > > 
> > > > > > 
> > > > > > -- 
> > > > > > Sincerely,
> > > > > > ?? Lyude Paul (she/her)
> > > > > > ?? Software Engineer at Red Hat
> > > > > > ?? 
> > > > > > Note: I deal with a lot of emails and have a lot of bugs on my plate.
> > > > > > If
> > > > > > you've
> > > > > > asked me a question, are waiting for a review/merge on a patch, etc.
> > > > > > and I
> > > > > > haven't responded in a while, please feel free to send me another
> > > > > > email to
> > > > > > check
> > > > > > on my status. I don't bite!
> > > > > > 
> > > > 
> > > 
> > > -- 
> > > Sincerely,
> > > ?? Lyude Paul (she/her)
> > > ?? Software Engineer at Red Hat
> > > ?? 
> > > Note: I deal with a lot of emails and have a lot of bugs on my plate. If
> > > you've
> > > asked me a question, are waiting for a review/merge on a patch, etc. and I
> > > haven't responded in a while, please feel free to send me another email to
> > > check
> > > on my status. I don't bite!
> > > 
> 
> -- 
> Sincerely,
>    Lyude Paul (she/her)
>    Software Engineer at Red Hat
>    
> Note: I deal with a lot of emails and have a lot of bugs on my plate. If you've
> asked me a question, are waiting for a review/merge on a patch, etc. and I
> haven't responded in a while, please feel free to send me another email to check
> on my status. I don't bite!
>