On 22/8/25 11:24, CJ Chen wrote:
When it comes to this pattern: .valid.unaligned = false and
impl.unaligned = true, is effectlvely contradictory. The .valid
structure indicates that unaligned access should be rejected at
the access validation phase, yet .impl suggests the underlying
device implementation can handle unaligned operations. As a result,
the upper-layer code will never even reach the .impl logic, leading
to confusion.

Signed-off-by: CJ Chen <[email protected]>
Tested-by: CJ Chen <[email protected]>

It is normal for contributors to test their patches ;)

Suggested-by: Peter Xu <[email protected]>
Acked-by: Tomoyuki Hirose <[email protected]>
---
  system/memory.c | 1 +
  1 file changed, 1 insertion(+)

Reviewed-by: Philippe Mathieu-Daudé <[email protected]>


Reply via email to