diff options
author | Lionel Landwerlin <lionel.g.landwerlin@intel.com> | 2016-04-05 14:13:53 +0100 |
---|---|---|
committer | Marius Vlad <marius.c.vlad@intel.com> | 2016-04-05 17:09:29 +0300 |
commit | cd8da3f65d6d6956b50e4629cb43d3a8d55faab3 (patch) | |
tree | 18f77819d8084c9837f3473c48e126cf28c0121a | |
parent | 54b8a1f2d3b1651448645c89dda278536406cbda (diff) |
lib: kms: move framebuffer scanout offset/size to plane
This fixes potential crashes when the framebuffer is unset from a
given plane.
v2: s/with/within/ typo in header
Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Cc: Marius Vlad <marius.c.vlad@intel.com>
Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Marius Vlad <marius.c.vlad@intel.com>
-rw-r--r-- | lib/igt_fb.h | 4 | ||||
-rw-r--r-- | lib/igt_kms.c | 32 | ||||
-rw-r--r-- | lib/igt_kms.h | 8 |
3 files changed, 24 insertions, 20 deletions
diff --git a/lib/igt_fb.h b/lib/igt_fb.h index e8fe3ac4dd06..0a06899e17f1 100644 --- a/lib/igt_fb.h +++ b/lib/igt_fb.h @@ -55,10 +55,6 @@ struct igt_fb { unsigned size; cairo_surface_t *cairo_surface; unsigned domain; - uint32_t src_x; - uint32_t src_y; - uint32_t src_w; - uint32_t src_h; }; enum igt_text_align { diff --git a/lib/igt_kms.c b/lib/igt_kms.c index 82257a627e67..b63a59d60f5e 100644 --- a/lib/igt_kms.c +++ b/lib/igt_kms.c @@ -1586,10 +1586,10 @@ igt_atomic_prepare_plane_commit(igt_plane_t *plane, igt_output_t *output, } if (plane->position_changed || plane->size_changed) { - uint32_t src_x = IGT_FIXED(plane->fb->src_x, 0); /* src_x */ - uint32_t src_y = IGT_FIXED(plane->fb->src_y, 0); /* src_y */ - uint32_t src_w = IGT_FIXED(plane->fb->src_w, 0); /* src_w */ - uint32_t src_h = IGT_FIXED(plane->fb->src_h, 0); /* src_h */ + uint32_t src_x = IGT_FIXED(plane->src_x, 0); /* src_x */ + uint32_t src_y = IGT_FIXED(plane->src_y, 0); /* src_y */ + uint32_t src_w = IGT_FIXED(plane->src_w, 0); /* src_w */ + uint32_t src_h = IGT_FIXED(plane->src_h, 0); /* src_h */ int32_t crtc_x = plane->crtc_x; int32_t crtc_y = plane->crtc_y; uint32_t crtc_w = plane->crtc_w; @@ -1677,10 +1677,10 @@ static int igt_drm_plane_commit(igt_plane_t *plane, CHECK_RETURN(ret, fail_on_error); } else if (plane->fb_changed || plane->position_changed || plane->size_changed) { - src_x = IGT_FIXED(plane->fb->src_x,0); /* src_x */ - src_y = IGT_FIXED(plane->fb->src_y,0); /* src_y */ - src_w = IGT_FIXED(plane->fb->src_w,0); /* src_w */ - src_h = IGT_FIXED(plane->fb->src_h,0); /* src_h */ + src_x = IGT_FIXED(plane->src_x,0); /* src_x */ + src_y = IGT_FIXED(plane->src_y,0); /* src_y */ + src_w = IGT_FIXED(plane->src_w,0); /* src_w */ + src_h = IGT_FIXED(plane->src_h,0); /* src_h */ crtc_x = plane->crtc_x; crtc_y = plane->crtc_y; crtc_w = plane->crtc_w; @@ -2201,10 +2201,10 @@ void igt_plane_set_fb(igt_plane_t *plane, struct igt_fb *fb) plane->crtc_h = fb->height; /* set default src pos/size as fb size */ - fb->src_x = 0; - fb->src_y = 0; - fb->src_w = fb->width; - fb->src_h = fb->height; + plane->src_x = 0; + plane->src_y = 0; + plane->src_w = fb->width; + plane->src_h = fb->height; } else { plane->crtc_w = 0; plane->crtc_h = 0; @@ -2271,8 +2271,8 @@ void igt_fb_set_position(struct igt_fb *fb, igt_plane_t *plane, LOG(display, "%s.%d: fb_set_position(%d,%d)\n", kmstest_pipe_name(pipe->pipe), plane->index, x, y); - fb->src_x = x; - fb->src_y = y; + plane->src_x = x; + plane->src_y = y; plane->fb_changed = true; } @@ -2297,8 +2297,8 @@ void igt_fb_set_size(struct igt_fb *fb, igt_plane_t *plane, LOG(display, "%s.%d: fb_set_size(%dx%d)\n", kmstest_pipe_name(pipe->pipe), plane->index, w, h); - fb->src_w = w; - fb->src_h = h; + plane->src_w = w; + plane->src_h = h; plane->fb_changed = true; } diff --git a/lib/igt_kms.h b/lib/igt_kms.h index 8ae119242dfc..b763120398f6 100644 --- a/lib/igt_kms.h +++ b/lib/igt_kms.h @@ -243,6 +243,14 @@ typedef struct { int crtc_x, crtc_y; /* size within pipe_src_w x pipe_src_h */ int crtc_w, crtc_h; + + /* position within the framebuffer */ + uint32_t src_x; + uint32_t src_y; + /* size within the framebuffer*/ + uint32_t src_w; + uint32_t src_h; + /* panning offset within the fb */ unsigned int pan_x, pan_y; igt_rotation_t rotation; |