| Message ID | 20211208145800.27246-2-ramalingam.c@intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | drm/i915: Sanity Check for device memory region | expand |
On Wed, Dec 08, 2021 at 08:27:58PM +0530, Ramalingam C wrote: > From: Chris Wilson <chris@chris-wilson.co.uk> > > Remove the portion of stolen memory reserved for private use from driver > access. > > Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> > cc: Matthew Auld <matthew.auld@intel.com> > Signed-off-by: Ramalingam C <ramalingam.c@intel.com> > Reviewed-by: Matthew Auld <matthew.auld@intel.com> > --- > drivers/gpu/drm/i915/gem/i915_gem_stolen.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/gpu/drm/i915/gem/i915_gem_stolen.c b/drivers/gpu/drm/i915/gem/i915_gem_stolen.c > index bce03d74a0b4..6ea3ca21cdf3 100644 > --- a/drivers/gpu/drm/i915/gem/i915_gem_stolen.c > +++ b/drivers/gpu/drm/i915/gem/i915_gem_stolen.c > @@ -488,6 +488,9 @@ static int i915_gem_init_stolen(struct intel_memory_region *mem) > return 0; > } > > + /* Exclude the reserved region from driver use */ > + mem->region.end = reserved_base - 1; > + > /* It is possible for the reserved area to end before the end of stolen > * memory, so just consider the start. */ > reserved_total = stolen_top - reserved_base; > -- > 2.20.1
Hi Ram, On Wed, Dec 08, 2021 at 08:27:58PM +0530, Ramalingam C wrote: > From: Chris Wilson <chris@chris-wilson.co.uk> > > Remove the portion of stolen memory reserved for private use from driver > access. > > Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> > cc: Matthew Auld <matthew.auld@intel.com> > Signed-off-by: Ramalingam C <ramalingam.c@intel.com> > Reviewed-by: Matthew Auld <matthew.auld@intel.com> Reviewed-by: Andi Shyti <andi.shyti@linux.intel.com> Thanks, Andi
diff --git a/drivers/gpu/drm/i915/gem/i915_gem_stolen.c b/drivers/gpu/drm/i915/gem/i915_gem_stolen.c index bce03d74a0b4..6ea3ca21cdf3 100644 --- a/drivers/gpu/drm/i915/gem/i915_gem_stolen.c +++ b/drivers/gpu/drm/i915/gem/i915_gem_stolen.c @@ -488,6 +488,9 @@ static int i915_gem_init_stolen(struct intel_memory_region *mem) return 0; } + /* Exclude the reserved region from driver use */ + mem->region.end = reserved_base - 1; + /* It is possible for the reserved area to end before the end of stolen * memory, so just consider the start. */ reserved_total = stolen_top - reserved_base;