On 5/19/21 2:40 PM, Philippe Mathieu-Daudé wrote:
On 5/19/21 9:20 PM, Richard Henderson wrote:
On 5/18/21 1:36 PM, Philippe Mathieu-Daudé wrote:
The series is decomposed as:
- cleanups (1-6)
- clean ldst API using macros (7-13)
- add aligned ldst methods (14)
- add aligned memory methods (15-16)
- similar changes in virtio (17-24)
- use the new methods on vring aligned values (25)

There are some checkpatch errors related to the macro used.

I think we should emphasize the atomicness of the access as opposed to
the alignedness.  That's the only thing that's important to virtio.

Thus s/aligned/atomic/g and use qatomic_read/qatomic_set.

OK.

Do you think patches 1-6, 17 (Use correct type sizes) could
go in meanwhile, or should I repost them separately?


I think 1-6 can be queued; I'm had a question about 17.


r~

Reply via email to