On 01/03/2016 13:14, Markus Armbruster wrote: >> > Can you explain why we don't do that anymore to someone who hasn't read >> > patches 4 to 28? :-) Is it patch 23? > Yes, but you also need 24 to complete the job. > > Commit message could perhaps explain it like this: > > ivshmem_realize() puts the shared memory region in a container > region. Used to be necessary to permit delayed mapping of the > shared memory. However, we recently moved to synchronous mapping, > in "ivshmem: Receive shared memory synchronously in realize()" and > the commit following it. The container is redundant since then. > Drop it. > > Better?
Yes, thanks! Paolo
