-----Original Message----- From: Hajda, Andrzej <[email protected]> Sent: Wednesday, April 26, 2023 8:14 AM To: [email protected]; [email protected]; Cavitt, Jonathan <[email protected]>; Hajda, Andrzej <[email protected]>; Auld, Matthew <[email protected]> Subject: [PATCH v7 0/2] drm/i915: Hugepage manager and test for MTL > >This patchset patches sent by Jonathan and Andi, with >addressed CI failures: >1. Fixed checking alignment of 64K pages on both Pre-Gen12 and Gen12. >2. Fixed start alignment of 2M pages. > >Regards >Andrzej > >Jonathan Cavitt (2): > drm/i915: Migrate platform-dependent mock hugepage selftests to live > drm/i915: Use correct huge page manager for MTL > >.../gpu/drm/i915/gem/selftests/huge_pages.c | 88 +++++++++++++++---- > drivers/gpu/drm/i915/gt/gen8_ppgtt.c | 3 +- > 2 files changed, 71 insertions(+), 20 deletions(-) > >Cc: [email protected] >Cc: [email protected] >Cc: Jonathan Cavitt <[email protected]> >Cc: Andrzej Hajda <[email protected]> >Cc: Matthew Auld <[email protected]> >-- >2.39.2 > >--- >Jonathan Cavitt (2): > drm/i915: Migrate platform-dependent mock hugepage selftests to live > drm/i915: Use correct huge page manager for MTL > > drivers/gpu/drm/i915/gem/selftests/huge_pages.c | 105 ++++++++++++++++++------ > drivers/gpu/drm/i915/gt/gen8_ppgtt.c | 3 +- > 2 files changed, 82 insertions(+), 26 deletions(-) >--- >base-commit: 4d0066a1c0763d50b6fb017e27d12b081ce21b57 >change-id: 20230425-hugepage-migrate-1869aaf31a6d > >Best regards, >-- >Andrzej Hajda <[email protected]>
Just reviewed the changes proper. It's been a while, so I don't remember everything about the prior version, but I think I recognized what was changed: - I wasn't aware a 21 bit alignment was required for 2M page sizes. I'm glad you caught that. - The extra debugging/error information will be helpful in the case of a failure. - Grabbing the per-context VM instead of the ppgtt vm sounds good to me. Everything here looks amenable. Acked-by: Jonathan Cavitt <[email protected]> Though, given I'm also one of the signed-off-bys, I don't know if me acking it is exactly above-board. -Jonathan Cavitt >
