From: Sean Paul <seanpaul@chromium.org>
To: mark.yao@rock-chips.com, dri-devel@lists.freedesktop.org,
ykk@rock-chips.com
Cc: marcheu@chromium.org, dianders@chromium.org,
linux-rockchip@lists.infradead.org
Subject: [PATCH v2 3/6] drm/rockchip: Use a spinlock to protect psr state
Date: Tue, 16 Aug 2016 18:11:13 -0700 [thread overview]
Message-ID: <1471396276-8222-4-git-send-email-seanpaul@chromium.org> (raw)
In-Reply-To: <1471396276-8222-1-git-send-email-seanpaul@chromium.org>
The handling of psr state is racey, shore that up with
a per-psr driver lock.
Signed-off-by: Sean Paul <seanpaul@chromium.org>
---
Changes in v2:
- Rebased on https://cgit.freedesktop.org/~seanpaul/dogwood/log/?h=for-next
drivers/gpu/drm/rockchip/rockchip_drm_psr.c | 26 +++++++++++++++++---------
1 file changed, 17 insertions(+), 9 deletions(-)
diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_psr.c b/drivers/gpu/drm/rockchip/rockchip_drm_psr.c
index 4c645d7..5bd54f2 100644
--- a/drivers/gpu/drm/rockchip/rockchip_drm_psr.c
+++ b/drivers/gpu/drm/rockchip/rockchip_drm_psr.c
@@ -30,6 +30,7 @@ struct psr_drv {
struct list_head list;
struct drm_encoder *encoder;
+ spinlock_t lock;
enum psr_state state;
struct timer_list flush_timer;
@@ -55,7 +56,7 @@ out:
return psr;
}
-static void psr_set_state(struct psr_drv *psr, enum psr_state state)
+static void psr_set_state_locked(struct psr_drv *psr, enum psr_state state)
{
/*
* Allowed finite state machine:
@@ -75,7 +76,6 @@ static void psr_set_state(struct psr_drv *psr, enum psr_state state)
if (state == PSR_FLUSH && psr->state == PSR_DISABLE)
return;
- /* Only wrote in this work, no need lock protection */
psr->state = state;
/* Allow but no need hardware change, just need assign the state */
@@ -95,18 +95,28 @@ static void psr_set_state(struct psr_drv *psr, enum psr_state state)
}
}
+static void psr_set_state(struct psr_drv *psr, enum psr_state state)
+{
+ unsigned long flags;
+
+ spin_lock_irqsave(&psr->lock, flags);
+ psr_set_state_locked(psr, state);
+ spin_unlock_irqrestore(&psr->lock, flags);
+}
+
static void psr_flush_handler(unsigned long data)
{
struct psr_drv *psr = (struct psr_drv *)data;
+ unsigned long flags;
if (!psr)
return;
/* State changed between flush time, then keep it */
- if (psr->state != PSR_FLUSH)
- return;
-
- psr_set_state(psr, PSR_ENABLE);
+ spin_lock_irqsave(&psr->lock, flags);
+ if (psr->state == PSR_FLUSH)
+ psr_set_state_locked(psr, PSR_ENABLE);
+ spin_unlock_irqrestore(&psr->lock, flags);
}
/**
@@ -167,9 +177,6 @@ void rockchip_drm_psr_flush(struct drm_device *dev)
spin_lock_irqsave(&drm_drv->psr_list_lock, flags);
list_for_each_entry(psr, &drm_drv->psr_list, list) {
- if (psr->state == PSR_DISABLE)
- continue;
-
mod_timer(&psr->flush_timer,
round_jiffies_up(jiffies + PSR_FLUSH_TIMEOUT));
@@ -202,6 +209,7 @@ int rockchip_drm_psr_register(struct drm_encoder *encoder,
return -ENOMEM;
setup_timer(&psr->flush_timer, psr_flush_handler, (unsigned long)psr);
+ spin_lock_init(&psr->lock);
psr->state = PSR_DISABLE;
psr->encoder = encoder;
--
2.8.0.rc3.226.g39d4020
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2016-08-17 1:11 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-08-17 1:11 [PATCH v2 0/6] drm/rockchip: Some patches to update the PSR series Sean Paul
2016-08-17 1:11 ` [PATCH v2 1/6] drm/rockchip: Convert psr_list_mutex to spinlock and use it Sean Paul
2016-08-17 2:29 ` Yakir Yang
2016-08-17 1:11 ` [PATCH v2 2/6] drm/rockchip: Don't use a delayed worker for psr state changes Sean Paul
2016-08-17 2:31 ` Yakir Yang
2016-08-17 1:11 ` Sean Paul [this message]
2016-08-17 2:33 ` [PATCH v2 3/6] drm/rockchip: Use a spinlock to protect psr state Yakir Yang
2016-08-17 1:11 ` [PATCH v2 4/6] drm/rockchip: A couple small fixes to psr Sean Paul
2016-08-17 2:34 ` Yakir Yang
2016-08-17 1:11 ` [PATCH v2 5/6] drm/rockchip: Improve analogix-dp psr handling Sean Paul
2016-08-17 2:35 ` Yakir Yang
2016-08-17 1:11 ` [PATCH v2 6/6] drm/rockchip: Enable vblank without event Sean Paul
2016-08-17 3:27 ` Yakir Yang
2016-08-17 2:41 ` [PATCH v2 0/6] drm/rockchip: Some patches to update the PSR series Yakir Yang
2016-08-17 2:45 ` Sean Paul
2016-08-17 3:20 ` Yakir Yang
2016-08-17 2:45 ` Yakir Yang
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=1471396276-8222-4-git-send-email-seanpaul@chromium.org \
--to=seanpaul@chromium.org \
--cc=dianders@chromium.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=marcheu@chromium.org \
--cc=mark.yao@rock-chips.com \
--cc=ykk@rock-chips.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.