From: Chris Wilson <chris@chris-wilson.co.uk> To: intel-gfx@lists.freedesktop.org Cc: Jani Nikula <jani.nikula@intel.com>, Daniel Vetter <daniel.vetter@ffwll.ch>, dri-devel@lists.freedesktop.org, Chris Wilson <chris@chris-wilson.co.uk> Subject: [PATCH] drm/i915: Autoselect CONFIG_CHECKPOINT_RESTORE for SYS_kcmp Date: Fri, 5 Feb 2021 13:03:07 +0000 [thread overview] Message-ID: <20210205130307.8294-1-chris@chris-wilson.co.uk> (raw) gallium (iris) depends on os_same_file_description() to disambiguate screens and so avoid importing the same screen fd twice as two distinct entities (that share all the kernel resources, so actions on screen affect the other and would cause random faiure). As they depend on it, so must we. os_same_file_description() uses SYS_kcmp to check the file tables for the equivalent struct file, but SYS_kcmp is hidden behind CONFIG_CHECKPOINT_RESTORE. As this is not default, we must select it for ourselves to ensure that our userspace is fully supported. Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/3046 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Jani Nikula <jani.nikula@intel.com> Cc: Daniel Vetter <daniel.vetter@ffwll.ch> --- drivers/gpu/drm/i915/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/i915/Kconfig b/drivers/gpu/drm/i915/Kconfig index 1e1cb245fca7..470a5214bd33 100644 --- a/drivers/gpu/drm/i915/Kconfig +++ b/drivers/gpu/drm/i915/Kconfig @@ -21,6 +21,7 @@ config DRM_I915 select ACPI_VIDEO if ACPI select ACPI_BUTTON if ACPI select SYNC_FILE + select CHECKPOINT_RESTORE # gallium depends on SYS_kcmp select IOSF_MBI select CRC32 select SND_HDA_I915 if SND_HDA_CORE -- 2.20.1 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
WARNING: multiple messages have this Message-ID (diff)
From: Chris Wilson <chris@chris-wilson.co.uk> To: intel-gfx@lists.freedesktop.org Cc: Jani Nikula <jani.nikula@intel.com>, Daniel Vetter <daniel.vetter@ffwll.ch>, dri-devel@lists.freedesktop.org, Chris Wilson <chris@chris-wilson.co.uk> Subject: [Intel-gfx] [PATCH] drm/i915: Autoselect CONFIG_CHECKPOINT_RESTORE for SYS_kcmp Date: Fri, 5 Feb 2021 13:03:07 +0000 [thread overview] Message-ID: <20210205130307.8294-1-chris@chris-wilson.co.uk> (raw) gallium (iris) depends on os_same_file_description() to disambiguate screens and so avoid importing the same screen fd twice as two distinct entities (that share all the kernel resources, so actions on screen affect the other and would cause random faiure). As they depend on it, so must we. os_same_file_description() uses SYS_kcmp to check the file tables for the equivalent struct file, but SYS_kcmp is hidden behind CONFIG_CHECKPOINT_RESTORE. As this is not default, we must select it for ourselves to ensure that our userspace is fully supported. Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/3046 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Jani Nikula <jani.nikula@intel.com> Cc: Daniel Vetter <daniel.vetter@ffwll.ch> --- drivers/gpu/drm/i915/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/i915/Kconfig b/drivers/gpu/drm/i915/Kconfig index 1e1cb245fca7..470a5214bd33 100644 --- a/drivers/gpu/drm/i915/Kconfig +++ b/drivers/gpu/drm/i915/Kconfig @@ -21,6 +21,7 @@ config DRM_I915 select ACPI_VIDEO if ACPI select ACPI_BUTTON if ACPI select SYNC_FILE + select CHECKPOINT_RESTORE # gallium depends on SYS_kcmp select IOSF_MBI select CRC32 select SND_HDA_I915 if SND_HDA_CORE -- 2.20.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
next reply other threads:[~2021-02-05 13:03 UTC|newest] Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-02-05 13:03 Chris Wilson [this message] 2021-02-05 13:03 ` [Intel-gfx] [PATCH] drm/i915: Autoselect CONFIG_CHECKPOINT_RESTORE for SYS_kcmp Chris Wilson 2021-02-05 15:58 ` Daniel Vetter 2021-02-05 15:58 ` [Intel-gfx] " Daniel Vetter 2021-02-05 15:59 ` Daniel Vetter 2021-02-05 15:59 ` [Intel-gfx] " Daniel Vetter 2021-02-05 16:18 ` Lucas Stach 2021-02-05 16:18 ` [Intel-gfx] " Lucas Stach 2021-02-05 18:15 ` [Intel-gfx] ✓ Fi.CI.BAT: success for " Patchwork 2021-02-06 4:21 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
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=20210205130307.8294-1-chris@chris-wilson.co.uk \ --to=chris@chris-wilson.co.uk \ --cc=daniel.vetter@ffwll.ch \ --cc=dri-devel@lists.freedesktop.org \ --cc=intel-gfx@lists.freedesktop.org \ --cc=jani.nikula@intel.com \ /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: linkBe 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.