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?
