From: Daniel Vetter <daniel@ffwll.ch>
To: Paulo Zanoni <paulo.r.zanoni@intel.com>
Cc: intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH 4/4] drm/i915/fbc: enable FBC on SKL too
Date: Tue, 22 Mar 2016 12:16:06 +0100 [thread overview]
Message-ID: <20160322111606.GB28483@phenom.ffwll.local> (raw)
In-Reply-To: <1458588418-31129-6-git-send-email-paulo.r.zanoni@intel.com>
On Mon, Mar 21, 2016 at 04:26:58PM -0300, Paulo Zanoni wrote:
> Now that we're more protected against user space doing frontbuffer
> mmap rendering, the last - how many times did I say this before? -
> SKL problem seems to be solved. So let's give it a try.
>
> If you reached this commit through git bisect or if you just want more
> information about FBC, please see:
> commit a98ee79317b4091cafb502b4ffdbbbe1335e298c
> Author: Paulo Zanoni <paulo.r.zanoni@intel.com>
> Date: Tue Feb 16 18:47:21 2016 -0200
> drm/i915/fbc: enable FBC by default on HSW and BDW
>
> Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
> ---
> drivers/gpu/drm/i915/intel_fbc.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_fbc.c b/drivers/gpu/drm/i915/intel_fbc.c
> index 718ac38..67f8810 100644
> --- a/drivers/gpu/drm/i915/intel_fbc.c
> +++ b/drivers/gpu/drm/i915/intel_fbc.c
> @@ -1270,7 +1270,8 @@ void intel_fbc_init(struct drm_i915_private *dev_priv)
> * know what's going on. */
> if (i915.enable_fbc < 0) {
> i915.enable_fbc = IS_HASWELL(dev_priv) ||
> - IS_BROADWELL(dev_priv);
> + IS_BROADWELL(dev_priv) ||
> + IS_SKYLAKE(dev_priv);
Can we just future-proof this and enable on everything gen8+ where we have
fbc? Apparently bsw/bxt simply lack this ...
-Daniel
> DRM_DEBUG_KMS("Sanitized enable_fbc value: %d\n",
> i915.enable_fbc);
> }
> --
> 2.7.0
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2016-03-22 11:15 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-03-21 19:26 [PATCH 0/4] Enable FBC on SKL Paulo Zanoni
2016-03-21 19:26 ` [PATCH 1/4] drm/i915/fbc: update busy_bits even for GTT and flip flushes Paulo Zanoni
2016-03-22 11:13 ` Daniel Vetter
2016-03-22 21:33 ` Zanoni, Paulo R
2016-03-21 19:26 ` [RFC xf86-video-intel] sna: Call dirtyfb for all non-tear-free cases Paulo Zanoni
2016-03-22 11:31 ` Daniel Vetter
2016-03-22 11:36 ` Chris Wilson
2016-03-22 21:35 ` Zanoni, Paulo R
2016-03-23 8:50 ` Daniel Vetter
2016-03-23 13:16 ` Zanoni, Paulo R
2016-03-21 19:26 ` [PATCH 2/4] drm/i915/fbc: sanitize i915.enable_fbc during FBC init Paulo Zanoni
2016-03-21 19:26 ` [PATCH 3/4] drm/i915: opt-out CPU and WC mmaps from FBC Paulo Zanoni
2016-03-22 2:19 ` kbuild test robot
2016-03-22 10:28 ` Jani Nikula
2016-03-22 11:15 ` Daniel Vetter
2016-03-22 13:52 ` Jani Nikula
2016-03-22 11:29 ` Daniel Vetter
2016-03-22 21:48 ` Zanoni, Paulo R
2016-03-23 8:53 ` Daniel Vetter
2016-03-23 16:04 ` Vivi, Rodrigo
2016-03-21 19:26 ` [PATCH 4/4] drm/i915/fbc: enable FBC on SKL too Paulo Zanoni
2016-03-22 11:16 ` Daniel Vetter [this message]
2016-03-22 21:51 ` Zanoni, Paulo R
2016-03-23 8:55 ` Daniel Vetter
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=20160322111606.GB28483@phenom.ffwll.local \
--to=daniel@ffwll.ch \
--cc=intel-gfx@lists.freedesktop.org \
--cc=paulo.r.zanoni@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: 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.