[PATCH] drm/xe/migrate: Fix alignment check
Lucas De Marchi
lucas.demarchi at intel.com
Thu Jul 10 22:42:50 UTC 2025
On Thu, Jul 10, 2025 at 03:36:55PM -0700, Matthew Brost wrote:
>On Thu, Jul 10, 2025 at 02:34:41PM -0700, Lucas De Marchi wrote:
>> The check would fail if the address is unaligned, but not when
>> accounting the offset. Instead of `buf | offset` it should have
>> been `buf + offset`. To make it more readable and also drop the
>> uintptr_t, just use the IS_ALIGNED() macro.
>>
>> Cc: Matthew Auld <matthew.auld at intel.com>
>> Cc: Matthew Brost <matthew.brost at intel.com>
>
>Probably a fixes tag even though this data path is not used at the moment?
yes, not sure what happened here.. I think when I added the Cc I ended
up replacing the Fixes I had added. I will add it back later when
merging.
>
>Reviewed-by: Matthew Brost <matthew.brost at intel.com>
thanks
Lucas De Marchi
More information about the Intel-xe
mailing list