lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:   Tue, 05 Oct 2021 16:04:40 -0700
From:   khsieh@...eaurora.org
To:     Stephen Boyd <swboyd@...omium.org>
Cc:     Bjorn Andersson <bjorn.andersson@...aro.org>,
        Abhinav Kumar <abhinavk@...eaurora.org>,
        Daniel Vetter <daniel@...ll.ch>,
        David Airlie <airlied@...ux.ie>,
        Dmitry Baryshkov <dmitry.baryshkov@...aro.org>,
        Rob Clark <robdclark@...il.com>, Sean Paul <sean@...rly.run>,
        linux-arm-msm@...r.kernel.org, dri-devel@...ts.freedesktop.org,
        freedreno@...ts.freedesktop.org, linux-kernel@...r.kernel.org,
        Sankeerth Billakanti <sbillaka@...eaurora.org>
Subject: Re: [PATCH] drm/msm/dp: Shorten SETUP timeout

On 2021-10-05 15:36, Stephen Boyd wrote:
> Quoting Bjorn Andersson (2021-10-05 14:40:38)
>> On Tue 05 Oct 11:45 PDT 2021, Stephen Boyd wrote:
>> 
>> > Quoting Bjorn Andersson (2021-10-04 19:37:50)
>> > > Found in the middle of a patch from Sankeerth was the reduction of the
>> > > INIT_SETUP timeout from 10s to 100ms. Upon INIT_SETUP timeout the host
>> > > is initalized and HPD interrupt start to be serviced, so in the case of
>> > > eDP this reduction improves the user experience dramatically - i.e.
>> > > removes 9.9s of bland screen time at boot.
>> > >
>> > > Suggested-by: Sankeerth Billakanti <sbillaka@...eaurora.org>
>> > > Signed-off-by: Bjorn Andersson <bjorn.andersson@...aro.org>
>> > > ---
>> >
>> > Any Fixes tag? BTW, the delay design is pretty convoluted. I had to go
>> > re-read the code a couple times to understand that it's waiting 100ms
>> > times the 'delay' number. Whaaaaat?
>> >
>> 
>> I assume you're happy with the current 10s delay on the current
>> devices, so I don't think we should push for this to be backported.
>> I have no need for it to be backported on my side at least.
>> 
> 
> Sure. Fixes tag != backported to stable trees but it is close.
> 
>> > Reviewed-by: Stephen Boyd <swboyd@...omium.org>
>> 
   dp_add_event(dp, EV_HPD_INIT_SETUP, 0, 1); <== to 100ms

This patch will prevent usb3 from working due to dp driver initialize 
phy earlier than usb3 which cause timeout error at power up usb3 phy 
when both edp and dp are enabled.
I had prepared a patch (drm/msm/dp: do not initialize combo phy until 
plugin interrupt) to fix this problem.
Unfortunately, my patch is depend on Bjorn's patch (PATCH v3 3/5] 
drm/msm/dp: Support up to 3 DP controllers).
I will submit my patch for review once Bjorn's patches merged in.
Therefore I would think this patch should go after both Bjorn's patches 
and my patch.



Powered by blists - more mailing lists