From: Jani Nikula <jani.nikula@linux.intel.com>
To: Thomas Zimmermann <tzimmermann@suse.de>,
Arun R Murthy <arun.r.murthy@intel.com>,
Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
Maxime Ripard <mripard@kernel.org>,
David Airlie <airlied@gmail.com>, Simona Vetter <simona@ffwll.ch>,
Rodrigo Vivi <rodrigo.vivi@intel.com>,
Joonas Lahtinen <joonas.lahtinen@linux.intel.com>,
Tvrtko Ursulin <tursulin@ursulin.net>,
xaver.hugl@kde.org, harry.wentland@amd.com,
uma.shankar@intel.com, louis.chauvet@bootlin.com,
naveen1.kumar@intel.com, ramya.krishna.yella@intel.com
Cc: dri-devel@lists.freedesktop.org, intel-gfx@lists.freedesktop.org,
intel-xe@lists.freedesktop.org,
Suraj Kandpal <suraj.kandpal@intel.com>
Subject: Re: [PATCH v11 1/7] drm: Define user readable error codes for atomic ioctl
Date: Mon, 05 Oct 2026 15:07:17 +0300 [thread overview]
Message-ID: <1f3c2a41a80a9e2902e72be385596ce285b96b2e@intel.com> (raw)
In-Reply-To: <e9be1ef8-9884-4117-9eed-ba179248ffc6@suse.de>
On Mon, 05 Oct 2026, Thomas Zimmermann <tzimmermann@suse.de> wrote:
> I have serious doubts about these failure codes. The core issue to me is
> that when the kernel driver detects an impossible commit, it probably
> knows best how to fix it. Yet the failure codes often lack actionable
> meaning.
From a different perspective, one of my concerns is that not only the
error codes become ABI, but also the behaviour becomes ABI.
When you're doing atomic check, you usually bail out at the first
problem you can't wiggle around. It's hard or impossible to figure out
the next problem or a bigger problem, because you've just detected you
have a configuration you can't support anyway. You hit the first problem
and you report that.
Now, let's assume userspace adapts to an error code, retries with
different settings, and things work. What if we want to do atomic checks
in a different order in the driver, because it makes sense or is better
or whatever. Is it a regression to report a different error code because
we now do the checks in a different order? Is that painting ourselves in
the corner?
BR,
Jani.
--
Jani Nikula, Intel
next prev parent reply other threads:[~2026-10-05 12:07 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-03-31 9:03 [PATCH v11 0/7] User readable error codes on atomic_ioctl failure Arun R Murthy
2026-03-31 9:03 ` [PATCH v11 1/7] drm: Define user readable error codes for atomic ioctl Arun R Murthy
2026-06-25 23:09 ` Xaver Hugl
2026-07-20 7:03 ` Murthy, Arun R
2026-10-05 9:08 ` Thomas Zimmermann
2026-10-05 11:39 ` Xaver Hugl
2026-10-05 12:22 ` Thomas Zimmermann
2026-10-05 14:00 ` Xaver Hugl
2026-10-07 7:03 ` Murthy, Arun R
2026-10-07 8:41 ` Thomas Zimmermann
2026-10-05 12:07 ` Jani Nikula [this message]
2026-03-31 9:03 ` [PATCH v11 2/7] drm/atomic: Add error_code element in atomic_state Arun R Murthy
2026-04-02 6:17 ` kernel test robot
2026-03-31 9:03 ` [PATCH v11 3/7] drm/atomic: Call complete_signaling only if prepare_signaling is done Arun R Murthy
2026-03-31 9:03 ` [PATCH v11 4/7] drm/atomic: Allocate atomic_state at the beginning of atomic_ioctl Arun R Murthy
2026-03-31 9:03 ` [PATCH v11 5/7] drm/atomic: Return user readable error in atomic_ioctl Arun R Murthy
2026-03-31 9:03 ` [PATCH v11 6/7] drm/i915/display: Error codes for async flip failures Arun R Murthy
2026-03-31 9:03 ` [PATCH v11 7/7] drm: Introduce DRM_CAP_ATOMIC_ERROR_REPORTING Arun R Murthy
2026-04-20 8:32 ` [PATCH v11 0/7] User readable error codes on atomic_ioctl failure Kumar, Naveen1
2026-04-21 8:53 ` Michel Dänzer
2026-04-24 11:37 ` Kumar, Naveen1
2026-04-24 14:00 ` Michel Dänzer
2026-10-07 8:51 ` Thomas Zimmermann
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=1f3c2a41a80a9e2902e72be385596ce285b96b2e@intel.com \
--to=jani.nikula@linux.intel.com \
--cc=airlied@gmail.com \
--cc=arun.r.murthy@intel.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=harry.wentland@amd.com \
--cc=intel-gfx@lists.freedesktop.org \
--cc=intel-xe@lists.freedesktop.org \
--cc=joonas.lahtinen@linux.intel.com \
--cc=louis.chauvet@bootlin.com \
--cc=maarten.lankhorst@linux.intel.com \
--cc=mripard@kernel.org \
--cc=naveen1.kumar@intel.com \
--cc=ramya.krishna.yella@intel.com \
--cc=rodrigo.vivi@intel.com \
--cc=simona@ffwll.ch \
--cc=suraj.kandpal@intel.com \
--cc=tursulin@ursulin.net \
--cc=tzimmermann@suse.de \
--cc=uma.shankar@intel.com \
--cc=xaver.hugl@kde.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox