[Differential] D14473: userboot: add callbacks to set unrestricted guest mode
grehan (Peter Grehan)
phabric-noreply at FreeBSD.org
Thu Feb 22 23:13:58 UTC 2018
grehan added inline comments.
INLINE COMMENTS
> bhyveload.c:566
> +{
> + return (vm_get_capability(ctx, vcpu,
> + VM_CAP_UNRESTRICTED_GUEST, retval));
Minor style(9) - should be an empty line before the rreturn - see the routine above.
> bhyveload.c:567
> + return (vm_get_capability(ctx, vcpu,
> + VM_CAP_UNRESTRICTED_GUEST, retval));
> +}
Minor style(9) - continued lines should be indented by 4 spaces.
> bhyveload.c:574
> + int error, tmp;
> + if ((error = cb_vm_get_unrestricted_guest(arg, vcpu, &tmp)))
> + return error;
Minor style(9) - blank line between declarations and statement.
> bhyveload.c:575
> + if ((error = cb_vm_get_unrestricted_guest(arg, vcpu, &tmp)))
> + return error;
> +
Minor style(9) - return expression should be enclosed in parens.
> bhyveload.c:584
> +{
> + return vcpu_reset(ctx, vcpu);
> +}
Minor style(9) - blank line needed before statement, and return expression should be in parens.
REPOSITORY
rS FreeBSD src repository
REVISION DETAIL
https://reviews.freebsd.org/D14473
EMAIL PREFERENCES
https://reviews.freebsd.org/settings/panel/emailpreferences/
To: fabian.freyer_physik.tu-berlin.de, imp, #bhyve, grehan
Cc: grehan, imp, freebsd-virtualization-list, #contributor_reviews_base
More information about the freebsd-virtualization
mailing list