[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