All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
To: Rodrigo Vivi <rodrigo.vivi@intel.com>
Cc: dim-tools@lists.freedesktop.org,
	Daniel Vetter <daniel.vetter@ffwll.ch>,
	intel-gfx@lists.freedesktop.org, Sean Paul <sean@poorly.run>,
	dri-devel@lists.freedesktop.org
Subject: Re: [PULL] drm-intel-fixes v2
Date: Wed, 9 Dec 2020 14:11:42 +0200	[thread overview]
Message-ID: <X9C+/nvvyls/vj1b@intel.com> (raw)
In-Reply-To: <20201203134705.GA1575873@intel.com>

On Thu, Dec 03, 2020 at 05:47:05AM -0800, Rodrigo Vivi wrote:
> Hi Dave and Daniel,
> 
> Please ignore the pull request I had sent yesterday and use only this one.
> 
> I had missed one patch: 14d1eaf08845 ("drm/i915/gt: Protect context lifetime with RCU").
> 
> Also, please notice that the commit 6db58901c2aa ("drm/i915/display: return earlier from
> +intel_modeset_init() without display") was not actually a crucial fix, but it
> +allowed a clean pick of the use-after-free one.
> 
> Here goes drm-intel-fixes-2020-12-03:
> Fixes for GPU hang, null dereference, suspend-resume, power consumption, and use-after-free.
> 
> - Program mocs:63 for cache eviction on gen9 (Chris)
> - Protect context lifetime with RCU (Chris)
> - Split the breadcrumb spinlock between global and contexts (Chris)
> - Retain default context state across shrinking (Venkata)
> - Limit frequency drop to RPe on parking (Chris)
> - Return earlier from intel_modeset_init() without display (Jani)
> - Defer initial modeset until after GGTT is initialized (Chris)
> 
> Thanks,
> Rodrigo.
> 
> The following changes since commit b65054597872ce3aefbc6a666385eabdf9e288da:
> 
>   Linux 5.10-rc6 (2020-11-29 15:50:50 -0800)
> 
> are available in the Git repository at:
> 
>   git://anongit.freedesktop.org/drm/drm-intel tags/drm-intel-fixes-2020-12-03
> 
> for you to fetch changes up to ccc9e67ab26feda7e62749bb54c05d7abe07dca9:
> 
>   drm/i915/display: Defer initial modeset until after GGTT is initialised (2020-12-02 17:05:58 -0800)
> 
> ----------------------------------------------------------------
> Fixes for GPU hang, null dereference, suspend-resume, power consumption, and use-after-free.
> 
> - Program mocs:63 for cache eviction on gen9 (Chris)
> - Protect context lifetime with RCU (Chris)
> - Split the breadcrumb spinlock between global and contexts (Chris)
> - Retain default context state across shrinking (Venkata)
> - Limit frequency drop to RPe on parking (Chris)
> - Return earlier from intel_modeset_init() without display (Jani)
> - Defer initial modeset until after GGTT is initialized (Chris)
> 
> ----------------------------------------------------------------
> Chris Wilson (5):
>       drm/i915/gt: Program mocs:63 for cache eviction on gen9

That also needs
commit 444fbf5d7058 ("drm/i915/gt: Declare gen9 has 64 mocs entries!")
which seems to have not made it into this pull.

>       drm/i915/gt: Protect context lifetime with RCU
>       drm/i915/gt: Split the breadcrumb spinlock between global and contexts
>       drm/i915/gt: Limit frequency drop to RPe on parking
>       drm/i915/display: Defer initial modeset until after GGTT is initialised
> 
> Jani Nikula (1):
>       drm/i915/display: return earlier from intel_modeset_init() without display
> 
> Venkata Ramana Nayana (1):
>       drm/i915/gt: Retain default context state across shrinking
> 
>  drivers/gpu/drm/i915/display/intel_display.c      |  24 ++--
>  drivers/gpu/drm/i915/gt/intel_breadcrumbs.c       | 168 ++++++++++------------
>  drivers/gpu/drm/i915/gt/intel_breadcrumbs_types.h |   6 +-
>  drivers/gpu/drm/i915/gt/intel_context.c           |  15 +-
>  drivers/gpu/drm/i915/gt/intel_context_types.h     |  23 ++-
>  drivers/gpu/drm/i915/gt/intel_mocs.c              |  14 +-
>  drivers/gpu/drm/i915/gt/intel_rps.c               |   4 +
>  drivers/gpu/drm/i915/gt/shmem_utils.c             |   7 +-
>  drivers/gpu/drm/i915/i915_request.h               |   6 +-
>  9 files changed, 143 insertions(+), 124 deletions(-)
> _______________________________________________
> dim-tools mailing list
> dim-tools@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dim-tools

-- 
Ville Syrjälä
Intel
_______________________________________________
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: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
To: Rodrigo Vivi <rodrigo.vivi@intel.com>
Cc: dim-tools@lists.freedesktop.org,
	Daniel Vetter <daniel.vetter@ffwll.ch>,
	intel-gfx@lists.freedesktop.org,
	Maxime Ripard <mripard@kernel.org>,
	dri-devel@lists.freedesktop.org
Subject: Re: [Intel-gfx] [PULL] drm-intel-fixes v2
Date: Wed, 9 Dec 2020 14:11:42 +0200	[thread overview]
Message-ID: <X9C+/nvvyls/vj1b@intel.com> (raw)
In-Reply-To: <20201203134705.GA1575873@intel.com>

On Thu, Dec 03, 2020 at 05:47:05AM -0800, Rodrigo Vivi wrote:
> Hi Dave and Daniel,
> 
> Please ignore the pull request I had sent yesterday and use only this one.
> 
> I had missed one patch: 14d1eaf08845 ("drm/i915/gt: Protect context lifetime with RCU").
> 
> Also, please notice that the commit 6db58901c2aa ("drm/i915/display: return earlier from
> +intel_modeset_init() without display") was not actually a crucial fix, but it
> +allowed a clean pick of the use-after-free one.
> 
> Here goes drm-intel-fixes-2020-12-03:
> Fixes for GPU hang, null dereference, suspend-resume, power consumption, and use-after-free.
> 
> - Program mocs:63 for cache eviction on gen9 (Chris)
> - Protect context lifetime with RCU (Chris)
> - Split the breadcrumb spinlock between global and contexts (Chris)
> - Retain default context state across shrinking (Venkata)
> - Limit frequency drop to RPe on parking (Chris)
> - Return earlier from intel_modeset_init() without display (Jani)
> - Defer initial modeset until after GGTT is initialized (Chris)
> 
> Thanks,
> Rodrigo.
> 
> The following changes since commit b65054597872ce3aefbc6a666385eabdf9e288da:
> 
>   Linux 5.10-rc6 (2020-11-29 15:50:50 -0800)
> 
> are available in the Git repository at:
> 
>   git://anongit.freedesktop.org/drm/drm-intel tags/drm-intel-fixes-2020-12-03
> 
> for you to fetch changes up to ccc9e67ab26feda7e62749bb54c05d7abe07dca9:
> 
>   drm/i915/display: Defer initial modeset until after GGTT is initialised (2020-12-02 17:05:58 -0800)
> 
> ----------------------------------------------------------------
> Fixes for GPU hang, null dereference, suspend-resume, power consumption, and use-after-free.
> 
> - Program mocs:63 for cache eviction on gen9 (Chris)
> - Protect context lifetime with RCU (Chris)
> - Split the breadcrumb spinlock between global and contexts (Chris)
> - Retain default context state across shrinking (Venkata)
> - Limit frequency drop to RPe on parking (Chris)
> - Return earlier from intel_modeset_init() without display (Jani)
> - Defer initial modeset until after GGTT is initialized (Chris)
> 
> ----------------------------------------------------------------
> Chris Wilson (5):
>       drm/i915/gt: Program mocs:63 for cache eviction on gen9

That also needs
commit 444fbf5d7058 ("drm/i915/gt: Declare gen9 has 64 mocs entries!")
which seems to have not made it into this pull.

>       drm/i915/gt: Protect context lifetime with RCU
>       drm/i915/gt: Split the breadcrumb spinlock between global and contexts
>       drm/i915/gt: Limit frequency drop to RPe on parking
>       drm/i915/display: Defer initial modeset until after GGTT is initialised
> 
> Jani Nikula (1):
>       drm/i915/display: return earlier from intel_modeset_init() without display
> 
> Venkata Ramana Nayana (1):
>       drm/i915/gt: Retain default context state across shrinking
> 
>  drivers/gpu/drm/i915/display/intel_display.c      |  24 ++--
>  drivers/gpu/drm/i915/gt/intel_breadcrumbs.c       | 168 ++++++++++------------
>  drivers/gpu/drm/i915/gt/intel_breadcrumbs_types.h |   6 +-
>  drivers/gpu/drm/i915/gt/intel_context.c           |  15 +-
>  drivers/gpu/drm/i915/gt/intel_context_types.h     |  23 ++-
>  drivers/gpu/drm/i915/gt/intel_mocs.c              |  14 +-
>  drivers/gpu/drm/i915/gt/intel_rps.c               |   4 +
>  drivers/gpu/drm/i915/gt/shmem_utils.c             |   7 +-
>  drivers/gpu/drm/i915/i915_request.h               |   6 +-
>  9 files changed, 143 insertions(+), 124 deletions(-)
> _______________________________________________
> dim-tools mailing list
> dim-tools@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dim-tools

-- 
Ville Syrjälä
Intel
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

  reply	other threads:[~2020-12-09 12:11 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-12-03 13:47 [PULL] drm-intel-fixes v2 Rodrigo Vivi
2020-12-03 13:47 ` [Intel-gfx] " Rodrigo Vivi
2020-12-09 12:11 ` Ville Syrjälä [this message]
2020-12-09 12:11   ` Ville Syrjälä
2020-12-09 17:14   ` Vivi, Rodrigo
2020-12-09 17:14     ` [Intel-gfx] " Vivi, Rodrigo
  -- strict thread matches above, loose matches on Subject: below --
2017-09-20 12:37 Rodrigo Vivi

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=X9C+/nvvyls/vj1b@intel.com \
    --to=ville.syrjala@linux.intel.com \
    --cc=daniel.vetter@ffwll.ch \
    --cc=dim-tools@lists.freedesktop.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=rodrigo.vivi@intel.com \
    --cc=sean@poorly.run \
    /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 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.