From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [RFC 09/44] drm/i915: Start of GPU scheduler Date: Mon, 7 Jul 2014 21:02:10 +0200 Message-ID: <20140707190210.GA5821@phenom.ffwll.local> References: <1403803475-16337-1-git-send-email-John.C.Harrison@Intel.com> <1403803475-16337-10-git-send-email-John.C.Harrison@Intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-wg0-f41.google.com (mail-wg0-f41.google.com [74.125.82.41]) by gabe.freedesktop.org (Postfix) with ESMTP id B277489ADC for ; Mon, 7 Jul 2014 12:02:00 -0700 (PDT) Received: by mail-wg0-f41.google.com with SMTP id k14so2191339wgh.0 for ; Mon, 07 Jul 2014 12:01:59 -0700 (PDT) Content-Disposition: inline In-Reply-To: <1403803475-16337-10-git-send-email-John.C.Harrison@Intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: John.C.Harrison@Intel.com Cc: Intel-GFX@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Thu, Jun 26, 2014 at 06:24:00PM +0100, John.C.Harrison@Intel.com wrote: > From: John Harrison > > Created GPU scheduler source files with only a basic init function. Same critique as for Oscar's execlist: Please don't order patches by adding unused leave code and structures first, but start by wiring up wired up (but mayb be still partially stubbed-out) code. The aim is to make review of individual patches possible with as little context as required - for otherwise (i.e. if you have to keep all the code in mind till the end since only then it really gets plugged in) splitting up the patches is a superficial exercise and doesn't really help the reviewer. /rant -Daniel > --- > drivers/gpu/drm/i915/Makefile | 1 + > drivers/gpu/drm/i915/i915_drv.h | 4 +++ > drivers/gpu/drm/i915/i915_gem.c | 3 ++ > drivers/gpu/drm/i915/i915_scheduler.c | 59 +++++++++++++++++++++++++++++++++ > drivers/gpu/drm/i915/i915_scheduler.h | 40 ++++++++++++++++++++++ > 5 files changed, 107 insertions(+) > create mode 100644 drivers/gpu/drm/i915/i915_scheduler.c > create mode 100644 drivers/gpu/drm/i915/i915_scheduler.h > > diff --git a/drivers/gpu/drm/i915/Makefile b/drivers/gpu/drm/i915/Makefile > index cad1683..12817a8 100644 > --- a/drivers/gpu/drm/i915/Makefile > +++ b/drivers/gpu/drm/i915/Makefile > @@ -11,6 +11,7 @@ i915-y := i915_drv.o \ > i915_params.o \ > i915_suspend.o \ > i915_sysfs.o \ > + i915_scheduler.o \ > intel_pm.o > i915-$(CONFIG_COMPAT) += i915_ioc32.o > i915-$(CONFIG_DEBUG_FS) += i915_debugfs.o > diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h > index 53f6fe5..6e592d3 100644 > --- a/drivers/gpu/drm/i915/i915_drv.h > +++ b/drivers/gpu/drm/i915/i915_drv.h > @@ -1331,6 +1331,8 @@ struct intel_pipe_crc { > wait_queue_head_t wq; > }; > > +struct i915_scheduler; > + > struct drm_i915_private { > struct drm_device *dev; > struct kmem_cache *slab; > @@ -1540,6 +1542,8 @@ struct drm_i915_private { > > struct i915_runtime_pm pm; > > + struct i915_scheduler *scheduler; > + > /* Old dri1 support infrastructure, beware the dragons ya fools entering > * here! */ > struct i915_dri1_state dri1; > diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c > index 898660c..b784eb2 100644 > --- a/drivers/gpu/drm/i915/i915_gem.c > +++ b/drivers/gpu/drm/i915/i915_gem.c > @@ -37,6 +37,7 @@ > #include > #include > #include > +#include "i915_scheduler.h" > > static void i915_gem_object_flush_gtt_write_domain(struct drm_i915_gem_object *obj); > static void i915_gem_object_flush_cpu_write_domain(struct drm_i915_gem_object *obj, > @@ -4669,6 +4670,8 @@ static int i915_gem_init_rings(struct drm_device *dev) > goto cleanup_vebox_ring; > } > > + i915_scheduler_init(dev); > + > ret = i915_gem_set_seqno(dev, ((u32)~0 - 0x1000)); > if (ret) > goto cleanup_bsd2_ring; > diff --git a/drivers/gpu/drm/i915/i915_scheduler.c b/drivers/gpu/drm/i915/i915_scheduler.c > new file mode 100644 > index 0000000..9ec0225 > --- /dev/null > +++ b/drivers/gpu/drm/i915/i915_scheduler.c > @@ -0,0 +1,59 @@ > +/* > + * Copyright (c) 2014 Intel Corporation > + * > + * Permission is hereby granted, free of charge, to any person obtaining a > + * copy of this software and associated documentation files (the "Software"), > + * to deal in the Software without restriction, including without limitation > + * the rights to use, copy, modify, merge, publish, distribute, sublicense, > + * and/or sell copies of the Software, and to permit persons to whom the > + * Software is furnished to do so, subject to the following conditions: > + * > + * The above copyright notice and this permission notice (including the next > + * paragraph) shall be included in all copies or substantial portions of the > + * Software. > + * > + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR > + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, > + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL > + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER > + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING > + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS > + * IN THE SOFTWARE. > + * > + */ > + > +#include "i915_drv.h" > +#include "intel_drv.h" > +#include "i915_scheduler.h" > + > +#ifdef CONFIG_DRM_I915_SCHEDULER > + > +int i915_scheduler_init(struct drm_device *dev) > +{ > + struct drm_i915_private *dev_priv = dev->dev_private; > + struct i915_scheduler *scheduler = dev_priv->scheduler; > + > + if (scheduler) > + return 0; > + > + scheduler = kzalloc(sizeof(*scheduler), GFP_KERNEL); > + if (!scheduler) > + return -ENOMEM; > + > + spin_lock_init(&scheduler->lock); > + > + scheduler->index = 1; > + > + dev_priv->scheduler = scheduler; > + > + return 0; > +} > + > +#else /* CONFIG_DRM_I915_SCHEDULER */ > + > +int i915_scheduler_init(struct drm_device *dev) > +{ > + return 0; > +} > + > +#endif /* CONFIG_DRM_I915_SCHEDULER */ > diff --git a/drivers/gpu/drm/i915/i915_scheduler.h b/drivers/gpu/drm/i915/i915_scheduler.h > new file mode 100644 > index 0000000..bbe1934 > --- /dev/null > +++ b/drivers/gpu/drm/i915/i915_scheduler.h > @@ -0,0 +1,40 @@ > +/* > + * Copyright (c) 2014 Intel Corporation > + * > + * Permission is hereby granted, free of charge, to any person obtaining a > + * copy of this software and associated documentation files (the "Software"), > + * to deal in the Software without restriction, including without limitation > + * the rights to use, copy, modify, merge, publish, distribute, sublicense, > + * and/or sell copies of the Software, and to permit persons to whom the > + * Software is furnished to do so, subject to the following conditions: > + * > + * The above copyright notice and this permission notice (including the next > + * paragraph) shall be included in all copies or substantial portions of the > + * Software. > + * > + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR > + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, > + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL > + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER > + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING > + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS > + * IN THE SOFTWARE. > + * > + */ > + > +#ifndef _I915_SCHEDULER_H_ > +#define _I915_SCHEDULER_H_ > + > +int i915_scheduler_init(struct drm_device *dev); > + > +#ifdef CONFIG_DRM_I915_SCHEDULER > + > +struct i915_scheduler { > + uint32_t flags[I915_NUM_RINGS]; > + spinlock_t lock; > + uint32_t index; > +}; > + > +#endif /* CONFIG_DRM_I915_SCHEDULER */ > + > +#endif /* _I915_SCHEDULER_H_ */ > -- > 1.7.9.5 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch