dri-devel May 2015

dri-devel@lists.freedesktop.org

191 participants
1132 discussions

[PATCH] drm/atomic: fix out of bounds read in for_each_*_in_state helpers
by Andrey Ryabinin 26 May '15

26 May '15

for_each_*_in_state validate array index after access to array elements, thus perform out of bounds read. Fix this by validating index in the first place and read array element iff validation was successful. Fixes: df63b9994eaf ("drm/atomic: Add for_each_{connector,crtc,plane}_in_state helper macros") Signed-off-by: Andrey Ryabinin <a.ryabinin(a)samsung.com> --- include/drm/drm_atomic.h | 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/include/drm/drm_atomic.h b/include/drm/drm_atomic.h index c1571034..3f13b91 100644 --- a/include/drm/drm_atomic.h +++ b/include/drm/drm_atomic.h @@ -77,26 +77,26 @@ int __must_check drm_atomic_async_commit(struct drm_atomic_state *state); #define for_each_connector_in_state(state, connector, connector_state, __i) \ for ((__i) = 0; \ - (connector) = (state)->connectors[__i], \ - (connector_state) = (state)->connector_states[__i], \ - (__i) < (state)->num_connector; \ + (__i) < (state)->num_connector && \ + ((connector) = (state)->connectors[__i], \ + (connector_state) = (state)->connector_states[__i], 1); \ (__i)++) \ if (connector) #define for_each_crtc_in_state(state, crtc, crtc_state, __i) \ for ((__i) = 0; \ - (crtc) = (state)->crtcs[__i], \ - (crtc_state) = (state)->crtc_states[__i], \ - (__i) < (state)->dev->mode_config.num_crtc; \ + (__i) < (state)->dev->mode_config.num_crtc && \ + ((crtc) = (state)->crtcs[__i], \ + (crtc_state) = (state)->crtc_states[__i], 1); \ (__i)++) \ if (crtc_state) -#define for_each_plane_in_state(state, plane, plane_state, __i) \ - for ((__i) = 0; \ - (plane) = (state)->planes[__i], \ - (plane_state) = (state)->plane_states[__i], \ - (__i) < (state)->dev->mode_config.num_total_plane; \ - (__i)++) \ +#define for_each_plane_in_state(state, plane, plane_state, __i) \ + for ((__i) = 0; \ + (__i) < (state)->dev->mode_config.num_total_plane && \ + ((plane) = (state)->planes[__i], \ + (plane_state) = (state)->plane_states[__i], 1); \ + (__i)++) \ if (plane_state) #endif /* DRM_ATOMIC_H_ */ -- 2.4.1

4 7

[PATCH] drm: Remove misleading paragraph from drm_vblank_pre_modeset kerneldoc comment
by Michel Dänzer 26 May '15

26 May '15

From: Michel Dänzer <michel.daenzer(a)amd.com> drm_vblank_pre/post_modeset work fine for the radeon driver even though it uses hardware counters, including suspend/resume. Signed-off-by: Michel Dänzer <michel.daenzer(a)amd.com> --- drivers/gpu/drm/drm_irq.c | 7 ------- 1 file changed, 7 deletions(-) diff --git a/drivers/gpu/drm/drm_irq.c b/drivers/gpu/drm/drm_irq.c index f9634da..0dcd73d 100644 --- a/drivers/gpu/drm/drm_irq.c +++ b/drivers/gpu/drm/drm_irq.c @@ -1346,13 +1346,6 @@ EXPORT_SYMBOL(drm_crtc_vblank_on); * software-side vblank frame counting will ensure that there are no jumps or * discontinuities. * - * Unfortunately this approach is racy and also doesn't work when the vblank - * interrupt stops running, e.g. across system suspend resume. It is therefore - * highly recommended that drivers use the newer drm_vblank_off() and - * drm_vblank_on() instead. drm_vblank_pre_modeset() only works correctly when - * using "cooked" software vblank frame counters and not relying on any hardware - * counters. - * * Drivers must call drm_vblank_post_modeset() when re-enabling the same crtc * again. */ -- 2.1.4

2 1

[PATCH] drm/radeon: stop using addr to check for BO move
by Christian König 26 May '15

26 May '15

