From: "Koenig, Christian" <Christian.Koenig@amd.com> To: Davidlohr Bueso <dave@stgolabs.net>, "akpm@linux-foundation.org" <akpm@linux-foundation.org> Cc: "walken@google.com" <walken@google.com>, "peterz@infradead.org" <peterz@infradead.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "linux-mm@kvack.org" <linux-mm@kvack.org>, "dri-devel@lists.freedesktop.org" <dri-devel@lists.freedesktop.org>, "linux-rdma@vger.kernel.org" <linux-rdma@vger.kernel.org>, Jerome Glisse <jglisse@redhat.com>, "Deucher, Alexander" <Alexander.Deucher@amd.com>, Daniel Vetter <daniel@ffwll.ch>, "amd-gfx@lists.freedesktop.org" <amd-gfx@lists.freedesktop.org>, Davidlohr Bueso <dbueso@suse.de> Subject: Re: [PATCH 03/11] drm/amdgpu: convert amdgpu_vm_it to half closed intervals Date: Fri, 4 Oct 2019 06:54:54 +0000 [thread overview] Message-ID: <dc9cc8c4-7275-43be-5bed-91384e3246ae@amd.com> (raw) In-Reply-To: <20191003201858.11666-4-dave@stgolabs.net> Am 03.10.19 um 22:18 schrieb Davidlohr Bueso: > The amdgpu_vm interval tree really wants [a, b) intervals, NAK, we explicitly do need an [a, b[ interval here. Regards, Christian. > not fully closed ones. As such convert it to use the new > interval_tree_gen.h, and also rename the 'last' endpoint > in the node to 'end', which is both a more suitable name > for the half closed interval and also reduces the chances > of missing a conversion when doing insertion or lookup. > > Cc: Jerome Glisse <jglisse@redhat.com> > Cc: Alex Deucher <alexander.deucher@amd.com> > Cc: "Christian König" <christian.koenig@amd.com> > Cc: Daniel Vetter <daniel@ffwll.ch> > Cc: amd-gfx@lists.freedesktop.org > Signed-off-by: Davidlohr Bueso <dbueso@suse.de> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 2 +- > drivers/gpu/drm/amd/amdgpu/amdgpu_object.h | 2 +- > drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h | 18 ++++++------ > drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c | 2 +- > drivers/gpu/drm/amd/amdgpu/amdgpu_vce.c | 3 +- > drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 46 +++++++++++++++--------------- > 6 files changed, 36 insertions(+), 37 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c > index 49b767b7238f..290bfe820890 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c > @@ -756,7 +756,7 @@ static int amdgpu_cs_vm_handling(struct amdgpu_cs_parser *p) > } > > if ((va_start + chunk_ib->ib_bytes) > > - (m->last + 1) * AMDGPU_GPU_PAGE_SIZE) { > + m->end * AMDGPU_GPU_PAGE_SIZE) { > DRM_ERROR("IB va_start+ib_bytes is invalid\n"); > return -EINVAL; > } > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.h > index 7e99f6c58c48..60b73bc4d11a 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.h > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.h > @@ -51,7 +51,7 @@ struct amdgpu_bo_va_mapping { > struct list_head list; > struct rb_node rb; > uint64_t start; > - uint64_t last; > + uint64_t end; > uint64_t __subtree_last; > uint64_t offset; > uint64_t flags; > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h > index 8227ebd0f511..c5b0e88d019c 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h > @@ -247,7 +247,7 @@ TRACE_EVENT(amdgpu_vm_bo_map, > TP_STRUCT__entry( > __field(struct amdgpu_bo *, bo) > __field(long, start) > - __field(long, last) > + __field(long, end) > __field(u64, offset) > __field(u64, flags) > ), > @@ -255,12 +255,12 @@ TRACE_EVENT(amdgpu_vm_bo_map, > TP_fast_assign( > __entry->bo = bo_va ? bo_va->base.bo : NULL; > __entry->start = mapping->start; > - __entry->last = mapping->last; > + __entry->end = mapping->end; > __entry->offset = mapping->offset; > __entry->flags = mapping->flags; > ), > - TP_printk("bo=%p, start=%lx, last=%lx, offset=%010llx, flags=%llx", > - __entry->bo, __entry->start, __entry->last, > + TP_printk("bo=%p, start=%lx, end=%lx, offset=%010llx, flags=%llx", > + __entry->bo, __entry->start, __entry->end, > __entry->offset, __entry->flags) > ); > > @@ -271,7 +271,7 @@ TRACE_EVENT(amdgpu_vm_bo_unmap, > TP_STRUCT__entry( > __field(struct amdgpu_bo *, bo) > __field(long, start) > - __field(long, last) > + __field(long, end) > __field(u64, offset) > __field(u64, flags) > ), > @@ -279,12 +279,12 @@ TRACE_EVENT(amdgpu_vm_bo_unmap, > TP_fast_assign( > __entry->bo = bo_va ? bo_va->base.bo : NULL; > __entry->start = mapping->start; > - __entry->last = mapping->last; > + __entry->end = mapping->end; > __entry->offset = mapping->offset; > __entry->flags = mapping->flags; > ), > - TP_printk("bo=%p, start=%lx, last=%lx, offset=%010llx, flags=%llx", > - __entry->bo, __entry->start, __entry->last, > + TP_printk("bo=%p, start=%lx, end=%lx, offset=%010llx, flags=%llx", > + __entry->bo, __entry->start, __entry->end, > __entry->offset, __entry->flags) > ); > > @@ -299,7 +299,7 @@ DECLARE_EVENT_CLASS(amdgpu_vm_mapping, > > TP_fast_assign( > __entry->soffset = mapping->start; > - __entry->eoffset = mapping->last + 1; > + __entry->eoffset = mapping->end; > __entry->flags = mapping->flags; > ), > TP_printk("soffs=%010llx, eoffs=%010llx, flags=%llx", > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c > index b2c364b8695f..8094dd0b0332 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c > @@ -819,7 +819,7 @@ static int amdgpu_uvd_cs_pass2(struct amdgpu_uvd_cs_ctx *ctx) > > start = amdgpu_bo_gpu_offset(bo); > > - end = (mapping->last + 1 - mapping->start); > + end = mapping->end - mapping->start; > end = end * AMDGPU_GPU_PAGE_SIZE + start; > > addr -= mapping->start * AMDGPU_GPU_PAGE_SIZE; > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vce.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vce.c > index b70b3c45bb29..d6511bf446df 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vce.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vce.c > @@ -642,8 +642,7 @@ static int amdgpu_vce_cs_reloc(struct amdgpu_cs_parser *p, uint32_t ib_idx, > return r; > } > > - if ((addr + (uint64_t)size) > > - (mapping->last + 1) * AMDGPU_GPU_PAGE_SIZE) { > + if ((addr + (uint64_t)size) > mapping->end * AMDGPU_GPU_PAGE_SIZE) { > DRM_ERROR("BO to small for addr 0x%010Lx %d %d\n", > addr, lo, hi); > return -EINVAL; > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c > index a2c797e34a29..2f618017617e 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c > @@ -26,7 +26,7 @@ > * Jerome Glisse > */ > #include <linux/dma-fence-array.h> > -#include <linux/interval_tree_generic.h> > +#include <linux/interval_tree_gen.h> > #include <linux/idr.h> > > #include <drm/amdgpu_drm.h> > @@ -58,13 +58,13 @@ > */ > > #define START(node) ((node)->start) > -#define LAST(node) ((node)->last) > +#define END(node) ((node)->end) > > INTERVAL_TREE_DEFINE(struct amdgpu_bo_va_mapping, rb, uint64_t, __subtree_last, > - START, LAST, static, amdgpu_vm_it) > + START, END, static, amdgpu_vm_it) > > #undef START > -#undef LAST > +#undef END > > /** > * struct amdgpu_prt_cb - Helper to disable partial resident texture feature from a fence callback > @@ -1616,7 +1616,7 @@ static int amdgpu_vm_bo_split_mapping(struct amdgpu_device *adev, > do { > dma_addr_t *dma_addr = NULL; > uint64_t max_entries; > - uint64_t addr, last; > + uint64_t addr, end; > > if (nodes) { > addr = nodes->start << PAGE_SHIFT; > @@ -1654,21 +1654,21 @@ static int amdgpu_vm_bo_split_mapping(struct amdgpu_device *adev, > addr += pfn << PAGE_SHIFT; > } > > - last = min((uint64_t)mapping->last, start + max_entries - 1); > + end = min((uint64_t)mapping->end, start + max_entries); > r = amdgpu_vm_bo_update_mapping(adev, vm, false, exclusive, > - start, last, flags, addr, > + start, end, flags, addr, > dma_addr, fence); > if (r) > return r; > > - pfn += (last - start + 1) / AMDGPU_GPU_PAGES_IN_CPU_PAGE; > + pfn += (end - start) / AMDGPU_GPU_PAGES_IN_CPU_PAGE; > if (nodes && nodes->size == pfn) { > pfn = 0; > ++nodes; > } > - start = last + 1; > + start = end; > > - } while (unlikely(start != mapping->last + 1)); > + } while (unlikely(start != mapping->end)); > > return 0; > } > @@ -1946,7 +1946,7 @@ int amdgpu_vm_clear_freed(struct amdgpu_device *adev, > init_pte_value = AMDGPU_PTE_DEFAULT_ATC; > > r = amdgpu_vm_bo_update_mapping(adev, vm, false, NULL, > - mapping->start, mapping->last, > + mapping->start, mapping->end, > init_pte_value, 0, NULL, &f); > amdgpu_vm_free_mapping(adev, vm, mapping, f); > if (r) { > @@ -2129,7 +2129,7 @@ int amdgpu_vm_bo_map(struct amdgpu_device *adev, > return -EINVAL; > > /* make sure object fit at this offset */ > - eaddr = saddr + size - 1; > + eaddr = saddr + size; > if (saddr >= eaddr || > (bo && offset + size > amdgpu_bo_size(bo))) > return -EINVAL; > @@ -2142,7 +2142,7 @@ int amdgpu_vm_bo_map(struct amdgpu_device *adev, > /* bo and tmp overlap, invalid addr */ > dev_err(adev->dev, "bo %p va 0x%010Lx-0x%010Lx conflict with " > "0x%010Lx-0x%010Lx\n", bo, saddr, eaddr, > - tmp->start, tmp->last + 1); > + tmp->start, tmp->end); > return -EINVAL; > } > > @@ -2151,7 +2151,7 @@ int amdgpu_vm_bo_map(struct amdgpu_device *adev, > return -ENOMEM; > > mapping->start = saddr; > - mapping->last = eaddr; > + mapping->end = eaddr; > mapping->offset = offset; > mapping->flags = flags; > > @@ -2194,7 +2194,7 @@ int amdgpu_vm_bo_replace_map(struct amdgpu_device *adev, > return -EINVAL; > > /* make sure object fit at this offset */ > - eaddr = saddr + size - 1; > + eaddr = saddr + size; > if (saddr >= eaddr || > (bo && offset + size > amdgpu_bo_size(bo))) > return -EINVAL; > @@ -2214,7 +2214,7 @@ int amdgpu_vm_bo_replace_map(struct amdgpu_device *adev, > eaddr /= AMDGPU_GPU_PAGE_SIZE; > > mapping->start = saddr; > - mapping->last = eaddr; > + mapping->end = eaddr; > mapping->offset = offset; > mapping->flags = flags; > > @@ -2299,7 +2299,7 @@ int amdgpu_vm_bo_clear_mappings(struct amdgpu_device *adev, > LIST_HEAD(removed); > uint64_t eaddr; > > - eaddr = saddr + size - 1; > + eaddr = saddr + size; > saddr /= AMDGPU_GPU_PAGE_SIZE; > eaddr /= AMDGPU_GPU_PAGE_SIZE; > > @@ -2322,7 +2322,7 @@ int amdgpu_vm_bo_clear_mappings(struct amdgpu_device *adev, > /* Remember mapping split at the start */ > if (tmp->start < saddr) { > before->start = tmp->start; > - before->last = saddr - 1; > + before->end = saddr; > before->offset = tmp->offset; > before->flags = tmp->flags; > before->bo_va = tmp->bo_va; > @@ -2330,9 +2330,9 @@ int amdgpu_vm_bo_clear_mappings(struct amdgpu_device *adev, > } > > /* Remember mapping split at the end */ > - if (tmp->last > eaddr) { > - after->start = eaddr + 1; > - after->last = tmp->last; > + if (tmp->end > eaddr) { > + after->start = eaddr; > + after->end = tmp->end; > after->offset = tmp->offset; > after->offset += after->start - tmp->start; > after->flags = tmp->flags; > @@ -2353,8 +2353,8 @@ int amdgpu_vm_bo_clear_mappings(struct amdgpu_device *adev, > > if (tmp->start < saddr) > tmp->start = saddr; > - if (tmp->last > eaddr) > - tmp->last = eaddr; > + if (tmp->end > eaddr) > + tmp->end = eaddr; > > tmp->bo_va = NULL; > list_add(&tmp->list, &vm->freed);
WARNING: multiple messages have this Message-ID (diff)
From: "Koenig, Christian" <Christian.Koenig-5C7GfCeVMHo@public.gmane.org> To: Davidlohr Bueso <dave-h16yJtLeMjHk1uMJSBkQmQ@public.gmane.org>, "akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org" <akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org> Cc: Davidlohr Bueso <dbueso-l3A5Bk7waGM@public.gmane.org>, "peterz-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org" <peterz-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org>, "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" <linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>, "dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org" <dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org>, "linux-mm-Bw31MaZKKs3YtjvyW6yDsg@public.gmane.org" <linux-mm-Bw31MaZKKs3YtjvyW6yDsg@public.gmane.org>, Jerome Glisse <jglisse-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>, "amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org" <amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org>, Daniel Vetter <daniel-/w4YWyX8dFk@public.gmane.org>, "Deucher, Alexander" <Alexander.Deucher-5C7GfCeVMHo@public.gmane.org>, "walken-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org" <walken-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org>, "linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" <linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org> Subject: Re: [PATCH 03/11] drm/amdgpu: convert amdgpu_vm_it to half closed intervals Date: Fri, 4 Oct 2019 06:54:54 +0000 [thread overview] Message-ID: <dc9cc8c4-7275-43be-5bed-91384e3246ae@amd.com> (raw) In-Reply-To: <20191003201858.11666-4-dave-h16yJtLeMjHk1uMJSBkQmQ@public.gmane.org> Am 03.10.19 um 22:18 schrieb Davidlohr Bueso: > The amdgpu_vm interval tree really wants [a, b) intervals, NAK, we explicitly do need an [a, b[ interval here. Regards, Christian. > not fully closed ones. As such convert it to use the new > interval_tree_gen.h, and also rename the 'last' endpoint > in the node to 'end', which is both a more suitable name > for the half closed interval and also reduces the chances > of missing a conversion when doing insertion or lookup. > > Cc: Jerome Glisse <jglisse@redhat.com> > Cc: Alex Deucher <alexander.deucher@amd.com> > Cc: "Christian König" <christian.koenig@amd.com> > Cc: Daniel Vetter <daniel@ffwll.ch> > Cc: amd-gfx@lists.freedesktop.org > Signed-off-by: Davidlohr Bueso <dbueso@suse.de> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 2 +- > drivers/gpu/drm/amd/amdgpu/amdgpu_object.h | 2 +- > drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h | 18 ++++++------ > drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c | 2 +- > drivers/gpu/drm/amd/amdgpu/amdgpu_vce.c | 3 +- > drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 46 +++++++++++++++--------------- > 6 files changed, 36 insertions(+), 37 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c > index 49b767b7238f..290bfe820890 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c > @@ -756,7 +756,7 @@ static int amdgpu_cs_vm_handling(struct amdgpu_cs_parser *p) > } > > if ((va_start + chunk_ib->ib_bytes) > > - (m->last + 1) * AMDGPU_GPU_PAGE_SIZE) { > + m->end * AMDGPU_GPU_PAGE_SIZE) { > DRM_ERROR("IB va_start+ib_bytes is invalid\n"); > return -EINVAL; > } > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.h > index 7e99f6c58c48..60b73bc4d11a 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.h > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.h > @@ -51,7 +51,7 @@ struct amdgpu_bo_va_mapping { > struct list_head list; > struct rb_node rb; > uint64_t start; > - uint64_t last; > + uint64_t end; > uint64_t __subtree_last; > uint64_t offset; > uint64_t flags; > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h > index 8227ebd0f511..c5b0e88d019c 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h > @@ -247,7 +247,7 @@ TRACE_EVENT(amdgpu_vm_bo_map, > TP_STRUCT__entry( > __field(struct amdgpu_bo *, bo) > __field(long, start) > - __field(long, last) > + __field(long, end) > __field(u64, offset) > __field(u64, flags) > ), > @@ -255,12 +255,12 @@ TRACE_EVENT(amdgpu_vm_bo_map, > TP_fast_assign( > __entry->bo = bo_va ? bo_va->base.bo : NULL; > __entry->start = mapping->start; > - __entry->last = mapping->last; > + __entry->end = mapping->end; > __entry->offset = mapping->offset; > __entry->flags = mapping->flags; > ), > - TP_printk("bo=%p, start=%lx, last=%lx, offset=%010llx, flags=%llx", > - __entry->bo, __entry->start, __entry->last, > + TP_printk("bo=%p, start=%lx, end=%lx, offset=%010llx, flags=%llx", > + __entry->bo, __entry->start, __entry->end, > __entry->offset, __entry->flags) > ); > > @@ -271,7 +271,7 @@ TRACE_EVENT(amdgpu_vm_bo_unmap, > TP_STRUCT__entry( > __field(struct amdgpu_bo *, bo) > __field(long, start) > - __field(long, last) > + __field(long, end) > __field(u64, offset) > __field(u64, flags) > ), > @@ -279,12 +279,12 @@ TRACE_EVENT(amdgpu_vm_bo_unmap, > TP_fast_assign( > __entry->bo = bo_va ? bo_va->base.bo : NULL; > __entry->start = mapping->start; > - __entry->last = mapping->last; > + __entry->end = mapping->end; > __entry->offset = mapping->offset; > __entry->flags = mapping->flags; > ), > - TP_printk("bo=%p, start=%lx, last=%lx, offset=%010llx, flags=%llx", > - __entry->bo, __entry->start, __entry->last, > + TP_printk("bo=%p, start=%lx, end=%lx, offset=%010llx, flags=%llx", > + __entry->bo, __entry->start, __entry->end, > __entry->offset, __entry->flags) > ); > > @@ -299,7 +299,7 @@ DECLARE_EVENT_CLASS(amdgpu_vm_mapping, > > TP_fast_assign( > __entry->soffset = mapping->start; > - __entry->eoffset = mapping->last + 1; > + __entry->eoffset = mapping->end; > __entry->flags = mapping->flags; > ), > TP_printk("soffs=%010llx, eoffs=%010llx, flags=%llx", > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c > index b2c364b8695f..8094dd0b0332 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c > @@ -819,7 +819,7 @@ static int amdgpu_uvd_cs_pass2(struct amdgpu_uvd_cs_ctx *ctx) > > start = amdgpu_bo_gpu_offset(bo); > > - end = (mapping->last + 1 - mapping->start); > + end = mapping->end - mapping->start; > end = end * AMDGPU_GPU_PAGE_SIZE + start; > > addr -= mapping->start * AMDGPU_GPU_PAGE_SIZE; > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vce.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vce.c > index b70b3c45bb29..d6511bf446df 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vce.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vce.c > @@ -642,8 +642,7 @@ static int amdgpu_vce_cs_reloc(struct amdgpu_cs_parser *p, uint32_t ib_idx, > return r; > } > > - if ((addr + (uint64_t)size) > > - (mapping->last + 1) * AMDGPU_GPU_PAGE_SIZE) { > + if ((addr + (uint64_t)size) > mapping->end * AMDGPU_GPU_PAGE_SIZE) { > DRM_ERROR("BO to small for addr 0x%010Lx %d %d\n", > addr, lo, hi); > return -EINVAL; > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c > index a2c797e34a29..2f618017617e 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c > @@ -26,7 +26,7 @@ > * Jerome Glisse > */ > #include <linux/dma-fence-array.h> > -#include <linux/interval_tree_generic.h> > +#include <linux/interval_tree_gen.h> > #include <linux/idr.h> > > #include <drm/amdgpu_drm.h> > @@ -58,13 +58,13 @@ > */ > > #define START(node) ((node)->start) > -#define LAST(node) ((node)->last) > +#define END(node) ((node)->end) > > INTERVAL_TREE_DEFINE(struct amdgpu_bo_va_mapping, rb, uint64_t, __subtree_last, > - START, LAST, static, amdgpu_vm_it) > + START, END, static, amdgpu_vm_it) > > #undef START > -#undef LAST > +#undef END > > /** > * struct amdgpu_prt_cb - Helper to disable partial resident texture feature from a fence callback > @@ -1616,7 +1616,7 @@ static int amdgpu_vm_bo_split_mapping(struct amdgpu_device *adev, > do { > dma_addr_t *dma_addr = NULL; > uint64_t max_entries; > - uint64_t addr, last; > + uint64_t addr, end; > > if (nodes) { > addr = nodes->start << PAGE_SHIFT; > @@ -1654,21 +1654,21 @@ static int amdgpu_vm_bo_split_mapping(struct amdgpu_device *adev, > addr += pfn << PAGE_SHIFT; > } > > - last = min((uint64_t)mapping->last, start + max_entries - 1); > + end = min((uint64_t)mapping->end, start + max_entries); > r = amdgpu_vm_bo_update_mapping(adev, vm, false, exclusive, > - start, last, flags, addr, > + start, end, flags, addr, > dma_addr, fence); > if (r) > return r; > > - pfn += (last - start + 1) / AMDGPU_GPU_PAGES_IN_CPU_PAGE; > + pfn += (end - start) / AMDGPU_GPU_PAGES_IN_CPU_PAGE; > if (nodes && nodes->size == pfn) { > pfn = 0; > ++nodes; > } > - start = last + 1; > + start = end; > > - } while (unlikely(start != mapping->last + 1)); > + } while (unlikely(start != mapping->end)); > > return 0; > } > @@ -1946,7 +1946,7 @@ int amdgpu_vm_clear_freed(struct amdgpu_device *adev, > init_pte_value = AMDGPU_PTE_DEFAULT_ATC; > > r = amdgpu_vm_bo_update_mapping(adev, vm, false, NULL, > - mapping->start, mapping->last, > + mapping->start, mapping->end, > init_pte_value, 0, NULL, &f); > amdgpu_vm_free_mapping(adev, vm, mapping, f); > if (r) { > @@ -2129,7 +2129,7 @@ int amdgpu_vm_bo_map(struct amdgpu_device *adev, > return -EINVAL; > > /* make sure object fit at this offset */ > - eaddr = saddr + size - 1; > + eaddr = saddr + size; > if (saddr >= eaddr || > (bo && offset + size > amdgpu_bo_size(bo))) > return -EINVAL; > @@ -2142,7 +2142,7 @@ int amdgpu_vm_bo_map(struct amdgpu_device *adev, > /* bo and tmp overlap, invalid addr */ > dev_err(adev->dev, "bo %p va 0x%010Lx-0x%010Lx conflict with " > "0x%010Lx-0x%010Lx\n", bo, saddr, eaddr, > - tmp->start, tmp->last + 1); > + tmp->start, tmp->end); > return -EINVAL; > } > > @@ -2151,7 +2151,7 @@ int amdgpu_vm_bo_map(struct amdgpu_device *adev, > return -ENOMEM; > > mapping->start = saddr; > - mapping->last = eaddr; > + mapping->end = eaddr; > mapping->offset = offset; > mapping->flags = flags; > > @@ -2194,7 +2194,7 @@ int amdgpu_vm_bo_replace_map(struct amdgpu_device *adev, > return -EINVAL; > > /* make sure object fit at this offset */ > - eaddr = saddr + size - 1; > + eaddr = saddr + size; > if (saddr >= eaddr || > (bo && offset + size > amdgpu_bo_size(bo))) > return -EINVAL; > @@ -2214,7 +2214,7 @@ int amdgpu_vm_bo_replace_map(struct amdgpu_device *adev, > eaddr /= AMDGPU_GPU_PAGE_SIZE; > > mapping->start = saddr; > - mapping->last = eaddr; > + mapping->end = eaddr; > mapping->offset = offset; > mapping->flags = flags; > > @@ -2299,7 +2299,7 @@ int amdgpu_vm_bo_clear_mappings(struct amdgpu_device *adev, > LIST_HEAD(removed); > uint64_t eaddr; > > - eaddr = saddr + size - 1; > + eaddr = saddr + size; > saddr /= AMDGPU_GPU_PAGE_SIZE; > eaddr /= AMDGPU_GPU_PAGE_SIZE; > > @@ -2322,7 +2322,7 @@ int amdgpu_vm_bo_clear_mappings(struct amdgpu_device *adev, > /* Remember mapping split at the start */ > if (tmp->start < saddr) { > before->start = tmp->start; > - before->last = saddr - 1; > + before->end = saddr; > before->offset = tmp->offset; > before->flags = tmp->flags; > before->bo_va = tmp->bo_va; > @@ -2330,9 +2330,9 @@ int amdgpu_vm_bo_clear_mappings(struct amdgpu_device *adev, > } > > /* Remember mapping split at the end */ > - if (tmp->last > eaddr) { > - after->start = eaddr + 1; > - after->last = tmp->last; > + if (tmp->end > eaddr) { > + after->start = eaddr; > + after->end = tmp->end; > after->offset = tmp->offset; > after->offset += after->start - tmp->start; > after->flags = tmp->flags; > @@ -2353,8 +2353,8 @@ int amdgpu_vm_bo_clear_mappings(struct amdgpu_device *adev, > > if (tmp->start < saddr) > tmp->start = saddr; > - if (tmp->last > eaddr) > - tmp->last = eaddr; > + if (tmp->end > eaddr) > + tmp->end = eaddr; > > tmp->bo_va = NULL; > list_add(&tmp->list, &vm->freed); _______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
next prev parent reply other threads:[~2019-10-04 6:55 UTC|newest] Thread overview: 51+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-10-03 20:18 [PATCH -next 00/11] lib/interval-tree: move to half closed intervals Davidlohr Bueso 2019-10-03 20:18 ` [PATCH 01/11] mm: introduce vma_interval_tree_foreach_stab() Davidlohr Bueso 2019-10-03 20:18 ` [PATCH 02/11] lib/interval-tree: add an equivalent tree with [a,b) intervals Davidlohr Bueso 2019-10-04 11:02 ` Michel Lespinasse 2019-10-03 20:18 ` [PATCH 03/11] drm/amdgpu: convert amdgpu_vm_it to half closed intervals Davidlohr Bueso 2019-10-04 6:54 ` Koenig, Christian [this message] 2019-10-04 6:54 ` Koenig, Christian 2019-10-04 11:36 ` Michel Lespinasse 2019-10-04 12:39 ` Christian König 2019-10-03 20:18 ` [PATCH 04/11] drm: convert drm_mm_interval_tree " Davidlohr Bueso 2019-10-03 20:18 ` Davidlohr Bueso 2019-10-03 20:18 ` [PATCH 05/11] IB/hfi1: convert __mmu_int_rb " Davidlohr Bueso 2019-10-04 11:50 ` Michel Lespinasse 2019-10-04 19:41 ` Davidlohr Bueso 2019-10-03 20:18 ` [PATCH 06/11] IB,usnic: convert usnic_uiom_interval_tree " Davidlohr Bueso 2019-10-03 20:18 ` [PATCH 07/11] vhost: convert vhost_umem_interval_tree " Davidlohr Bueso 2019-10-03 20:18 ` Davidlohr Bueso 2019-10-04 12:10 ` Michel Lespinasse 2019-10-04 19:44 ` Davidlohr Bueso 2019-10-04 19:44 ` Davidlohr Bueso 2019-10-10 5:49 ` Jason Wang 2019-10-10 5:49 ` Jason Wang 2019-10-10 5:49 ` Jason Wang 2019-10-03 20:18 ` Davidlohr Bueso 2019-10-03 20:18 ` [PATCH 08/11] mm: convert vma_interval_tree " Davidlohr Bueso 2019-10-03 20:41 ` Matthew Wilcox 2019-10-04 12:30 ` Michel Lespinasse 2019-10-03 20:18 ` [PATCH 09/11] lib/interval-tree: convert interval_tree " Davidlohr Bueso 2019-10-03 22:50 ` kbuild test robot 2019-10-03 22:50 ` kbuild test robot 2019-10-03 22:50 ` kbuild test robot 2019-10-04 6:57 ` Koenig, Christian 2019-10-04 6:57 ` Koenig, Christian 2019-10-04 7:20 ` Koenig, Christian 2019-10-04 7:20 ` Koenig, Christian 2019-10-08 16:59 ` Davidlohr Bueso 2019-10-03 20:18 ` [PATCH 10/11] lib: drop interval_tree_generic.h Davidlohr Bueso 2019-10-03 20:18 ` [PATCH 11/11] x86/mm, pat: convert pat tree to generic interval tree Davidlohr Bueso 2019-10-07 15:33 ` Ingo Molnar 2019-10-07 15:33 ` Ingo Molnar 2019-10-21 23:24 ` Davidlohr Bueso 2019-10-03 20:32 ` [PATCH -next 00/11] lib/interval-tree: move to half closed intervals Matthew Wilcox 2019-10-03 21:10 ` Davidlohr Bueso 2019-10-04 12:43 ` Michel Lespinasse 2019-10-04 0:26 ` Jason Gunthorpe 2019-10-04 2:48 ` Davidlohr Bueso 2019-10-04 13:15 ` Michel Lespinasse 2019-10-04 13:15 ` Michel Lespinasse 2019-10-04 16:03 ` Matthew Wilcox 2019-10-04 19:35 ` Davidlohr Bueso 2019-10-04 17:45 ` Jason Gunthorpe
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=dc9cc8c4-7275-43be-5bed-91384e3246ae@amd.com \ --to=christian.koenig@amd.com \ --cc=Alexander.Deucher@amd.com \ --cc=akpm@linux-foundation.org \ --cc=amd-gfx@lists.freedesktop.org \ --cc=daniel@ffwll.ch \ --cc=dave@stgolabs.net \ --cc=dbueso@suse.de \ --cc=dri-devel@lists.freedesktop.org \ --cc=jglisse@redhat.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mm@kvack.org \ --cc=linux-rdma@vger.kernel.org \ --cc=peterz@infradead.org \ --cc=walken@google.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.