All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Michel Dänzer" <michel.daenzer@mailbox.org>
To: "Sebastian Wick" <sebastian.wick@redhat.com>,
	"André Almeida" <andrealmeid@igalia.com>
Cc: pierre-eric.pelloux-prayer@amd.com,
	"Marek Olšák" <maraeo@gmail.com>,
	"Italo Nicola" <italonicola@collabora.com>,
	"Randy Dunlap" <rdunlap@infradead.org>,
	linux-kernel@vger.kernel.org, amd-gfx@lists.freedesktop.org,
	wayland-devel@lists.freedesktop.org,
	"Pekka Paalanen" <ppaalanen@gmail.com>,
	dri-devel@lists.freedesktop.org, kernel-dev@igalia.com,
	alexander.deucher@amd.com, hwentlan@amd.com,
	christian.koenig@amd.com, joshua@froggi.es
Subject: Re: [PATCH v4 1/6] drm: allow DRM_MODE_PAGE_FLIP_ASYNC for atomic commits
Date: Wed, 5 Jul 2023 12:00:07 +0200	[thread overview]
Message-ID: <fdf51523-a955-e5d0-0ea4-4c614b6f45b2@mailbox.org> (raw)
In-Reply-To: <CA+hFU4y+kHitNsqb1WC6dHE5hr=1S6gJeyHXC5mv9Ads5mdgWw@mail.gmail.com>

On 7/4/23 19:06, Sebastian Wick wrote:
> On Sat, Jul 1, 2023 at 4:09 AM André Almeida <andrealmeid@igalia.com> wrote:
>>
>> @@ -949,6 +949,15 @@ struct hdr_output_metadata {
>>   * Request that the page-flip is performed as soon as possible, ie. with no
>>   * delay due to waiting for vblank. This may cause tearing to be visible on
>>   * the screen.
>> + *
>> + * When used with atomic uAPI, the driver will return an error if the hardware
>> + * doesn't support performing an asynchronous page-flip for this update.
>> + * User-space should handle this, e.g. by falling back to a regular page-flip.
>> + *
>> + * Note, some hardware might need to perform one last synchronous page-flip
>> + * before being able to switch to asynchronous page-flips. As an exception,
>> + * the driver will return success even though that first page-flip is not
>> + * asynchronous.
> 
> What would happen if one commits another async KMS update before the
> first page flip? Does one receive EAGAIN, does it amend the previous
> commit?

Should be the former. DRM_MODE_PAGE_FLIP_ASYNC just means the flip may complete outside of vertical blank, it doesn't change anything else.


-- 
Earthling Michel Dänzer            |                  https://redhat.com
Libre software enthusiast          |         Mesa and Xwayland developer


  parent reply	other threads:[~2023-07-05 10:00 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-07-01  2:09 [PATCH v4 0/6] drm: Add support for atomic async page-flip André Almeida
2023-07-01  2:09 ` André Almeida
2023-07-01  2:09 ` André Almeida
2023-07-01  2:09 ` [PATCH v4 1/6] drm: allow DRM_MODE_PAGE_FLIP_ASYNC for atomic commits André Almeida
2023-07-01  2:09   ` André Almeida
2023-07-01  2:09   ` André Almeida
2023-07-04 17:06   ` Sebastian Wick
2023-07-04 17:06     ` Sebastian Wick
2023-07-04 17:57     ` Simon Ser
2023-07-04 17:57       ` Simon Ser
2023-07-05 10:00     ` Michel Dänzer [this message]
2023-07-01  2:09 ` [PATCH v4 2/6] drm: introduce DRM_CAP_ATOMIC_ASYNC_PAGE_FLIP André Almeida
2023-07-01  2:09   ` André Almeida
2023-07-01  2:09   ` André Almeida
2023-07-01  2:09 ` [PATCH v4 3/6] drm: introduce drm_mode_config.atomic_async_page_flip_not_supported André Almeida
2023-07-01  2:09   ` André Almeida
2023-07-01  2:09   ` André Almeida
2023-07-01  2:09 ` [PATCH v4 4/6] amd/display: indicate support for atomic async page-flips on DC André Almeida
2023-07-01  2:09   ` André Almeida
2023-07-01  2:09   ` André Almeida
2023-07-01  2:09 ` [PATCH v4 5/6] drm: Refuse to async flip with atomic prop changes André Almeida
2023-07-01  2:09   ` André Almeida
2023-07-01  2:09   ` André Almeida
2023-07-01  2:09 ` [PATCH v4 6/6] drm/doc: Define KMS atomic state set André Almeida
2023-07-01  2:09   ` André Almeida
2023-07-01  2:09   ` André Almeida
2023-07-03  8:38   ` Pekka Paalanen
2023-07-03  8:38     ` Pekka Paalanen
2023-07-03  8:38     ` Pekka Paalanen
2023-07-03 16:32     ` André Almeida
2023-07-03 16:32       ` André Almeida
2023-07-03 16:32       ` André Almeida
2023-07-05  2:42       ` James Braden olin

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=fdf51523-a955-e5d0-0ea4-4c614b6f45b2@mailbox.org \
    --to=michel.daenzer@mailbox.org \
    --cc=alexander.deucher@amd.com \
    --cc=amd-gfx@lists.freedesktop.org \
    --cc=andrealmeid@igalia.com \
    --cc=christian.koenig@amd.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=hwentlan@amd.com \
    --cc=italonicola@collabora.com \
    --cc=joshua@froggi.es \
    --cc=kernel-dev@igalia.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=maraeo@gmail.com \
    --cc=pierre-eric.pelloux-prayer@amd.com \
    --cc=ppaalanen@gmail.com \
    --cc=rdunlap@infradead.org \
    --cc=sebastian.wick@redhat.com \
    --cc=wayland-devel@lists.freedesktop.org \
    /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.