2023-04-04 00:35:00

by yq882255

[permalink] [raw]
Subject: [PATCH 0/3] Revert lima fdinfo patchset

From: Qiang Yu <[email protected]>

Upstream has reverted the dependant commit
df622729ddbf ("drm/scheduler: track GPU active time per entity""),
but this patchset has been pushed to drm-misc-next which still
has that commit. To avoid other branch build fail after merge
drm-misc-next, just revert the lima patchset on drm-misc-next too.

Qiang Yu (3):
Revert "drm/lima: add show_fdinfo for drm usage stats"
Revert "drm/lima: allocate unique id per drm_file"
Revert "drm/lima: add usage counting method to ctx_mgr"

drivers/gpu/drm/lima/lima_ctx.c | 30 +--------------------
drivers/gpu/drm/lima/lima_ctx.h | 3 ---
drivers/gpu/drm/lima/lima_device.h | 3 ---
drivers/gpu/drm/lima/lima_drv.c | 43 +-----------------------------
drivers/gpu/drm/lima/lima_drv.h | 1 -
5 files changed, 2 insertions(+), 78 deletions(-)

--
2.25.1


2023-04-04 15:31:10

by Emil Velikov

[permalink] [raw]
Subject: Re: [PATCH 0/3] Revert lima fdinfo patchset

On Tue, 4 Apr 2023 at 08:13, <[email protected]> wrote:
>
> From: Qiang Yu <[email protected]>
>
> Upstream has reverted the dependant commit
> df622729ddbf ("drm/scheduler: track GPU active time per entity""),
> but this patchset has been pushed to drm-misc-next which still
> has that commit. To avoid other branch build fail after merge
> drm-misc-next, just revert the lima patchset on drm-misc-next too.
>

The bug/revert is unfortunate, although we better keep the build clean
or Linus will go bananas ^_^

Fwiw for the series:
Acked-by: Emil Velikov <[email protected]>

HTH
-Emil

2023-04-04 19:29:54

by Daniel Vetter

[permalink] [raw]
Subject: Re: [PATCH 0/3] Revert lima fdinfo patchset

On Tue, Apr 04, 2023 at 04:17:33PM +0100, Emil Velikov wrote:
> On Tue, 4 Apr 2023 at 08:13, <[email protected]> wrote:
> >
> > From: Qiang Yu <[email protected]>
> >
> > Upstream has reverted the dependant commit
> > df622729ddbf ("drm/scheduler: track GPU active time per entity""),
> > but this patchset has been pushed to drm-misc-next which still
> > has that commit. To avoid other branch build fail after merge
> > drm-misc-next, just revert the lima patchset on drm-misc-next too.
> >
>
> The bug/revert is unfortunate, although we better keep the build clean
> or Linus will go bananas ^_^
>
> Fwiw for the series:
> Acked-by: Emil Velikov <[email protected]>

Can you (eitehr of you really) please push asap and make sure this doesn't
miss the last drm-misc-next pull (-rc6 is this w/e)? Otherwise we'll have
a bit a mess.
-Daniel

>
> HTH
> -Emil

--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch

2023-04-05 01:02:34

by Qiang Yu

[permalink] [raw]
Subject: Re: [PATCH 0/3] Revert lima fdinfo patchset

Applied to drm-misc-next, sorry for the trouble.

Regards,
Qiang

On Wed, Apr 5, 2023 at 3:28 AM Daniel Vetter <[email protected]> wrote:
>
> On Tue, Apr 04, 2023 at 04:17:33PM +0100, Emil Velikov wrote:
> > On Tue, 4 Apr 2023 at 08:13, <[email protected]> wrote:
> > >
> > > From: Qiang Yu <[email protected]>
> > >
> > > Upstream has reverted the dependant commit
> > > df622729ddbf ("drm/scheduler: track GPU active time per entity""),
> > > but this patchset has been pushed to drm-misc-next which still
> > > has that commit. To avoid other branch build fail after merge
> > > drm-misc-next, just revert the lima patchset on drm-misc-next too.
> > >
> >
> > The bug/revert is unfortunate, although we better keep the build clean
> > or Linus will go bananas ^_^
> >
> > Fwiw for the series:
> > Acked-by: Emil Velikov <[email protected]>
>
> Can you (eitehr of you really) please push asap and make sure this doesn't
> miss the last drm-misc-next pull (-rc6 is this w/e)? Otherwise we'll have
> a bit a mess.
> -Daniel
>
> >
> > HTH
> > -Emil
>
> --
> Daniel Vetter
> Software Engineer, Intel Corporation
> http://blog.ffwll.ch

2023-04-05 08:53:10

by Daniel Vetter

[permalink] [raw]
Subject: Re: [PATCH 0/3] Revert lima fdinfo patchset

On Wed, Apr 05, 2023 at 08:50:06AM +0800, Qiang Yu wrote:
> Applied to drm-misc-next, sorry for the trouble.

No worries, I already complained to Lucas/etnaviv people that the sched
revert should have been at least posted/discussed on dri-devel. Imo this
isn't on you.
-Daniel

>
> Regards,
> Qiang
>
> On Wed, Apr 5, 2023 at 3:28 AM Daniel Vetter <[email protected]> wrote:
> >
> > On Tue, Apr 04, 2023 at 04:17:33PM +0100, Emil Velikov wrote:
> > > On Tue, 4 Apr 2023 at 08:13, <[email protected]> wrote:
> > > >
> > > > From: Qiang Yu <[email protected]>
> > > >
> > > > Upstream has reverted the dependant commit
> > > > df622729ddbf ("drm/scheduler: track GPU active time per entity""),
> > > > but this patchset has been pushed to drm-misc-next which still
> > > > has that commit. To avoid other branch build fail after merge
> > > > drm-misc-next, just revert the lima patchset on drm-misc-next too.
> > > >
> > >
> > > The bug/revert is unfortunate, although we better keep the build clean
> > > or Linus will go bananas ^_^
> > >
> > > Fwiw for the series:
> > > Acked-by: Emil Velikov <[email protected]>
> >
> > Can you (eitehr of you really) please push asap and make sure this doesn't
> > miss the last drm-misc-next pull (-rc6 is this w/e)? Otherwise we'll have
> > a bit a mess.
> > -Daniel
> >
> > >
> > > HTH
> > > -Emil
> >
> > --
> > Daniel Vetter
> > Software Engineer, Intel Corporation
> > http://blog.ffwll.ch

--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch