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
--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list