Re: [libvirt] [PATCH] qemu: process: Fix automatic setting of locked memory limit for VFIO

On Wed, Nov 04, 2015 at 17:16:53 -0700, Alex Williamson wrote: [...] The wrong think is that it doesn't work all the time. See below... Indeed, that is a separate bug, and if we figure out how to make this work all the time I'll fix that separately. Setting it to max memory won't actually fix the first referenced bug. The bug can be reproduced even if you don't use max memory at all on power pc (I didn't manage to update the BZ yet though with this information). Using max memory for that would basically add yet another workaround for setting the mlock size large enough. The bug happens if you set up a guest with 1GiB of ram and pass a AMD FirePro 2270 graphics card into it. Libvirt sets the memory limit to 1+1GiB and starts qemu. qemu then aborts as the VFIO code cannot lock the memory. This does not happen on larger guests though (2GiB of ram or more) which leads to the suspicion that the limit doesn't take into account some kind of overhead. As the original comment in the code was hinting that this is just a guess and it proved to be unreliable we shouldn't special case such configurations. Setting it to max memory + 1G would actually work around the second mentioned bug to the current state. I'm happy to set something large enough, but the value we set must be the absolute upper bound of anything that might be necessary so that it will 'just work'. We decided to be nice enough to the users to set the limit to somethig that works so we shouldn't special case any configuration. Peter