CVS commit: src/sys/external/bsd/drm2/dist/drm/i915/gem

["Taylor R Campbell" <riastradh%netbsd.org@localhost>]

Module Name:    src
Committed By:   riastradh
Date:           Sun Jun 23 13:01:44 UTC 2024

Modified Files:
        src/sys/external/bsd/drm2/dist/drm/i915/gem: i915_gem_mman.c

Log Message:
Revert "i915: Reduce diff in fault routine."

Evidently this had more side effects than I thought:

warning: ../../../../external/bsd/drm2/dist/drm/i915/intel_uncore.c:1197: Unclaimed read from register 0x61204
...
{drm:netbsd:wait_panel_status+0x74} *ERROR* PPS state mismatch
...
warning: ../../../../external/bsd/drm2/dist/drm/i915/display/intel_dp.c:1175: eDP powered off while attempting aux channel communication.
...
warning: ../../../../external/bsd/drm2/dist/drm/i915/display/intel_dp.c:5616: Missing case (((&(dev_priv)->__info)->gen) == 9)

So let's roll it back until I have a better understanding of what
changed.

Between making the change and writing the commit message, I forgot
that the change affected the errno values returned by the fault
routine, which may affect how uvm handles the fault failure --
whether to retry, wait for memory and retry, or fail, or what.
Perhaps that explains the issue.


To generate a diff of this commit:
cvs rdiff -u -r1.24 -r1.25 \
    src/sys/external/bsd/drm2/dist/drm/i915/gem/i915_gem_mman.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.