Re: [PATCH 0/2] Move scheduler out of AMDGPU

[Lucas Stach <l.stach-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>]

Hi,

Am Samstag, den 02.12.2017, 12:14 +0000 schrieb Liu, Monk:
> I'm wondering if GPU reset still work after this home move ...

Why wouldn't it continue to work? After all this is just a code move
that doesn't change anything about the inner workings of the scheduler.

Regards,
Lucas

> 
> -----Original Message-----
> From: amd-gfx [mailto:amd-gfx-bounces@lists.freedesktop.org] On
> Behalf Of Christian K?nig
> Sent: Friday, December 1, 2017 11:55 PM
> To: Lucas Stach <l.stach@pengutronix.de>; Deucher, Alexander <Alexand
> er.Deucher@amd.com>
> Cc: Grodzovsky, Andrey <Andrey.Grodzovsky@amd.com>; David Airlie <air
> lied@linux.ie>; amd-gfx@lists.freedesktop.org; patchwork-lst@pengutro
> nix.de; dri-devel@lists.freedesktop.org; kernel@pengutronix.de
> Subject: Re: [PATCH 0/2] Move scheduler out of AMDGPU
> 
> Am 01.12.2017 um 16:28 schrieb Lucas Stach:
> > Hi all,
> > 
> > so this is the first step to make the marvelous AMDGPU scheduler 
> > useable for other drivers. I have a (mostly) working prototype of 
> > Etnaviv using the scheduler, but those patches need to keep baking
> > for a while.
> > 
> > I'm sending this out as I want to avoid rebasing this change too
> > much 
> > and don't want to take people by surprise when the Etnaviv 
> > implementation surfaces. Also this might need some coordination 
> > between AMDGPU and Etnaviv, which might be good to get going now.
> > 
> > Please speak up now if you have any objections or comments.
> 
> Looks good to me, but question is what is this based upon?
> 
> I strongly assume drm-next, so question is now if we have any patches
> inside amd branches we should apply before doing this.
> 
> CCing Andrey as well cause he has some tasks assigned around the
> scheduler as well.
> 
> Regards,
> Christian.
> 
> > 
> > Regards,
> > Lucas
> > 
> > Lucas Stach (2):
> >    drm: move amd_gpu_scheduler into common location
> >    drm/sched: move fence slab handling to module init/exit
> > 
> >   drivers/gpu/drm/Kconfig                            |   5 +
> >   drivers/gpu/drm/Makefile                           |   1 +
> >   drivers/gpu/drm/amd/amdgpu/Makefile                |   5 +-
> >   drivers/gpu/drm/amd/amdgpu/amdgpu.h                |  16 +-
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c             |   8 +-
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c            |  38 +--
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_device.c         |  12 +-
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c            |   8 -
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c          |   4 +-
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_job.c            |  22 +-
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_ring.c           |  14 +-
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_ring.h           |  12 +-
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_sched.c          |  20 +-
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_sched.h          |   2 +-
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_sync.c           |   6 +-
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c            |   8 +-
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.h            |   4 +-
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c            |   8 +-
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.h            |   4 +-
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_vce.c            |   8 +-
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_vce.h            |   2 +-
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c            |  14 +-
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.h            |   4 +-
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c             |  10 +-
> >   drivers/gpu/drm/amd/amdgpu/amdgpu_vm.h             |   4 +-
> >   drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c              |   4 +-
> >   drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c              |   8 +-
> >   drivers/gpu/drm/amd/amdgpu/uvd_v7_0.c              |   8 +-
> >   drivers/gpu/drm/amd/scheduler/gpu_scheduler.h      | 185 --------
> > ------
> >   drivers/gpu/drm/scheduler/Makefile                 |   4 +
> >   .../gpu/drm/{amd => }/scheduler/gpu_scheduler.c    | 281
> > +++++++++++----------
> >   drivers/gpu/drm/{amd => }/scheduler/sched_fence.c  | 122 +++++---
> > -
> >   include/drm/gpu_scheduler.h                        | 171
> > +++++++++++++
> >   .../drm/gpu_scheduler_trace.h                      |  14 +-
> >   34 files changed, 525 insertions(+), 511 deletions(-)
> >   delete mode 100644 drivers/gpu/drm/amd/scheduler/gpu_scheduler.h
> >   create mode 100644 drivers/gpu/drm/scheduler/Makefile
> >   rename drivers/gpu/drm/{amd => }/scheduler/gpu_scheduler.c (64%)
> >   rename drivers/gpu/drm/{amd => }/scheduler/sched_fence.c (58%)
> >   create mode 100644 include/drm/gpu_scheduler.h
> >   rename drivers/gpu/drm/amd/scheduler/gpu_sched_trace.h => 
> > include/drm/gpu_scheduler_trace.h (83%)
> > 
> 
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx