diff mbox

[igt] tests/kms_ccs: Don't overallocate CCS surface

Message ID 20170804163752.10304-1-daniels@collabora.com (mailing list archive)
State New, archived
Headers show

Commit Message

Daniel Stone Aug. 4, 2017, 4:37 p.m. UTC
Due to a mix-up in kernel branches being used, I'd mangled Jason's
original CCS test to hopelessly overallocate the CCS surface size.
Restore it back to its original.

Signed-off-by: Daniel Stone <daniels@collabora.com>
Cc: Jason Ekstrand <jason@jlekstrand.net>
---
 tests/kms_ccs.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Jason Ekstrand Aug. 4, 2017, 5:03 p.m. UTC | #1
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>

On Fri, Aug 4, 2017 at 9:37 AM, Daniel Stone <daniels@collabora.com> wrote:

> Due to a mix-up in kernel branches being used, I'd mangled Jason's
> original CCS test to hopelessly overallocate the CCS surface size.
> Restore it back to its original.
>
> Signed-off-by: Daniel Stone <daniels@collabora.com>
> Cc: Jason Ekstrand <jason@jlekstrand.net>
> ---
>  tests/kms_ccs.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/kms_ccs.c b/tests/kms_ccs.c
> index 0524a43e..a40d6c10 100644
> --- a/tests/kms_ccs.c
> +++ b/tests/kms_ccs.c
> @@ -188,7 +188,7 @@ static void display_fb(data_t *data, int compressed)
>                 f.pitches[1] = ALIGN(width * 1, 128);
>                 f.modifier[1] = modifier;
>                 f.offsets[1] = size[0];
> -               size[1] = f.pitches[1] * ALIGN(f.height, 32);
> +               size[1] = f.pitches[1] * ALIGN(height, 32);
>
>                 f.handles[0] = gem_create(data->drm_fd, size[0] + size[1]);
>                 f.handles[1] = f.handles[0];
> --
> 2.13.4
>
>
diff mbox

Patch

diff --git a/tests/kms_ccs.c b/tests/kms_ccs.c
index 0524a43e..a40d6c10 100644
--- a/tests/kms_ccs.c
+++ b/tests/kms_ccs.c
@@ -188,7 +188,7 @@  static void display_fb(data_t *data, int compressed)
 		f.pitches[1] = ALIGN(width * 1, 128);
 		f.modifier[1] = modifier;
 		f.offsets[1] = size[0];
-		size[1] = f.pitches[1] * ALIGN(f.height, 32);
+		size[1] = f.pitches[1] * ALIGN(height, 32);
 
 		f.handles[0] = gem_create(data->drm_fd, size[0] + size[1]);
 		f.handles[1] = f.handles[0];