* [PATCH] drm/sun4i: Unbind components before releasing DRM and mem at master unbind
@ 2019-04-19 8:02 Paul Kocialkowski
2019-04-19 16:02 ` [linux-sunxi] " Chen-Yu Tsai
0 siblings, 1 reply; 6+ messages in thread
From: Paul Kocialkowski @ 2019-04-19 8:02 UTC (permalink / raw)
To: dri-devel, linux-arm-kernel, linux-kernel, linux-sunxi
Cc: Maxime Ripard, David Airlie, Daniel Vetter, Chen-Yu Tsai,
Thomas Petazzoni, Paul Kocialkowski
Our components may still be using the DRM device driver (if only to
access our driver's private data), so make sure to unbind them before
the final drm_dev_put.
Also release our resserved memory adter unbind to match reverse
creation order.
Fixes: f5a9ed867c83 ("drm/sun4i: Fix component unbinding and component master deletion")
Signed-off-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com>
---
drivers/gpu/drm/sun4i/sun4i_drv.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/sun4i/sun4i_drv.c b/drivers/gpu/drm/sun4i/sun4i_drv.c
index 843b86661833..29258b404e54 100644
--- a/drivers/gpu/drm/sun4i/sun4i_drv.c
+++ b/drivers/gpu/drm/sun4i/sun4i_drv.c
@@ -149,10 +149,11 @@ static void sun4i_drv_unbind(struct device *dev)
drm_kms_helper_poll_fini(drm);
drm_atomic_helper_shutdown(drm);
drm_mode_config_cleanup(drm);
- of_reserved_mem_device_release(dev);
- drm_dev_put(drm);
component_unbind_all(dev, NULL);
+ of_reserved_mem_device_release(dev);
+
+ drm_dev_put(drm);
}
static const struct component_master_ops sun4i_drv_master_ops = {
--
2.21.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [linux-sunxi] [PATCH] drm/sun4i: Unbind components before releasing DRM and mem at master unbind
2019-04-19 8:02 [PATCH] drm/sun4i: Unbind components before releasing DRM and mem at master unbind Paul Kocialkowski
@ 2019-04-19 16:02 ` Chen-Yu Tsai
2019-04-19 17:10 ` Paul Kocialkowski
0 siblings, 1 reply; 6+ messages in thread
From: Chen-Yu Tsai @ 2019-04-19 16:02 UTC (permalink / raw)
To: Paul Kocialkowski
Cc: dri-devel, linux-arm-kernel, linux-kernel, linux-sunxi,
Maxime Ripard, David Airlie, Daniel Vetter, Thomas Petazzoni
On Fri, Apr 19, 2019 at 1:03 AM Paul Kocialkowski
<paul.kocialkowski@bootlin.com> wrote:
>
> Our components may still be using the DRM device driver (if only to
> access our driver's private data), so make sure to unbind them before
> the final drm_dev_put.
>
> Also release our resserved memory adter unbind to match reverse
typos...
> creation order.
>
> Fixes: f5a9ed867c83 ("drm/sun4i: Fix component unbinding and component master deletion")
> Signed-off-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com>
> ---
> drivers/gpu/drm/sun4i/sun4i_drv.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/sun4i/sun4i_drv.c b/drivers/gpu/drm/sun4i/sun4i_drv.c
> index 843b86661833..29258b404e54 100644
> --- a/drivers/gpu/drm/sun4i/sun4i_drv.c
> +++ b/drivers/gpu/drm/sun4i/sun4i_drv.c
> @@ -149,10 +149,11 @@ static void sun4i_drv_unbind(struct device *dev)
> drm_kms_helper_poll_fini(drm);
> drm_atomic_helper_shutdown(drm);
> drm_mode_config_cleanup(drm);
> - of_reserved_mem_device_release(dev);
> - drm_dev_put(drm);
>
> component_unbind_all(dev, NULL);
> + of_reserved_mem_device_release(dev);
You should probably mention this change in the commit log as well.
Otherwise,
Reviewed-by: Chen-Yu Tsai <wens@csie.org>
> +
> + drm_dev_put(drm);
> }
>
> static const struct component_master_ops sun4i_drv_master_ops = {
> --
> 2.21.0
>
> --
> You received this message because you are subscribed to the Google Groups "linux-sunxi" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [linux-sunxi] [PATCH] drm/sun4i: Unbind components before releasing DRM and mem at master unbind
2019-04-19 16:02 ` [linux-sunxi] " Chen-Yu Tsai
@ 2019-04-19 17:10 ` Paul Kocialkowski
2019-04-23 17:06 ` Paul Kocialkowski
0 siblings, 1 reply; 6+ messages in thread
From: Paul Kocialkowski @ 2019-04-19 17:10 UTC (permalink / raw)
To: Chen-Yu Tsai
Cc: dri-devel, linux-arm-kernel, linux-kernel, linux-sunxi,
Maxime Ripard, David Airlie, Daniel Vetter, Thomas Petazzoni
Hi,
On Fri, 2019-04-19 at 09:02 -0700, Chen-Yu Tsai wrote:
> On Fri, Apr 19, 2019 at 1:03 AM Paul Kocialkowski
> <paul.kocialkowski@bootlin.com> wrote:
> > Our components may still be using the DRM device driver (if only to
> > access our driver's private data), so make sure to unbind them before
> > the final drm_dev_put.
> >
> > Also release our resserved memory adter unbind to match reverse
>
> typos...
I'll probably spin up a v2 to fix them, they annoy me as well.
> > creation order.
> >
> > Fixes: f5a9ed867c83 ("drm/sun4i: Fix component unbinding and component master deletion")
> > Signed-off-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com>
> > ---
> > drivers/gpu/drm/sun4i/sun4i_drv.c | 5 +++--
> > 1 file changed, 3 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/sun4i/sun4i_drv.c b/drivers/gpu/drm/sun4i/sun4i_drv.c
> > index 843b86661833..29258b404e54 100644
> > --- a/drivers/gpu/drm/sun4i/sun4i_drv.c
> > +++ b/drivers/gpu/drm/sun4i/sun4i_drv.c
> > @@ -149,10 +149,11 @@ static void sun4i_drv_unbind(struct device *dev)
> > drm_kms_helper_poll_fini(drm);
> > drm_atomic_helper_shutdown(drm);
> > drm_mode_config_cleanup(drm);
> > - of_reserved_mem_device_release(dev);
> > - drm_dev_put(drm);
> >
> > component_unbind_all(dev, NULL);
> > + of_reserved_mem_device_release(dev);
>
> You should probably mention this change in the commit log as well.
That's what I meant with the line that has typos. Maybe I should make
it slightly more explicit as:
Also released our reserved memory after component unbind instead of
before to match reverse creation order.
What do you think?
> Otherwise,
>
> Reviewed-by: Chen-Yu Tsai <wens@csie.org>
Thanks for the review!
Cheers,
Paul
> > +
> > + drm_dev_put(drm);
> > }
> >
> > static const struct component_master_ops sun4i_drv_master_ops = {
> > --
> > 2.21.0
> >
> > --
> > You received this message because you are subscribed to the Google Groups "linux-sunxi" group.
> > To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe@googlegroups.com.
> > For more options, visit https://groups.google.com/d/optout.
--
Paul Kocialkowski, Bootlin
Embedded Linux and kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [linux-sunxi] [PATCH] drm/sun4i: Unbind components before releasing DRM and mem at master unbind
2019-04-19 17:10 ` Paul Kocialkowski
@ 2019-04-23 17:06 ` Paul Kocialkowski
2019-04-23 17:14 ` Chen-Yu Tsai
0 siblings, 1 reply; 6+ messages in thread
From: Paul Kocialkowski @ 2019-04-23 17:06 UTC (permalink / raw)
To: Chen-Yu Tsai
Cc: dri-devel, linux-arm-kernel, linux-kernel, linux-sunxi,
Maxime Ripard, David Airlie, Daniel Vetter, Thomas Petazzoni
Hi,
Le vendredi 19 avril 2019 à 19:10 +0200, Paul Kocialkowski a écrit :
> Hi,
>
> On Fri, 2019-04-19 at 09:02 -0700, Chen-Yu Tsai wrote:
> > On Fri, Apr 19, 2019 at 1:03 AM Paul Kocialkowski
> > <paul.kocialkowski@bootlin.com> wrote:
> > > Our components may still be using the DRM device driver (if only to
> > > access our driver's private data), so make sure to unbind them before
> > > the final drm_dev_put.
> > >
> > > Also release our resserved memory adter unbind to match reverse
> >
> > typos...
>
> I'll probably spin up a v2 to fix them, they annoy me as well.
>
> > > creation order.
> > >
> > > Fixes: f5a9ed867c83 ("drm/sun4i: Fix component unbinding and component master deletion")
> > > Signed-off-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com>
> > > ---
> > > drivers/gpu/drm/sun4i/sun4i_drv.c | 5 +++--
> > > 1 file changed, 3 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/drivers/gpu/drm/sun4i/sun4i_drv.c b/drivers/gpu/drm/sun4i/sun4i_drv.c
> > > index 843b86661833..29258b404e54 100644
> > > --- a/drivers/gpu/drm/sun4i/sun4i_drv.c
> > > +++ b/drivers/gpu/drm/sun4i/sun4i_drv.c
> > > @@ -149,10 +149,11 @@ static void sun4i_drv_unbind(struct device *dev)
> > > drm_kms_helper_poll_fini(drm);
> > > drm_atomic_helper_shutdown(drm);
> > > drm_mode_config_cleanup(drm);
> > > - of_reserved_mem_device_release(dev);
> > > - drm_dev_put(drm);
> > >
> > > component_unbind_all(dev, NULL);
> > > + of_reserved_mem_device_release(dev);
> >
> > You should probably mention this change in the commit log as well.
>
> That's what I meant with the line that has typos. Maybe I should make
> it slightly more explicit as:
>
> Also released our reserved memory after component unbind instead of
> before to match reverse creation order.
>
> What do you think?
>
> > Otherwise,
> >
> > Reviewed-by: Chen-Yu Tsai <wens@csie.org>
>
> Thanks for the review!
Unless you have objections at this point, I'll send out the patch with
your Reviewed-by soon to get the patch out ASAP.
Cheers,
Paul
> Cheers,
>
> Paul
>
> > > +
> > > + drm_dev_put(drm);
> > > }
> > >
> > > static const struct component_master_ops sun4i_drv_master_ops = {
> > > --
> > > 2.21.0
> > >
> > > --
> > > You received this message because you are subscribed to the Google Groups "linux-sunxi" group.
> > > To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe@googlegroups.com.
> > > For more options, visit https://groups.google.com/d/optout.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [linux-sunxi] [PATCH] drm/sun4i: Unbind components before releasing DRM and mem at master unbind
2019-04-23 17:06 ` Paul Kocialkowski
@ 2019-04-23 17:14 ` Chen-Yu Tsai
2019-04-23 17:16 ` Paul Kocialkowski
0 siblings, 1 reply; 6+ messages in thread
From: Chen-Yu Tsai @ 2019-04-23 17:14 UTC (permalink / raw)
To: Paul Kocialkowski
Cc: dri-devel, linux-arm-kernel, linux-kernel, linux-sunxi,
Maxime Ripard, David Airlie, Daniel Vetter, Thomas Petazzoni
On Tue, Apr 23, 2019 at 10:06 AM Paul Kocialkowski
<paul.kocialkowski@bootlin.com> wrote:
>
> Hi,
>
> Le vendredi 19 avril 2019 à 19:10 +0200, Paul Kocialkowski a écrit :
> > Hi,
> >
> > On Fri, 2019-04-19 at 09:02 -0700, Chen-Yu Tsai wrote:
> > > On Fri, Apr 19, 2019 at 1:03 AM Paul Kocialkowski
> > > <paul.kocialkowski@bootlin.com> wrote:
> > > > Our components may still be using the DRM device driver (if only to
> > > > access our driver's private data), so make sure to unbind them before
> > > > the final drm_dev_put.
> > > >
> > > > Also release our resserved memory adter unbind to match reverse
> > >
> > > typos...
> >
> > I'll probably spin up a v2 to fix them, they annoy me as well.
> >
> > > > creation order.
> > > >
> > > > Fixes: f5a9ed867c83 ("drm/sun4i: Fix component unbinding and component master deletion")
> > > > Signed-off-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com>
> > > > ---
> > > > drivers/gpu/drm/sun4i/sun4i_drv.c | 5 +++--
> > > > 1 file changed, 3 insertions(+), 2 deletions(-)
> > > >
> > > > diff --git a/drivers/gpu/drm/sun4i/sun4i_drv.c b/drivers/gpu/drm/sun4i/sun4i_drv.c
> > > > index 843b86661833..29258b404e54 100644
> > > > --- a/drivers/gpu/drm/sun4i/sun4i_drv.c
> > > > +++ b/drivers/gpu/drm/sun4i/sun4i_drv.c
> > > > @@ -149,10 +149,11 @@ static void sun4i_drv_unbind(struct device *dev)
> > > > drm_kms_helper_poll_fini(drm);
> > > > drm_atomic_helper_shutdown(drm);
> > > > drm_mode_config_cleanup(drm);
> > > > - of_reserved_mem_device_release(dev);
> > > > - drm_dev_put(drm);
> > > >
> > > > component_unbind_all(dev, NULL);
> > > > + of_reserved_mem_device_release(dev);
> > >
> > > You should probably mention this change in the commit log as well.
> >
> > That's what I meant with the line that has typos. Maybe I should make
> > it slightly more explicit as:
> >
> > Also released our reserved memory after component unbind instead of
> > before to match reverse creation order.
> >
> > What do you think?
Forgot to reply ...
Looking again, the original already made sense. I'm not sure what I
was doing ...
> > > Otherwise,
> > >
> > > Reviewed-by: Chen-Yu Tsai <wens@csie.org>
> >
> > Thanks for the review!
>
> Unless you have objections at this point, I'll send out the patch with
> your Reviewed-by soon to get the patch out ASAP.
Works for me. Thanks!
ChenYu
> Cheers,
>
> Paul
>
> > Cheers,
> >
> > Paul
> >
> > > > +
> > > > + drm_dev_put(drm);
> > > > }
> > > >
> > > > static const struct component_master_ops sun4i_drv_master_ops = {
> > > > --
> > > > 2.21.0
> > > >
> > > > --
> > > > You received this message because you are subscribed to the Google Groups "linux-sunxi" group.
> > > > To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe@googlegroups.com.
> > > > For more options, visit https://groups.google.com/d/optout.
>
> --
> You received this message because you are subscribed to the Google Groups "linux-sunxi" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [linux-sunxi] [PATCH] drm/sun4i: Unbind components before releasing DRM and mem at master unbind
2019-04-23 17:14 ` Chen-Yu Tsai
@ 2019-04-23 17:16 ` Paul Kocialkowski
0 siblings, 0 replies; 6+ messages in thread
From: Paul Kocialkowski @ 2019-04-23 17:16 UTC (permalink / raw)
To: Chen-Yu Tsai
Cc: dri-devel, linux-arm-kernel, linux-kernel, linux-sunxi,
Maxime Ripard, David Airlie, Daniel Vetter, Thomas Petazzoni
Hi,
Le mardi 23 avril 2019 à 10:14 -0700, Chen-Yu Tsai a écrit :
> On Tue, Apr 23, 2019 at 10:06 AM Paul Kocialkowski
> <paul.kocialkowski@bootlin.com> wrote:
> > Hi,
> >
> > Le vendredi 19 avril 2019 à 19:10 +0200, Paul Kocialkowski a écrit :
> > > Hi,
> > >
> > > On Fri, 2019-04-19 at 09:02 -0700, Chen-Yu Tsai wrote:
> > > > On Fri, Apr 19, 2019 at 1:03 AM Paul Kocialkowski
> > > > <paul.kocialkowski@bootlin.com> wrote:
> > > > > Our components may still be using the DRM device driver (if only to
> > > > > access our driver's private data), so make sure to unbind them before
> > > > > the final drm_dev_put.
> > > > >
> > > > > Also release our resserved memory adter unbind to match reverse
> > > >
> > > > typos...
> > >
> > > I'll probably spin up a v2 to fix them, they annoy me as well.
> > >
> > > > > creation order.
> > > > >
> > > > > Fixes: f5a9ed867c83 ("drm/sun4i: Fix component unbinding and component master deletion")
> > > > > Signed-off-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com>
> > > > > ---
> > > > > drivers/gpu/drm/sun4i/sun4i_drv.c | 5 +++--
> > > > > 1 file changed, 3 insertions(+), 2 deletions(-)
> > > > >
> > > > > diff --git a/drivers/gpu/drm/sun4i/sun4i_drv.c b/drivers/gpu/drm/sun4i/sun4i_drv.c
> > > > > index 843b86661833..29258b404e54 100644
> > > > > --- a/drivers/gpu/drm/sun4i/sun4i_drv.c
> > > > > +++ b/drivers/gpu/drm/sun4i/sun4i_drv.c
> > > > > @@ -149,10 +149,11 @@ static void sun4i_drv_unbind(struct device *dev)
> > > > > drm_kms_helper_poll_fini(drm);
> > > > > drm_atomic_helper_shutdown(drm);
> > > > > drm_mode_config_cleanup(drm);
> > > > > - of_reserved_mem_device_release(dev);
> > > > > - drm_dev_put(drm);
> > > > >
> > > > > component_unbind_all(dev, NULL);
> > > > > + of_reserved_mem_device_release(dev);
> > > >
> > > > You should probably mention this change in the commit log as well.
> > >
> > > That's what I meant with the line that has typos. Maybe I should make
> > > it slightly more explicit as:
> > >
> > > Also released our reserved memory after component unbind instead of
> > > before to match reverse creation order.
> > >
> > > What do you think?
>
> Forgot to reply ...
>
> Looking again, the original already made sense. I'm not sure what I
> was doing ...
I like the improved description better anyway so I'll go with that :)
> > > > Otherwise,
> > > >
> > > > Reviewed-by: Chen-Yu Tsai <wens@csie.org>
> > >
> > > Thanks for the review!
> >
> > Unless you have objections at this point, I'll send out the patch with
> > your Reviewed-by soon to get the patch out ASAP.
>
> Works for me. Thanks!
Great, thanks!
Cheers,
Paul
> ChenYu
>
> > Cheers,
> >
> > Paul
> >
> > > Cheers,
> > >
> > > Paul
> > >
> > > > > +
> > > > > + drm_dev_put(drm);
> > > > > }
> > > > >
> > > > > static const struct component_master_ops sun4i_drv_master_ops = {
> > > > > --
> > > > > 2.21.0
> > > > >
> > > > > --
> > > > > You received this message because you are subscribed to the Google Groups "linux-sunxi" group.
> > > > > To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe@googlegroups.com.
> > > > > For more options, visit https://groups.google.com/d/optout.
> >
> > --
> > You received this message because you are subscribed to the Google Groups "linux-sunxi" group.
> > To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe@googlegroups.com.
> > For more options, visit https://groups.google.com/d/optout.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-04-23 17:16 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-04-19 8:02 [PATCH] drm/sun4i: Unbind components before releasing DRM and mem at master unbind Paul Kocialkowski
2019-04-19 16:02 ` [linux-sunxi] " Chen-Yu Tsai
2019-04-19 17:10 ` Paul Kocialkowski
2019-04-23 17:06 ` Paul Kocialkowski
2019-04-23 17:14 ` Chen-Yu Tsai
2019-04-23 17:16 ` Paul Kocialkowski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).