On Thu, Sep 29, 2016 at 04:35:51PM +0100, Tvrtko Ursulin wrote:
> From: Tvrtko Ursulin <[email protected]>
>
> If we store this in the uncore structure we are on a good way to
> show more commonality between the per-platform implementations.
>
> Signed-off-by: Tvrtko Ursulin <[email protected]>
> ---
> drivers/gpu/drm/i915/i915_drv.h | 11 ++++++
> drivers/gpu/drm/i915/intel_uncore.c | 74
> ++++++++++++++++++-------------------
> 2 files changed, 47 insertions(+), 38 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
> index 4cd727376d1d..7c8bd07767fe 100644
> --- a/drivers/gpu/drm/i915/i915_drv.h
> +++ b/drivers/gpu/drm/i915/i915_drv.h
> @@ -582,9 +582,20 @@ struct intel_uncore_funcs {
> uint32_t val, bool trace);
> };
>
> +struct intel_forcewake_range
> +{
struct intel_forcewake_range {
> + u32 start;
> + u32 end;
> +
> + enum forcewake_domains domains;
> +};
> +
> struct intel_uncore {
> spinlock_t lock; /** lock is also taken in irq contexts. */
>
> + struct intel_forcewake_range *fw_domains_table;
> + unsigned int fw_domains_table_entries;
constify?
> +
> struct intel_uncore_funcs funcs;
>
> unsigned fifo_count;
> diff --git a/drivers/gpu/drm/i915/intel_uncore.c
> b/drivers/gpu/drm/i915/intel_uncore.c
> index a41a6a2019be..2197c0a43dda 100644
> --- a/drivers/gpu/drm/i915/intel_uncore.c
> +++ b/drivers/gpu/drm/i915/intel_uncore.c
> @@ -582,14 +582,6 @@ void assert_forcewakes_inactive(struct drm_i915_private
> *dev_priv)
> __fwd; \
> })
>
> -struct intel_forcewake_range
> -{
> - u32 start;
> - u32 end;
> -
> - enum forcewake_domains domains;
> -};
> -
> static int fw_range_cmp(const void *key, const void *elt)
> {
> struct intel_forcewake_range *entry =
> @@ -605,28 +597,36 @@ static int fw_range_cmp(const void *key, const void
> *elt)
> }
>
> static enum forcewake_domains
> -find_fw_domain(u32 offset, const struct intel_forcewake_range *ranges,
> - unsigned int num_ranges)
> +find_fw_domain(struct drm_i915_private *dev_priv, u32 offset)
> {
> - struct intel_forcewake_range *entry;
> + struct intel_forcewake_range *table, *entry;
> + unsigned int num_entries;
>
> - entry = bsearch((void *)(unsigned long)offset, (const void *)ranges,
> - num_ranges, sizeof(struct intel_forcewake_range),
> + table = dev_priv->uncore.fw_domains_table;
> + num_entries = dev_priv->uncore.fw_domains_table_entries;
> +
> + entry = bsearch((void *)(unsigned long)offset, (const void *)table,
> + num_entries, sizeof(struct intel_forcewake_range),
> fw_range_cmp);
>
> return entry ? entry->domains : 0;
> }
>
> static void
> -intel_fw_table_check(const struct intel_forcewake_range *ranges,
> - unsigned int num_ranges)
> +intel_fw_table_check(struct drm_i915_private *dev_priv)
> {
> unsigned int i;
> - struct intel_forcewake_range *entry =
> - (struct intel_forcewake_range *)ranges;
> - s32 prev = -1;
> + struct intel_forcewake_range *entry;
> + unsigned int num_ranges;
> + s32 prev;
>
> - for (i = 0; i < num_ranges; i++, entry++) {
> + entry = dev_priv->uncore.fw_domains_table;
> + if (!entry)
> + return;
> +
> + num_ranges = dev_priv->uncore.fw_domains_table_entries;
> +
> + for (i = 0, prev = -1; i < num_ranges; i++, entry++) {
> WARN_ON_ONCE(prev >= (s32)entry->start);
> prev = entry->start;
> WARN_ON_ONCE(prev >= (s32)entry->end);
> @@ -652,8 +652,7 @@ static const struct intel_forcewake_range
> __vlv_fw_ranges[] = {
> ({ \
> enum forcewake_domains __fwd = 0; \
> if (NEEDS_FORCE_WAKE((offset))) \
> - __fwd = find_fw_domain(offset, __vlv_fw_ranges, \
> - ARRAY_SIZE(__vlv_fw_ranges)); \
> + __fwd = find_fw_domain(dev_priv, offset); \
> __fwd; \
> })
>
> @@ -711,8 +710,7 @@ static const struct intel_forcewake_range
> __chv_fw_ranges[] = {
> ({ \
> enum forcewake_domains __fwd = 0; \
> if (NEEDS_FORCE_WAKE((offset))) \
> - __fwd = find_fw_domain(offset, __chv_fw_ranges, \
> - ARRAY_SIZE(__chv_fw_ranges)); \
> + __fwd = find_fw_domain(dev_priv, offset); \
> __fwd; \
> })
>
> @@ -720,8 +718,7 @@ static const struct intel_forcewake_range
> __chv_fw_ranges[] = {
> ({ \
> enum forcewake_domains __fwd = 0; \
> if (NEEDS_FORCE_WAKE((offset)) && !is_gen8_shadowed(offset)) \
> - __fwd = find_fw_domain(offset, __chv_fw_ranges, \
> - ARRAY_SIZE(__chv_fw_ranges)); \
> + __fwd = find_fw_domain(dev_priv, offset); \
> __fwd; \
> })
>
> @@ -765,8 +762,7 @@ static const struct intel_forcewake_range
> __gen9_fw_ranges[] = {
> ({ \
> enum forcewake_domains __fwd = 0; \
> if (NEEDS_FORCE_WAKE((offset))) \
> - __fwd = find_fw_domain(offset, __gen9_fw_ranges, \
> - ARRAY_SIZE(__gen9_fw_ranges)); \
> + __fwd = find_fw_domain(dev_priv, offset); \
> __fwd; \
> })
>
> @@ -794,8 +790,7 @@ static bool is_gen9_shadowed(u32 offset)
> ({ \
> enum forcewake_domains __fwd = 0; \
> if (NEEDS_FORCE_WAKE((offset)) && !is_gen9_shadowed(offset)) \
> - __fwd = find_fw_domain(offset, __gen9_fw_ranges, \
> - ARRAY_SIZE(__gen9_fw_ranges)); \
> + __fwd = find_fw_domain(dev_priv, offset); \
> __fwd; \
> })
>
> @@ -1318,6 +1313,13 @@ static void intel_uncore_fw_domains_init(struct
> drm_i915_private *dev_priv)
> WARN_ON(dev_priv->uncore.fw_domains == 0);
> }
>
> +#define ASSIGN_FW_DOMAINS_TABLE(d) \
> +{ \
> + dev_priv->uncore.fw_domains_table = \
> + (struct intel_forcewake_range *)(d); \
> + dev_priv->uncore.fw_domains_table_entries = ARRAY_SIZE((d)); \
> +}
> +
> void intel_uncore_init(struct drm_i915_private *dev_priv)
> {
> i915_check_vgpu(dev_priv);
> @@ -1331,17 +1333,13 @@ void intel_uncore_init(struct drm_i915_private
> *dev_priv)
> switch (INTEL_INFO(dev_priv)->gen) {
> default:
> case 9:
> - intel_fw_table_check(__gen9_fw_ranges,
> - ARRAY_SIZE(__gen9_fw_ranges));
> -
> + ASSIGN_FW_DOMAINS_TABLE(__gen9_fw_ranges);
> ASSIGN_WRITE_MMIO_VFUNCS(gen9);
> ASSIGN_READ_MMIO_VFUNCS(gen9);
> break;
> case 8:
> if (IS_CHERRYVIEW(dev_priv)) {
> - intel_fw_table_check(__chv_fw_ranges,
> - ARRAY_SIZE(__chv_fw_ranges));
> -
> + ASSIGN_FW_DOMAINS_TABLE(__chv_fw_ranges);
> ASSIGN_WRITE_MMIO_VFUNCS(chv);
> ASSIGN_READ_MMIO_VFUNCS(chv);
>
> @@ -1355,9 +1353,7 @@ void intel_uncore_init(struct drm_i915_private
> *dev_priv)
> ASSIGN_WRITE_MMIO_VFUNCS(gen6);
>
> if (IS_VALLEYVIEW(dev_priv)) {
> - intel_fw_table_check(__vlv_fw_ranges,
> - ARRAY_SIZE(__vlv_fw_ranges));
> -
> + ASSIGN_FW_DOMAINS_TABLE(__vlv_fw_ranges);
> ASSIGN_READ_MMIO_VFUNCS(vlv);
> } else {
> ASSIGN_READ_MMIO_VFUNCS(gen6);
> @@ -1375,6 +1371,8 @@ void intel_uncore_init(struct drm_i915_private
> *dev_priv)
> break;
> }
>
> + intel_fw_table_check(dev_priv);
Looking better...
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
