From: Chris Wilson <chris@chris-wilson.co.uk>
To: Daniel Vetter <daniel@ffwll.ch>
Cc: Krzysztof Kozlowski <k.kozlowski@samsung.com>,
Heiko Stuebner <heiko@sntech.de>, David Airlie <airlied@linux.ie>,
dri-devel <dri-devel@lists.freedesktop.org>,
"open list:VIRTIO CORE,
NET..." <virtualization@lists.linux-foundation.org>,
Matthias Brugger <matthias.bgg@gmail.com>,
Thierry Re ding <thierry.reding@gmail.com>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Benjamin Gaignard <benjamin.gaignard@linaro.org>,
Daniel Vetter <daniel.vetter@intel.com>,
Boris Brezillon <boris.brezillon@free-electrons.com>,
linux-samsung-soc <linux-samsung-soc@vger.kernel.org>,
Joonyoung Shim <jy0922.shim@samsung.com>,
Alexey Brodkin <abrodkin@synopsys.com>,
"open list:ARM/Rockchip SoC..."
<linux-rockchip@lists.infradead.org>,
Chen-Yu Tsai <wens@csie.org>, Kukjin Kim <kgene@kernel.org>,
Stephen Warren <swarren@wwwdotorg.org>,
linux-arm-msm <linux-arm-msm@vger.kernel.org>in
Subject: Re: [Intel-gfx] [PATCH 01/20] drm/atomic: Fix remaining places where !funcs->best_encoder is valid
Date: Thu, 2 Jun 2016 23:40:04 +0100 [thread overview]
Message-ID: <20160602224004.GA1819__20888.8725880117$1465252414$gmane$org@nuc-i3427.alporthouse.com> (raw)
In-Reply-To: <CAKMK7uHyqNbA8Scw2fE0geAyNt0a0XVtzf81nwWjEhbZ-gAvKw@mail.gmail.com>
On Thu, Jun 02, 2016 at 11:57:02PM +0200, Daniel Vetter wrote:
> drm_encoder_find is an idr lookup. That should be plenty fast,
> especially for modeset code. Usually what's too expensive even for
> modeset code is linear list walks. But Chris just submitted patches to
> convert most of them into simple lookups.
For the idr_find, I'm tempted to replace the mutex with a rwlock. It
helps pathological cases, but in reality there are more crucial
locks around the hw that limit concurrency. ;-)
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
next prev parent reply other threads:[~2016-06-02 22:40 UTC|newest]
Thread overview: 126+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-06-02 14:31 [PATCH 00/20] drm/atomic: Provide default ->best_encoder() behavior Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` [PATCH 01/20] drm/atomic: Fix remaining places where !funcs->best_encoder is valid Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 21:05 ` Laurent Pinchart
2016-06-02 21:05 ` Laurent Pinchart
2016-06-02 21:05 ` Laurent Pinchart
2016-06-02 21:05 ` Laurent Pinchart
2016-06-02 21:57 ` Daniel Vetter
2016-06-02 21:57 ` Daniel Vetter
2016-06-02 21:57 ` Daniel Vetter
2016-06-02 21:57 ` Daniel Vetter
2016-06-02 21:57 ` Daniel Vetter
2016-06-02 22:40 ` Chris Wilson [this message]
2016-06-02 22:40 ` Chris Wilson
2016-06-02 22:40 ` [Intel-gfx] " Chris Wilson
2016-06-02 22:40 ` Chris Wilson
2016-06-02 22:40 ` Chris Wilson
2016-06-03 7:37 ` Boris Brezillon
2016-06-03 7:37 ` Boris Brezillon
2016-06-03 7:37 ` Boris Brezillon
2016-06-03 7:37 ` Boris Brezillon
2016-06-03 7:37 ` Boris Brezillon
2016-06-03 19:01 ` Daniel Vetter
2016-06-03 19:01 ` Daniel Vetter
2016-06-03 19:01 ` Daniel Vetter
2016-06-03 19:01 ` Daniel Vetter
2016-06-03 19:01 ` Daniel Vetter
2016-06-03 7:36 ` Boris Brezillon
2016-06-03 7:36 ` Boris Brezillon
2016-06-03 7:36 ` Boris Brezillon
2016-06-02 14:31 ` [PATCH 02/20] drm: arc: Rely on the default ->best_encoder() behavior Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` [PATCH 03/20] drm: atmel-hlcdc: " Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` [PATCH 04/20] drm: exynos: " Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` [PATCH 05/20] drm: fsl-dcu: " Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` [PATCH 06/20] drm: i915: Rely on the default ->best_encoder() behavior where appropriate Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` [PATCH 07/20] drm: mediatek: Rely on the default ->best_encoder() behavior Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` [PATCH 08/20] drm: msm: Rely on the default ->best_encoder() behavior where appropriate Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` [PATCH 09/20] drm: rcar-du: Rely on the default ->best_encoder() behavior Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 20:57 ` Laurent Pinchart
2016-06-02 20:57 ` Laurent Pinchart
2016-06-02 20:57 ` Laurent Pinchart
2016-06-03 7:38 ` Boris Brezillon
2016-06-03 7:38 ` Boris Brezillon
2016-06-03 7:38 ` Boris Brezillon
2016-06-03 7:38 ` Boris Brezillon
2016-06-02 20:57 ` Laurent Pinchart
2016-06-02 14:31 ` [PATCH 10/20] drm: rockchip: " Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-07 0:51 ` Mark yao
2016-06-07 0:51 ` Mark yao
2016-06-07 0:51 ` Mark yao
2016-06-07 0:51 ` Mark yao
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` [PATCH 11/20] drm: sti: " Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` [PATCH 12/20] drm: sun4i: " Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` [PATCH 13/20] drm: tegra: " Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` [PATCH 14/20] drm: vc4: " Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` [PATCH 15/20] drm: virtgpu: " Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` [PATCH 16/20] drm: omap: " Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 21:00 ` Laurent Pinchart
2016-06-02 21:00 ` Laurent Pinchart
2016-06-02 21:00 ` Laurent Pinchart
2016-06-02 21:00 ` Laurent Pinchart
2016-06-02 14:31 ` [PATCH 17/20] drm/bridge: anx78xx: " Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` [PATCH 18/20] drm/bridge: ptn3460: " Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` [PATCH 19/20] drm/bridge: ps8622: " Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` [PATCH 20/20] drm/bridge: dw-hdmi: Use drm_atomic_helper_best_encoder() Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 14:31 ` Boris Brezillon
2016-06-02 17:34 ` ✗ Ro.CI.BAT: failure for drm/atomic: Provide default ->best_encoder() behavior 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='20160602224004.GA1819__20888.8725880117$1465252414$gmane$org@nuc-i3427.alporthouse.com' \
--to=chris@chris-wilson.co.uk \
--cc=abrodkin@synopsys.com \
--cc=airlied@linux.ie \
--cc=benjamin.gaignard@linaro.org \
--cc=boris.brezillon@free-electrons.com \
--cc=daniel.vetter@intel.com \
--cc=daniel@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
--cc=heiko@sntech.de \
--cc=jy0922.shim@samsung.com \
--cc=k.kozlowski@samsung.com \
--cc=kgene@kernel.org \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=matthias.bgg@gmail.com \
--cc=swarren@wwwdotorg.org \
--cc=thierry.reding@gmail.com \
--cc=virtualization@lists.linux-foundation.org \
--cc=wens@csie.org \
/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.