From: Manasi Navare <manasi.d.navare@intel.com>
To: Harry Wentland <hwentlan@amd.com>
Cc: "Scott Anderson" <scott@anderso.nz>,
"Michel Dänzer" <michel@daenzer.net>,
"DRI Development" <dri-devel@lists.freedesktop.org>,
"Alex Deucher" <alexander.deucher@amd.com>,
"Anthony.Koo@amd.com" <Anthony.Koo@amd.com>,
"Nicholas Kazlauskas" <nicholas.kazlauskas@amd.com>
Subject: Re: Variable Refresh Rate & flickering screens
Date: Wed, 25 Mar 2020 11:55:27 -0700 [thread overview]
Message-ID: <20200325185526.GA14320@intel.com> (raw)
In-Reply-To: <d921efb1-1074-1ac2-6eda-8ebb39a262d8@amd.com>
On Tue, Mar 17, 2020 at 09:25:33AM -0400, Harry Wentland wrote:
>
>
> On 2020-03-17 5:08 a.m., Simon Ser wrote:
> > On Thursday, March 12, 2020 3:43 PM, Harry Wentland <hwentlan@amd.com> wrote:
> >
> >> Not the main VRR expert and we're still discussing this internally but I
> >> think it'll very much depend on the display whether you'll see flicker
> >> in this case.
> >>
> >> The other complication is that for gaming we don't want to use the
> >> cursor as a VRR trigger and only look at page flips in order to allow
> >> for smooth gameplay. For a desktop use-case that's probably not the
> >> right policy.
> >
> > I think user-space can handle this and correctly synchronize cursor
> > updates with game updates via the KMS atomic API.
> >
> > However I still think flickering should be avoided by the hardware.
> > Flickering is a completely separate issue and user-space shouldn't add
> > workarounds for screen issues like this.
> >
> > Do you think that would be acceptable? Do you have any "slew rate
> > register" in AMD hardware?
> >
In case of Intel HW, we do have a way to program the maxshift so the max increment or
decrement in the vblank in successive frames. This is designed to be used for
the displays that have a restriction on the maximum change in refresh rate between two consecutive frames.
But I am still figuring out how the panel indicates this restriction that we need to program
in the HW registers.
Harry/SImon, do you know of any such panels that have these restrictions and if they
indicate this limitation or the maxshift through EDID or DPCD?
Manasi
>
> There are no slew rate registers in current AMD HW but I also think
> slewing would best be done in kernel space, either directly in HW by HW
> that supports it or in SW for HW that doesn't support it.
>
> Harry
>
> > Thanks,
> >
> > Simon
> >
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2020-03-25 18:53 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-03-11 20:09 Variable Refresh Rate & flickering screens Simon Ser
2020-03-12 10:45 ` Michel Dänzer
2020-03-12 11:25 ` Simon Ser
2020-03-12 14:43 ` Harry Wentland
2020-03-17 9:08 ` Simon Ser
2020-03-17 13:25 ` Harry Wentland
2020-03-25 18:55 ` Manasi Navare [this message]
2020-03-26 9:45 ` Simon Ser
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=20200325185526.GA14320@intel.com \
--to=manasi.d.navare@intel.com \
--cc=Anthony.Koo@amd.com \
--cc=alexander.deucher@amd.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=hwentlan@amd.com \
--cc=michel@daenzer.net \
--cc=nicholas.kazlauskas@amd.com \
--cc=scott@anderso.nz \
/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.