lkml.org 
[lkml]   [2022]   [Apr]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.17 217/219] drm/i915: Sunset igpu legacy mmap support based on GRAPHICS_VER_FULL
    Date
    From: Matt Roper <matthew.d.roper@intel.com>

    commit 1acb34e7dd7720a1fff00cbd4d000ec3219dc9d6 upstream.

    The intent of the version check in the mmap ioctl was to maintain
    support for existing platforms (i.e., ADL/RPL and earlier), but drop
    support on all future igpu platforms. As we've seen on the dgpu side,
    the hardware teams are using a more fine-grained numbering system for IP
    version numbers these days, so it's possible the version number
    associated with our next igpu could be some form of "12.xx" rather than
    13 or higher. Comparing against the full ver.release number will ensure
    the intent of the check is maintained no matter what numbering the
    hardware teams settle on.

    Fixes: d3f3baa3562a ("drm/i915: Reinstate the mmap ioctl for some platforms")
    Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com>
    Cc: Lucas De Marchi <lucas.demarchi@intel.com>
    Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
    Reviewed-by: Lucas De Marchi <lucas.demarchi@intel.com>
    Link: https://patchwork.freedesktop.org/patch/msgid/20220407161839.1073443-1-matthew.d.roper@intel.com
    (cherry picked from commit 8e7e5c077cd57ee9a36d58c65f07257dc49a88d5)
    Signed-off-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
    ---
    drivers/gpu/drm/i915/gem/i915_gem_mman.c | 2 +-
    1 file changed, 1 insertion(+), 1 deletion(-)

    --- a/drivers/gpu/drm/i915/gem/i915_gem_mman.c
    +++ b/drivers/gpu/drm/i915/gem/i915_gem_mman.c
    @@ -67,7 +67,7 @@ i915_gem_mmap_ioctl(struct drm_device *d
    * mmap ioctl is disallowed for all discrete platforms,
    * and for all platforms with GRAPHICS_VER > 12.
    */
    - if (IS_DGFX(i915) || GRAPHICS_VER(i915) > 12)
    + if (IS_DGFX(i915) || GRAPHICS_VER_FULL(i915) > IP_VER(12, 0))
    return -EOPNOTSUPP;

    if (args->flags & ~(I915_MMAP_WC))

    \
     
     \ /
      Last update: 2022-04-18 14:38    [W:4.070 / U:0.996 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site