diff options
author | Alex Waterman <alexw@nvidia.com> | 2017-04-17 19:26:28 -0400 |
---|---|---|
committer | mobile promotions <svcmobile_promotions@nvidia.com> | 2017-05-19 18:34:01 -0400 |
commit | d37e8f7dcf190f31f9c0c12583db2bb0c0d313c0 (patch) | |
tree | 4807c89bf40954c54804c3a8dd88c16849181f29 /drivers/gpu/nvgpu/gk20a/cde_gk20a.c | |
parent | dcb744acfbbc11e66cac2d0a674a42e62d908b9d (diff) |
gpu: nvgpu: Split VM interface out
This patch begins the major rework of the GPU's virtual memory manager
(VMM). The VMM is the piece of code that handles the userspace interface
to buffers and their mappings into the GMMU. The core data structure is
the VM - for now still known as 'struct vm_gk20a'. Each one of these
structs represents one addres space to which channels or TSGs may bind
themselves to.
The VMM splits the interface up into two broad categories. First there's
the common, OS independent interfaces; and second there's the OS specific
interfaces.
OS independent
--------------
This is the code that manages the lifetime of VMs, the buffers inside
VMs (search, batch mapping) creation, destruction, etc.
OS Specific
-----------
This handles mapping of buffers represented as they are represented by
the OS (dma_buf's for example on Linux).
This patch is by no means complete. There's still Linux specific functions
scattered in ostensibly OS independent code. This is the first step. A
patch that rewrites everything in one go would simply be too big to
effectively review.
Instead the goal of this change is to simply separate out the basic
OS specific and OS agnostic interfaces into their own header files. The
next series of patches will start to pull the relevant implementations
into OS specific C files and common C files.
JIRA NVGPU-12
JIRA NVGPU-30
Change-Id: I242c7206047b6c769296226d855b7e44d5c4bfa8
Signed-off-by: Alex Waterman <alexw@nvidia.com>
Reviewed-on: http://git-master/r/1464939
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
Diffstat (limited to 'drivers/gpu/nvgpu/gk20a/cde_gk20a.c')
-rw-r--r-- | drivers/gpu/nvgpu/gk20a/cde_gk20a.c | 16 |
1 files changed, 11 insertions, 5 deletions
diff --git a/drivers/gpu/nvgpu/gk20a/cde_gk20a.c b/drivers/gpu/nvgpu/gk20a/cde_gk20a.c index d5d75be5..cf95019b 100644 --- a/drivers/gpu/nvgpu/gk20a/cde_gk20a.c +++ b/drivers/gpu/nvgpu/gk20a/cde_gk20a.c | |||
@@ -46,6 +46,12 @@ | |||
46 | #include <nvgpu/hw/gk20a/hw_ccsr_gk20a.h> | 46 | #include <nvgpu/hw/gk20a/hw_ccsr_gk20a.h> |
47 | #include <nvgpu/hw/gk20a/hw_pbdma_gk20a.h> | 47 | #include <nvgpu/hw/gk20a/hw_pbdma_gk20a.h> |
48 | 48 | ||
49 | /* | ||
50 | * Currently this code uses nvgpu_vm_map() since it takes dmabuf FDs from the | ||
51 | * CDE ioctls. That has to change - instead this needs to take an nvgpu_mem. | ||
52 | */ | ||
53 | #include "common/linux/vm_priv.h" | ||
54 | |||
49 | static int gk20a_cde_load(struct gk20a_cde_ctx *cde_ctx); | 55 | static int gk20a_cde_load(struct gk20a_cde_ctx *cde_ctx); |
50 | static struct gk20a_cde_ctx *gk20a_cde_allocate_context(struct gk20a *g); | 56 | static struct gk20a_cde_ctx *gk20a_cde_allocate_context(struct gk20a *g); |
51 | 57 | ||
@@ -1016,8 +1022,8 @@ __releases(&cde_app->mutex) | |||
1016 | 1022 | ||
1017 | 1023 | ||
1018 | /* map the destination buffer */ | 1024 | /* map the destination buffer */ |
1019 | get_dma_buf(compbits_scatter_buf); /* a ref for gk20a_vm_map */ | 1025 | get_dma_buf(compbits_scatter_buf); /* a ref for nvgpu_vm_map */ |
1020 | map_vaddr = gk20a_vm_map(cde_ctx->vm, compbits_scatter_buf, 0, | 1026 | map_vaddr = nvgpu_vm_map(cde_ctx->vm, compbits_scatter_buf, 0, |
1021 | NVGPU_MAP_BUFFER_FLAGS_CACHEABLE_TRUE, | 1027 | NVGPU_MAP_BUFFER_FLAGS_CACHEABLE_TRUE, |
1022 | compbits_kind, NULL, true, | 1028 | compbits_kind, NULL, true, |
1023 | gk20a_mem_flag_none, | 1029 | gk20a_mem_flag_none, |
@@ -1136,7 +1142,7 @@ __releases(&cde_app->mutex) | |||
1136 | cde_ctx->init_cmd_executed = true; | 1142 | cde_ctx->init_cmd_executed = true; |
1137 | 1143 | ||
1138 | /* unmap the buffers - channel holds references to them now */ | 1144 | /* unmap the buffers - channel holds references to them now */ |
1139 | gk20a_vm_unmap(cde_ctx->vm, map_vaddr); | 1145 | nvgpu_vm_unmap(cde_ctx->vm, map_vaddr); |
1140 | 1146 | ||
1141 | return err; | 1147 | return err; |
1142 | 1148 | ||
@@ -1144,7 +1150,7 @@ exit_unmap_surface: | |||
1144 | if (surface) | 1150 | if (surface) |
1145 | dma_buf_vunmap(compbits_scatter_buf, surface); | 1151 | dma_buf_vunmap(compbits_scatter_buf, surface); |
1146 | exit_unmap_vaddr: | 1152 | exit_unmap_vaddr: |
1147 | gk20a_vm_unmap(cde_ctx->vm, map_vaddr); | 1153 | nvgpu_vm_unmap(cde_ctx->vm, map_vaddr); |
1148 | exit_idle: | 1154 | exit_idle: |
1149 | gk20a_idle(g); | 1155 | gk20a_idle(g); |
1150 | return err; | 1156 | return err; |
@@ -1277,7 +1283,7 @@ err_init_cde_img: | |||
1277 | nvgpu_gmmu_unmap(ch->vm, &g->gr.compbit_store.mem, vaddr); | 1283 | nvgpu_gmmu_unmap(ch->vm, &g->gr.compbit_store.mem, vaddr); |
1278 | err_map_backingstore: | 1284 | err_map_backingstore: |
1279 | err_alloc_gpfifo: | 1285 | err_alloc_gpfifo: |
1280 | gk20a_vm_put(ch->vm); | 1286 | nvgpu_vm_put(ch->vm); |
1281 | err_commit_va: | 1287 | err_commit_va: |
1282 | err_get_gk20a_channel: | 1288 | err_get_gk20a_channel: |
1283 | nvgpu_release_firmware(g, img); | 1289 | nvgpu_release_firmware(g, img); |