Attachment 121589 Details for Bug 93849 – Use uint64_t in eviction memory subtest parameters

[patch] Use uint64_t in eviction memory subtest parameters

0001-Use-uint64_t-in-eviction-memory-subtest-parameters.patch (text/plain), 1.60 KB, created by Mika Kuoppala on 2016-02-08 15:00:15 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Mika Kuoppala

Created: 2016-02-08 15:00:15 UTC

Size: 1.60 KB

patch

obsolete

>From 7f7dfc047f5aa76e4e932a463335b3d71313fab2 Mon Sep 17 00:00:00 2001
>From: Mika Kuoppala <mika.kuoppala@intel.com>
>Date: Mon, 8 Feb 2016 10:59:20 +0200
>Subject: [PATCH i-g-t] Use uint64_t in eviction memory subtest parameters
>
>With large apertures we need to use uint64_t for
>counts and sizes. commit 0e2071411a4d4e1488a821daf522dffde2809e03
>paved way for this but forgot to change the subtest parameters.
>
>References: https://bugs.freedesktop.org/show_bug.cgi?id=93849
>Cc: Chris Wilson <chris@chris-wilson.co.uk>
>Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com>
>---
> tests/gem_evict_alignment.c | 10 ++++++----
> 1 file changed, 6 insertions(+), 4 deletions(-)
>
>diff --git a/tests/gem_evict_alignment.c b/tests/gem_evict_alignment.c
>index 6fa70f0cd0f9..d7d0ed0aec73 100644
>--- a/tests/gem_evict_alignment.c
>+++ b/tests/gem_evict_alignment.c
>@@ -129,10 +129,11 @@ copy(int fd, uint32_t dst, uint32_t src, uint32_t *all_bo, int n_bo, int alignme
> 	free(obj);
> }
> 
>-static void minor_evictions(int fd, int size, int count)
>+static void minor_evictions(int fd, uint64_t size, uint64_t count)
> {
> 	uint32_t *bo, *sel;
>-	int n, m, alignment, pass, fail;
>+	uint64_t n, m, alignment;
>+	int pass, fail;
> 
> 	intel_require_memory(2 * count, size, CHECK_RAM);
> 
>@@ -159,9 +160,10 @@ static void minor_evictions(int fd, int size, int count)
> 	free(bo);
> }
> 
>-static void major_evictions(int fd, int size, int count)
>+static void major_evictions(int fd, uint64_t size, uint64_t count)
> {
>-	int n, m, loop, alignment, max;
>+	uint64_t n, m, alignment, max;
>+	int loop;
> 	uint32_t *bo;
> 
> 	intel_require_memory(count, size, CHECK_RAM);
>-- 
>2.5.0
>

Actions: View | Splinter Review

Attachments on bug 93849: 121264 | 121589

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.