On Wed, 2015-12-09 at 13:02 -0500, John Ferlan wrote:
On 11/24/2015 08:56 AM, Andrea Bolognani wrote:
[...]
> + }
> +
> + /* Don't do anything unless we're actually setting a limit */
> + if (bytes) {
> + if (virProcessSetMaxMemLock(vm->pid, bytes) < 0)
> + goto out;
> + }
virProcessSetMaxMemLock aready checks:
if (bytes == 0)
return 0;
So the "if (bytes)" is unnecessary
Good catch, I will remove the check and update the comment.
Cheers.
--
Andrea Bolognani
Software Engineer - Virtualization Team