On Tue, Oct 27, 2020 at 02:55:40PM +0100, Philippe Mathieu-Daudé wrote: > We want to get ride of the BlockDriverState pointer at some point, > so pass aio_context along.
Potential future changes are a weak justification. Someone else might decide that the aio_context argument is redundant and reverse this change. Either way works, so this change is somewhat arbitrary at the moment. A strong justification would be if the next patch removes the BDS pointer argument. Until this change is necessary I would prefer not to include it. That saves reviewer time, reduces the risk of introducing bugs, etc.
signature.asc
Description: PGP signature
