Skip to content

Commit d09aa85

Browse files
committed
drm/i915: move i915_coherent_map_type() to i915_gem_pages.c and un-inline
The inline function has no place in i915_drv.h. Move it away, un-inline, and untangle some header dependencies while at it. Cc: Matthew Auld <matthew.auld@intel.com> Cc: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com> Signed-off-by: Jani Nikula <jani.nikula@intel.com> Acked-by: Matthew Auld <matthew.auld@intel.com> Link: https://patchwork.freedesktop.org/patch/msgid/20220914163514.1837467-1-jani.nikula@intel.com
1 parent 45810b4 commit d09aa85

File tree

11 files changed

+24
-13
lines changed

11 files changed

+24
-13
lines changed

drivers/gpu/drm/i915/display/intel_dpt.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55

66
#include "gem/i915_gem_domain.h"
77
#include "gem/i915_gem_internal.h"
8+
#include "gem/i915_gem_lmem.h"
89
#include "gt/gen8_ppgtt.h"
910

1011
#include "i915_drv.h"

drivers/gpu/drm/i915/gem/i915_gem_object.h

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -482,6 +482,10 @@ void *__must_check i915_gem_object_pin_map(struct drm_i915_gem_object *obj,
482482
void *__must_check i915_gem_object_pin_map_unlocked(struct drm_i915_gem_object *obj,
483483
enum i915_map_type type);
484484

485+
enum i915_map_type i915_coherent_map_type(struct drm_i915_private *i915,
486+
struct drm_i915_gem_object *obj,
487+
bool always_coherent);
488+
485489
void __i915_gem_object_flush_map(struct drm_i915_gem_object *obj,
486490
unsigned long offset,
487491
unsigned long size);

drivers/gpu/drm/i915/gem/i915_gem_pages.c

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -466,6 +466,18 @@ void *i915_gem_object_pin_map_unlocked(struct drm_i915_gem_object *obj,
466466
return ret;
467467
}
468468

469+
enum i915_map_type i915_coherent_map_type(struct drm_i915_private *i915,
470+
struct drm_i915_gem_object *obj,
471+
bool always_coherent)
472+
{
473+
if (i915_gem_object_is_lmem(obj))
474+
return I915_MAP_WC;
475+
if (HAS_LLC(i915) || always_coherent)
476+
return I915_MAP_WB;
477+
else
478+
return I915_MAP_WC;
479+
}
480+
469481
void __i915_gem_object_flush_map(struct drm_i915_gem_object *obj,
470482
unsigned long offset,
471483
unsigned long size)

drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88
#include <linux/prime_numbers.h>
99

1010
#include "gem/i915_gem_internal.h"
11+
#include "gem/i915_gem_lmem.h"
1112
#include "gem/i915_gem_region.h"
1213
#include "gem/i915_gem_ttm.h"
1314
#include "gem/i915_gem_ttm_move.h"

drivers/gpu/drm/i915/gt/intel_gsc.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
#include <linux/mei_aux.h>
88
#include "i915_drv.h"
99
#include "i915_reg.h"
10+
#include "gem/i915_gem_lmem.h"
1011
#include "gem/i915_gem_region.h"
1112
#include "gt/intel_gsc.h"
1213
#include "gt/intel_gt.h"

drivers/gpu/drm/i915/gt/intel_migrate.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010
#include "intel_gtt.h"
1111
#include "intel_migrate.h"
1212
#include "intel_ring.h"
13+
#include "gem/i915_gem_lmem.h"
1314

1415
struct insert_pte_data {
1516
u64 offset;

drivers/gpu/drm/i915/gt/selftest_migrate.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
#include <linux/sort.h>
77

88
#include "gem/i915_gem_internal.h"
9+
#include "gem/i915_gem_lmem.h"
910

1011
#include "selftests/i915_random.h"
1112

drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55

66
#include <linux/bsearch.h>
77

8+
#include "gem/i915_gem_lmem.h"
89
#include "gt/intel_engine_regs.h"
910
#include "gt/intel_gt.h"
1011
#include "gt/intel_gt_mcr.h"

drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
#include <linux/circ_buf.h>
77

88
#include "gem/i915_gem_context.h"
9+
#include "gem/i915_gem_lmem.h"
910
#include "gt/gen8_engine_cs.h"
1011
#include "gt/intel_breadcrumbs.h"
1112
#include "gt/intel_context.h"

drivers/gpu/drm/i915/i915_drv.h

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,6 @@
5151
#include "display/intel_opregion.h"
5252

5353
#include "gem/i915_gem_context_types.h"
54-
#include "gem/i915_gem_lmem.h"
5554
#include "gem/i915_gem_shrinker.h"
5655
#include "gem/i915_gem_stolen.h"
5756

@@ -1508,16 +1507,4 @@ mkwrite_device_info(struct drm_i915_private *dev_priv)
15081507
return (struct intel_device_info *)INTEL_INFO(dev_priv);
15091508
}
15101509

1511-
static inline enum i915_map_type
1512-
i915_coherent_map_type(struct drm_i915_private *i915,
1513-
struct drm_i915_gem_object *obj, bool always_coherent)
1514-
{
1515-
if (i915_gem_object_is_lmem(obj))
1516-
return I915_MAP_WC;
1517-
if (HAS_LLC(i915) || always_coherent)
1518-
return I915_MAP_WB;
1519-
else
1520-
return I915_MAP_WC;
1521-
}
1522-
15231510
#endif

0 commit comments

Comments
 (0)