[igt-dev] [PATCH i-g-t] tests/i915/gem_shrink: Fix alloc_size unit

[Niranjana Vishwanathapura]

On Wed, Nov 02, 2022 at 10:32:44AM +0000, Matthew Auld wrote:
>On 02/11/2022 05:29, Niranjana Vishwanathapura wrote:
>>alloc_size is in MBs, but functions are expecting bytes.
>>Convert alloc_size to bytes before using it.
>>
>>TODO: Needs some testing
>>
>>Signed-off-by: Niranjana Vishwanathapura <niranjana.vishwanathapura at intel.com>
>
>Most of this test is blacklisted for some reason. CI looks to only run 
>the reclaim subtest.

Yah, I did run into some issues while running this test
with and without this fix. Looks like there are other issues
with these tests and hence blacklisted.
I hope get this patch merged anyway as it is the right thing anyway.

>
>Reviewed-by: Matthew Auld <matthew.auld at intel.com>

Thanks,
Niranjana

>
>>---
>>  tests/i915/gem_shrink.c | 1 +
>>  1 file changed, 1 insertion(+)
>>
>>diff --git a/tests/i915/gem_shrink.c b/tests/i915/gem_shrink.c
>>index 49df35c29..e3e20dfc9 100644
>>--- a/tests/i915/gem_shrink.c
>>+++ b/tests/i915/gem_shrink.c
>>@@ -455,6 +455,7 @@ igt_main
>>  		igt_info("Using %d processes and %'"PRIu64"MiB per process\n",
>>  			 num_processes, alloc_size);
>>+		alloc_size <<= 20;
>>  		igt_require_memory(num_processes, alloc_size,
>>  				   CHECK_SWAP | CHECK_RAM);