public inbox for igt-dev@lists.freedesktop.org
 help / color / mirror / Atom feed
* [igt-dev] [PATCH i-g-t 1/5] tests/dumb_buffer: Remove page nonaligned buffer tests
@ 2020-01-27 17:02 Ramalingam C
  2020-01-27 17:02 ` [igt-dev] [PATCH i-g-t 2/5] tests/i915/gem_create: Modify the page nonaligned tests Ramalingam C
                   ` (5 more replies)
  0 siblings, 6 replies; 8+ messages in thread
From: Ramalingam C @ 2020-01-27 17:02 UTC (permalink / raw)
  To: igt-dev

Considering that kernel returns the real memory size(page aligned)
allocated, nonaligned buffer tests are not valid anymore. Hence removing
them.

Signed-off-by: Ramalingam C <ramalingam.c@intel.com>
cc: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
---
 tests/dumb_buffer.c | 52 ---------------------------------------------
 1 file changed, 52 deletions(-)

diff --git a/tests/dumb_buffer.c b/tests/dumb_buffer.c
index 3d2dc9966d0b..b6fe6f5dc6ec 100644
--- a/tests/dumb_buffer.c
+++ b/tests/dumb_buffer.c
@@ -200,52 +200,6 @@ static void invalid_size_map(int fd)
 	dumb_destroy(fd, create.handle);
 }
 
-/*
- * Creating an dumb buffer with non-aligned size and trying to access it with an
- * offset, which is greater than the requested size but smaller than the
- * object's last page boundary. pwrite here must be successful.
- */
-static void valid_nonaligned_size(int fd)
-{
-	struct drm_mode_create_dumb create = {
-		.width = 24,
-		.height = 24,
-		.bpp = 32,
-	};
-	char buf[PAGE_SIZE];
-
-	igt_require(is_i915_device(fd));
-
-	dumb_create(fd, &create);
-
-	gem_write(fd, create.handle, PAGE_SIZE / 2, buf, PAGE_SIZE / 2);
-
-	dumb_destroy(fd, create.handle);
-}
-
-/*
- * Creating an object with non-aligned size and trying to access it with an
- * offset, which is greater than the requested size and larger than the
- * object's last page boundary. pwrite here must fail.
- */
-static void invalid_nonaligned_size(int fd)
-{
-	struct drm_mode_create_dumb create = {
-		.width = 24,
-		.height = 24,
-		.bpp = 32,
-	};
-	char buf[PAGE_SIZE];
-
-	igt_require(is_i915_device(fd));
-
-	dumb_create(fd, &create);
-	/* This should fail. Hence cannot use gem_write. */
-	igt_assert(__gem_write(fd, create.handle,
-			       create.size - (PAGE_SIZE / 2), buf, PAGE_SIZE));
-	dumb_destroy(fd, create.handle);
-}
-
 static uint64_t atomic_compare_swap_u64(_Atomic(uint64_t) *ptr,
 					uint64_t oldval, uint64_t newval)
 {
@@ -362,12 +316,6 @@ igt_main
 	igt_subtest("create-valid-dumb")
 		valid_dumb_creation_test(fd);
 
-	igt_subtest("create-valid-nonaligned")
-		valid_nonaligned_size(fd);
-
-	igt_subtest("create-invalid-nonaligned")
-		invalid_nonaligned_size(fd);
-
 	igt_subtest("map-valid")
 		valid_map(fd);
 
-- 
2.20.1

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply related	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2020-01-28 13:57 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-01-27 17:02 [igt-dev] [PATCH i-g-t 1/5] tests/dumb_buffer: Remove page nonaligned buffer tests Ramalingam C
2020-01-27 17:02 ` [igt-dev] [PATCH i-g-t 2/5] tests/i915/gem_create: Modify the page nonaligned tests Ramalingam C
2020-01-27 17:02 ` [igt-dev] [PATCH i-g-t 3/5] tests/dumb_buffer: Try to compute the largest possible dumb buffer Ramalingam C
2020-01-27 17:02 ` [igt-dev] [PATCH i-g-t 4/5] tests/dumb_duffer: extending for lmem coverage Ramalingam C
2020-01-27 17:13   ` Chris Wilson
2020-01-27 17:02 ` [igt-dev] [PATCH i-g-t 5/5] lib/i915/intel_memory_region: wrappers for memory regions ops Ramalingam C
2020-01-27 19:41 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/5] tests/dumb_buffer: Remove page nonaligned buffer tests Patchwork
2020-01-28 13:57 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox