[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 02/23] qapi/misc-target: Wrap long 'SEV Attestation Report
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [PATCH v2 02/23] qapi/misc-target: Wrap long 'SEV Attestation Report' long lines |
Date: |
Thu, 24 Jun 2021 12:04:26 +0100 |
User-agent: |
Mutt/2.0.7 (2021-05-04) |
* Philippe Mathieu-Daudé (philmd@redhat.com) wrote:
> Wrap long lines before 70 characters for legibility.
>
> Suggested-by: Markus Armbruster <armbru@redhat.com>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
> Note: it would be nice if checkpatch enforce 70 char for json
> (or at least QAPI json), that would save future
> developer modifying QAPI definitions reformating time.
> ---
> qapi/misc-target.json | 17 ++++++++++-------
> 1 file changed, 10 insertions(+), 7 deletions(-)
>
> diff --git a/qapi/misc-target.json b/qapi/misc-target.json
> index 5573dcf8f08..81646126267 100644
> --- a/qapi/misc-target.json
> +++ b/qapi/misc-target.json
> @@ -290,8 +290,8 @@
> ##
> # @SevAttestationReport:
> #
> -# The struct describes attestation report for a Secure Encrypted
> Virtualization
> -# feature.
> +# The struct describes attestation report for a Secure Encrypted
> +# Virtualization feature.
> #
> # @data: guest attestation report (base64 encoded)
> #
> @@ -305,10 +305,11 @@
> ##
> # @query-sev-attestation-report:
> #
> -# This command is used to get the SEV attestation report, and is supported
> on AMD
> -# X86 platforms only.
> +# This command is used to get the SEV attestation report, and is
> +# supported on AMD X86 platforms only.
> #
> -# @mnonce: a random 16 bytes value encoded in base64 (it will be included in
> report)
> +# @mnonce: a random 16 bytes value encoded in base64 (it will be
> +# included in report)
> #
> # Returns: SevAttestationReport objects.
> #
> @@ -316,10 +317,12 @@
> #
> # Example:
> #
> -# -> { "execute" : "query-sev-attestation-report", "arguments": { "mnonce":
> "aaaaaaa" } }
> +# -> { "execute" : "query-sev-attestation-report",
> + "arguments": { "mnonce": "aaaaaaa" } }
Shouldn't that start with a # ?
(Actually, last time I tried this command, I think I realised that we
should also put a real nonce value in there; it requirs a 16 byte value)
Dave
> # <- { "return" : { "data": "aaaaaaaabbbddddd"} }
> #
> ##
> -{ 'command': 'query-sev-attestation-report', 'data': { 'mnonce': 'str' },
> +{ 'command': 'query-sev-attestation-report',
> + 'data': { 'mnonce': 'str' },
> 'returns': 'SevAttestationReport',
> 'if': 'defined(TARGET_I386)' }
> --
> 2.31.1
>
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
- [PATCH v2 00/23] target/i386/sev: Housekeeping (OVMF + SEV-disabled binaries), Philippe Mathieu-Daudé, 2021/06/16
- [PATCH v2 02/23] qapi/misc-target: Wrap long 'SEV Attestation Report' long lines, Philippe Mathieu-Daudé, 2021/06/16
- Re: [PATCH v2 02/23] qapi/misc-target: Wrap long 'SEV Attestation Report' long lines,
Dr. David Alan Gilbert <=
- [PATCH v2 01/23] MAINTAINERS: Add Connor Kuehl as reviewer for AMD SEV, Philippe Mathieu-Daudé, 2021/06/16
- [PATCH v2 03/23] qapi/misc-target: Group SEV QAPI definitions, Philippe Mathieu-Daudé, 2021/06/16
- [PATCH v2 05/23] target/i386/cpu: Add missing 'qapi/error.h' header, Philippe Mathieu-Daudé, 2021/06/16
- [PATCH v2 04/23] target/i386/monitor: Return QMP error when SEV is disabled in build, Philippe Mathieu-Daudé, 2021/06/16
- [PATCH v2 06/23] target/i386/sev_i386.h: Remove unused headers, Philippe Mathieu-Daudé, 2021/06/16
- [PATCH v2 07/23] target/i386/sev: Remove sev_get_me_mask(), Philippe Mathieu-Daudé, 2021/06/16
- [PATCH v2 08/23] target/i386/sev: Mark unreachable code with g_assert_not_reached(), Philippe Mathieu-Daudé, 2021/06/16
- [PATCH v2 09/23] target/i386/sev: sev_get_attestation_report use g_autofree, Philippe Mathieu-Daudé, 2021/06/16