Any other comments/feedback on this? Thanks and Regards, Arun R Murthy -------------------
On 23-02-2026 14:45, Arun R Murthy wrote:
EDITME: Imported from [email protected] Please review before sending. The series focuses on providing a user readable error value on a failure in drm_atomic_ioctl(). Usually -EINVAL is returned in most of the error cases and it is difficult for the user to decode the error and get to know the real cause for the error. If user gets to know the reason for the error then corrective measurements can be taken up. User will have to check for the capability DRM_CAP_ATOMIC_ERROR_REPORTING before using this feature so as to ensure that the driver supports failure reporting. TODO: driver specific error codes are to be added and will be done in the follow-up patches. TODO: Once the series is merged the element 'reserved' used for sending the failure code in struct drm_mode_atomic is to changed to err_code. The IGT related changes are pushed for review @ https://patchwork.freedesktop.org/series/153330/ [RFC] changes for libdrm pushed for review @ https://gitlab.freedesktop.org/mesa/libdrm/-/merge_requests/450 To: Maarten Lankhorst <[email protected]> To: [email protected]> To: Maxime Ripard <[email protected]> To: Thomas Zimmermann <[email protected]> To: David Airlie <[email protected]> To: Simona Vetter <[email protected]> To: Jani Nikula <[email protected]> To: Rodrigo Vivi <[email protected]> To: Joonas Lahtinen <[email protected]> To: Tvrtko Ursulin <[email protected]> To: [email protected] To: [email protected] To: [email protected] To: [email protected] To: [email protected] To: [email protected] Cc: [email protected] Cc: [email protected] Cc: [email protected] Signed-off-by: Arun R Murthy <[email protected]> Signed-off-by: Arun R Murthy <[email protected]> --- Changes in v9: - EDITME: describe what is new in this series revision. - EDITME: use bulletpoints and terse descriptions. - Link to v8: https://lore.kernel.org/r/[email protected] --- Arun R Murthy (7): drm: Define user readable error codes for atomic ioctl drm/atomic: Add error_code element in atomic_state drm/atomic: Call complete_signaling only if prepare_signaling is done drm/atomic: Allocate atomic_state at the beginning of atomic_ioctl drm/atomic: Return user readable error in atomic_ioctl drm/i915/display: Error codes for async flip failures drm: Introduce DRM_CAP_ATOMIC_ERROR_REPORTING drivers/gpu/drm/drm_atomic.c | 31 ++++++++++ drivers/gpu/drm/drm_atomic_uapi.c | 89 ++++++++++++++++++++-------- drivers/gpu/drm/drm_ioctl.c | 3 + drivers/gpu/drm/i915/display/intel_display.c | 25 ++++---- include/drm/drm_atomic.h | 10 ++++ include/uapi/drm/drm.h | 7 +++ include/uapi/drm/drm_mode.h | 37 ++++++++++++ 7 files changed, 165 insertions(+), 37 deletions(-) --- base-commit: cec43d5c2696af219fc2ef71dd7e93db48c80f66 change-id: 20250728-atomic-c9713fd357e4 Best regards,