From: Christian König <christian.koenig(a)amd.com> It is theoretically possible that a swapped out BO gets the same GTT address, but different backing pages while being swapped in. Instead just use another VA state to note updated areas. Signed-off-by: Christian König <christian.koenig(a)amd.com> --- drivers/gpu/drm/radeon/radeon.h | 4 ++- drivers/gpu/drm/radeon/radeon_vm.c | 53 +++++++++++++++++++++----------------- 2 files changed, 32 insertions(+), 25 deletions(-) diff --git a/drivers/gpu/drm/radeon/radeon.h b/drivers/gpu/drm/radeon/radeon.h index 46eb0fa..ef7df51 100644 --- a/drivers/gpu/drm/radeon/radeon.h +++ b/drivers/gpu/drm/radeon/radeon.h @@ -467,7 +467,6 @@ struct radeon_bo_va { /* protected by bo being reserved */ struct list_head bo_list; uint32_t flags; - uint64_t addr; struct radeon_fence *last_pt_update; unsigned ref_count; @@ -941,6 +940,9 @@ struct radeon_vm { /* BOs freed, but not yet updated in the PT */ struct list_head freed; + /* BOs cleared in the PT */ + struct list_head cleared; + /* contains the page directory */ struct radeon_bo *page_directory; unsigned max_pde_used; diff --git a/drivers/gpu/drm/radeon/radeon_vm.c b/drivers/gpu/drm/radeon/radeon_vm.c index de42fc4..9739ded 100644 --- a/drivers/gpu/drm/radeon/radeon_vm.c +++ b/drivers/gpu/drm/radeon/radeon_vm.c @@ -331,7 +331,6 @@ struct radeon_bo_va *radeon_vm_bo_add(struct radeon_device *rdev, bo_va->it.start = 0; bo_va->it.last = 0; bo_va->flags = 0; - bo_va->addr = 0; bo_va->ref_count = 1; INIT_LIST_HEAD(&bo_va->bo_list); INIT_LIST_HEAD(&bo_va->vm_status); @@ -491,9 +490,11 @@ int radeon_vm_bo_set_addr(struct radeon_device *rdev, } if (bo_va->it.start || bo_va->it.last) { - if (bo_va->addr) { + spin_lock(&vm->status_lock); + if (list_empty(&bo_va->vm_status)) { /* add a clone of the bo_va to clear the old address */ struct radeon_bo_va *tmp; + spin_unlock(&vm->status_lock); tmp = kzalloc(sizeof(struct radeon_bo_va), GFP_KERNEL); if (!tmp) { mutex_unlock(&vm->mutex); @@ -502,14 +503,11 @@ int radeon_vm_bo_set_addr(struct radeon_device *rdev, tmp->it.start = bo_va->it.start; tmp->it.last = bo_va->it.last; tmp->vm = vm; - tmp->addr = bo_va->addr; tmp->bo = radeon_bo_ref(bo_va->bo); spin_lock(&vm->status_lock); list_add(&tmp->vm_status, &vm->freed); - spin_unlock(&vm->status_lock); - - bo_va->addr = 0; } + spin_unlock(&vm->status_lock); interval_tree_remove(&bo_va->it, &vm->va); bo_va->it.start = 0; @@ -520,10 +518,12 @@ int radeon_vm_bo_set_addr(struct radeon_device *rdev, bo_va->it.start = soffset; bo_va->it.last = eoffset - 1; interval_tree_insert(&bo_va->it, &vm->va); + spin_lock(&vm->status_lock); + list_add(&bo_va->vm_status, &vm->cleared); + spin_unlock(&vm->status_lock); } bo_va->flags = flags; - bo_va->addr = 0; soffset >>= radeon_vm_block_size; eoffset >>= radeon_vm_block_size; @@ -921,7 +921,16 @@ int radeon_vm_bo_update(struct radeon_device *rdev, } spin_lock(&vm->status_lock); - list_del_init(&bo_va->vm_status); + if (mem) { + if (list_empty(&bo_va->vm_status)) { + spin_unlock(&vm->status_lock); + return 0; + } + list_del_init(&bo_va->vm_status); + } else { + list_del(&bo_va->vm_status); + list_add(&bo_va->vm_status, &vm->cleared); + } spin_unlock(&vm->status_lock); bo_va->flags &= ~RADEON_VM_PAGE_VALID; @@ -947,10 +956,6 @@ int radeon_vm_bo_update(struct radeon_device *rdev, addr = 0; } - if (addr == bo_va->addr) - return 0; - bo_va->addr = addr; - trace_radeon_vm_bo_update(bo_va); nptes = bo_va->it.last - bo_va->it.start + 1; @@ -1038,7 +1043,7 @@ int radeon_vm_clear_freed(struct radeon_device *rdev, struct radeon_vm *vm) { struct radeon_bo_va *bo_va; - int r; + int r = 0; spin_lock(&vm->status_lock); while (!list_empty(&vm->freed)) { @@ -1049,14 +1054,15 @@ int radeon_vm_clear_freed(struct radeon_device *rdev, r = radeon_vm_bo_update(rdev, bo_va, NULL); radeon_bo_unref(&bo_va->bo); radeon_fence_unref(&bo_va->last_pt_update); + spin_lock(&vm->status_lock); + list_del(&bo_va->vm_status); kfree(bo_va); if (r) - return r; + break; - spin_lock(&vm->status_lock); } spin_unlock(&vm->status_lock); - return 0; + return r; } @@ -1114,14 +1120,14 @@ void radeon_vm_bo_rmv(struct radeon_device *rdev, mutex_lock(&vm->mutex); if (bo_va->it.start || bo_va->it.last) interval_tree_remove(&bo_va->it, &vm->va); - spin_lock(&vm->status_lock); - list_del(&bo_va->vm_status); - if (bo_va->addr) { + spin_lock(&vm->status_lock); + if (list_empty(&bo_va->vm_status)) { bo_va->bo = radeon_bo_ref(bo_va->bo); list_add(&bo_va->vm_status, &vm->freed); } else { radeon_fence_unref(&bo_va->last_pt_update); + list_del(&bo_va->vm_status); kfree(bo_va); } spin_unlock(&vm->status_lock); @@ -1144,12 +1150,10 @@ void radeon_vm_bo_invalidate(struct radeon_device *rdev, struct radeon_bo_va *bo_va; list_for_each_entry(bo_va, &bo->va, bo_list) { - if (bo_va->addr) { - spin_lock(&bo_va->vm->status_lock); - list_del(&bo_va->vm_status); + spin_lock(&bo_va->vm->status_lock); + if (list_empty(&bo_va->vm_status)) list_add(&bo_va->vm_status, &bo_va->vm->invalidated); - spin_unlock(&bo_va->vm->status_lock); - } + spin_unlock(&bo_va->vm->status_lock); } } @@ -1179,6 +1183,7 @@ int radeon_vm_init(struct radeon_device *rdev, struct radeon_vm *vm) spin_lock_init(&vm->status_lock); INIT_LIST_HEAD(&vm->invalidated); INIT_LIST_HEAD(&vm->freed); + INIT_LIST_HEAD(&vm->cleared); pd_size = radeon_vm_directory_size(rdev); pd_entries = radeon_vm_num_pdes(rdev); -- 1.9.1

2 1

[PATCH v2 0/4] CRTC state mode property
by Daniel Stone 26 May '15

26 May '15

Hi, Respin with the fixups from Daniel Vetter's review. The two changes visible to userspace is that we now require anyone clearing MODE_ID to explicitly set the CRTC active property to false, and that the property type is BLOB, rather than OBJECT/BLOB. The rest are fairly minor. Cheers, Daniel

2 6

[PATCH] drm: check for garbage in unused addfb2 fields
by Daniel Vetter 26 May '15

26 May '15

Unfortunately old userspace didn't clear this properly, but since we've added fb modifiers that's fixed. Checking properly that unused fields is important for abi extensions, and just right now there's a bunch of discussions going on about how exactly the additional aux planes for render compression should be specified. So let's first make sure that the values in those additional fields can be indeed used. Cc: Thierry Reding <thierry.reding(a)gmail.com> Testcase: igt/kms_addfb/unused-* Signed-off-by: Daniel Vetter <daniel.vetter(a)intel.com> --- drivers/gpu/drm/drm_crtc.c | 26 ++++++++++++++++++++++++++ 1 file changed, 26 insertions(+) diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c index 2e26988a9762..8ad582f4026b 100644 --- a/drivers/gpu/drm/drm_crtc.c +++ b/drivers/gpu/drm/drm_crtc.c @@ -3308,6 +3308,32 @@ static int framebuffer_check(const struct drm_mode_fb_cmd2 *r) } } + for (i = num_planes; i < 4; i++) { + if (r->modifier[i]) { + DRM_DEBUG_KMS("non-zero modifier for unused plane %d\n", i); + return -EINVAL; + } + + /* Pre-FB_MODIFIERS userspace didn't clear the structs properly. */ + if (!(r->flags & DRM_MODE_FB_MODIFIERS)) + continue; + + if (r->handles[i]) { + DRM_DEBUG_KMS("buffer object handle for unused plane %d\n", i); + return -EINVAL; + } + + if (r->pitches[i]) { + DRM_DEBUG_KMS("non-zero pitch for unused plane %d\n", i); + return -EINVAL; + } + + if (r->offsets[i]) { + DRM_DEBUG_KMS("non-zero offset for unused plane %d\n", i); + return -EINVAL; + } + } + return 0; } -- 2.1.4

3 2

[PATCH] drm/radeon: stop using addr to check for BO move
by Christian König 26 May '15

26 May '15

1 2

[PATCH] drm/plane_helper: Use helper for plane->CRTC linkage
by Daniel Stone 26 May '15

26 May '15

plane_state->crtc shouldn't be assigned directly, but instead use drm_atomic_set_crtc_for_plane, which also takes care of updating the plane_mask on each CRTC's state. Signed-off-by: Daniel Stone <daniels(a)collabora.com> --- drivers/gpu/drm/drm_plane_helper.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/drm_plane_helper.c b/drivers/gpu/drm/drm_plane_helper.c index 40c1db9..e215a07 100644 --- a/drivers/gpu/drm/drm_plane_helper.c +++ b/drivers/gpu/drm/drm_plane_helper.c @@ -530,7 +530,7 @@ int drm_plane_helper_update(struct drm_plane *plane, struct drm_crtc *crtc, return -ENOMEM; plane_state->plane = plane; - plane_state->crtc = crtc; + drm_atomic_set_crtc_for_plane(plane_state, crtc); drm_atomic_set_fb_for_plane(plane_state, fb); plane_state->crtc_x = crtc_x; plane_state->crtc_y = crtc_y; @@ -577,7 +577,7 @@ int drm_plane_helper_disable(struct drm_plane *plane) return -ENOMEM; plane_state->plane = plane; - plane_state->crtc = NULL; + drm_atomic_set_crtc_for_plane(plane_state, NULL); drm_atomic_set_fb_for_plane(plane_state, NULL); return drm_plane_helper_commit(plane, plane_state, plane->fb); -- 2.4.1

1 2

[PATCH] drm/nouveau/platform: add IOMMU dependency
by Arnd Bergmann 26 May '15

26 May '15

The recently added iommu code in the nouveau driver fails to build when the IOMMU support is disabled: drivers/gpu/drm/nouveau/nouveau_platform.c: In function 'nouveau_platform_probe_iommu': drivers/gpu/drm/nouveau/nouveau_platform.c:113:41: error: 'const struct iommu_ops' has no mem To avoid the build error, this now adds an explicit dependency on the IOMMU implementation. Signed-off-by: Arnd Bergmann <arnd(a)arndb.de> Fixes: 58fd9375c2c5 ("drm/nouveau/platform: probe IOMMU if present") diff --git a/drivers/gpu/drm/nouveau/Kconfig b/drivers/gpu/drm/nouveau/Kconfig index 5ab13e7939db..18dfa4af60ea 100644 --- a/drivers/gpu/drm/nouveau/Kconfig +++ b/drivers/gpu/drm/nouveau/Kconfig @@ -28,6 +28,7 @@ config DRM_NOUVEAU config NOUVEAU_PLATFORM_DRIVER bool "Nouveau (NVIDIA) SoC GPUs" depends on DRM_NOUVEAU && ARCH_TEGRA + depends on IOMMU default y help Support for Nouveau platform driver, used for SoC GPUs as found

4 8

Re: Build regressions/improvements in v4.1-rc5
by Geert Uytterhoeven 26 May '15

26 May '15

On Tue, May 26, 2015 at 9:39 AM, Geert Uytterhoeven <geert(a)linux-m68k.org> wrote: > JFYI, when comparing v4.1-rc5[1] to v4.1-rc4[3], the summaries are: > - build errors: +52/-28 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: 'L_PTE_MT_BUFFERABLE' undeclared here (not in a function): => 81:10 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: 'L_PTE_MT_DEV_CACHED' undeclared here (not in a function): => 117:10 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: 'L_PTE_MT_DEV_NONSHARED' undeclared here (not in a function): => 108:10 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: 'L_PTE_MT_DEV_SHARED' undeclared here (not in a function): => 103:10 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: 'L_PTE_MT_DEV_WC' undeclared here (not in a function): => 113:10 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: 'L_PTE_MT_MASK' undeclared here (not in a function): => 76:11 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: 'L_PTE_MT_MINICACHE' undeclared here (not in a function): => 94:10 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: 'L_PTE_MT_UNCACHED' undeclared here (not in a function): => 77:10 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: 'L_PTE_MT_WRITEALLOC' undeclared here (not in a function): => 99:10 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: 'L_PTE_MT_WRITEBACK' undeclared here (not in a function): => 89:10 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: 'L_PTE_MT_WRITETHROUGH' undeclared here (not in a function): => 85:10 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: 'L_PTE_RDONLY' undeclared here (not in a function): => 61:11 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: 'L_PTE_SHARED' undeclared here (not in a function): => 71:11 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: 'L_PTE_USER' undeclared here (not in a function): => 56:11 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: 'L_PTE_XN' undeclared here (not in a function): => 66:11 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: 'PMD_SECT_AP_READ' undeclared here (not in a function): => 153:13 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: 'PMD_SECT_AP_WRITE' undeclared here (not in a function): => 153:32 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: 'PMD_SECT_S' undeclared here (not in a function): => 175:11 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: 'PMD_SECT_XN' undeclared here (not in a function): => 170:11 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: 'PMD_SIZE' undeclared (first use in this function): => 276:22 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: 'PTRS_PER_PGD' undeclared (first use in this function): => 314:18 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: 'PTRS_PER_PMD' undeclared (first use in this function): => 275:18 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: 'PTRS_PER_PTE' undeclared (first use in this function): => 263:18 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: 'SECTION_SIZE' undeclared (first use in this function): => 282:7 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: (near initialization for 'section_bits[0].mask'): => 153:3 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: (near initialization for 'section_bits[1].mask'): => 157:3 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: (near initialization for 'section_bits[1].val'): => 158:3 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: (near initialization for 'section_bits[2].mask'): => 161:3 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: (near initialization for 'section_bits[2].val'): => 162:3 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: (near initialization for 'section_bits[3].mask'): => 165:3 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: (near initialization for 'section_bits[3].val'): => 166:3 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: (near initialization for 'section_bits[4].mask'): => 170:3 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: (near initialization for 'section_bits[4].val'): => 171:3 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: (near initialization for 'section_bits[5].mask'): => 175:3 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: (near initialization for 'section_bits[5].val'): => 176:3 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: implicit declaration of function 'pmd_large' [-Werror=implicit-function-declaration]: => 277:3 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: implicit declaration of function 'pmd_none' [-Werror=implicit-function-declaration]: => 277:3 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: implicit declaration of function 'pmd_present' [-Werror=implicit-function-declaration]: => 277:3 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: implicit declaration of function 'pte_offset_kernel' [-Werror=implicit-function-declaration]: => 259:2 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: initializer element is not constant: => 170:3, 153:3, 166:3, 162:3, 165:3, 157:3, 175:3, 176:3, 158:3, 171:3, 161:3 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: invalid operands to binary * (have 'unsigned int' and 'const struct prot_bits *'): => 276:20 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: invalid operands to binary | (have 'const struct prot_bits *' and 'const struct prot_bits *'): => 165:30, 166:30, 161:30, 153:30, 157:30 arm/arm-randconfig (reported before for v3.19-rc3). + /home/kisskb/slave/src/drivers/gpu/drm/nouveau/nvkm/subdev/instmem/gk20a.c: error: implicit declaration of function 'dma_alloc_attrs' [-Werror=implicit-function-declaration]: => 218:2 + /home/kisskb/slave/src/drivers/gpu/drm/nouveau/nvkm/subdev/instmem/gk20a.c: error: implicit declaration of function 'dma_free_attrs' [-Werror=implicit-function-declaration]: => 154:2 xtensa-allmodconfig + /home/kisskb/slave/src/fs/dax.c: error: implicit declaration of function 'copy_user_page' [-Werror=implicit-function-declaration]: => 265:2 sh-randconfig > [1] http://kisskb.ellerman.id.au/kisskb/head/8914/ (254 out of 257 configs) > [3] http://kisskb.ellerman.id.au/kisskb/head/8886/ (254 out of 257 configs) Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert(a)linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds

1 0

[PATCH 00/11] CRTC state mode property
by Daniel Stone 26 May '15

26 May '15

Hi, Building on the blob-property submission series[0], this provides a MODE_ID property that can be used to set the current mode for a CRTC under atomic modesetting, plus a way for users to create blob properties so they can specify their own modes. Using this[1] I've been able to validate the atomic userspace API with a ported version of Weston[2]. This is currently undergoing review, but seems pretty OK. Both modesetting for every output together, and per-output async pageflips with events, works as well as the driver does. Also tested with fbcon and X11, on Broadwell. Cheers, Daniel [0]: <1429554176-9865-1-git-send-email-daniels(a)collabora.com> [1]: And a pile of other work in the i915 driver from Maarten/Ander, and my own bits on top; see git://git.collabora.co.uk/git/user/daniels/linux in the wip/4.1.x/blob-modes-i915 branch. [2]: http://lists.freedesktop.org/archives/wayland-devel/2015-May/022106.html

3 18

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

dri-devel May 2015