[prev in list] [next in list] [prev in thread] [next in thread]
List: libvir-list
Subject: Re: [PATCH v14 07/15] domain_capabilities: Define SGX capabilities structs
From: Michal Prívozník <mprivozn () redhat ! com>
Date: 2022-07-28 14:05:04
Message-ID: 4c5999d2-ee58-b4fe-58dc-d4f54ef16a36 () redhat ! com
[Download RAW message or body]
On 7/28/22 10:35, Daniel P. Berrangé wrote:
> On Thu, Jul 28, 2022 at 10:04:23AM +0200, Peter Krempa wrote:
>> On Wed, Jul 27, 2022 at 12:34:53 +0200, Michal Privoznik wrote:
>>> From: Haibin Huang <haibin.huang@intel.com>
>>>
>>> Signed-off-by: Haibin Huang <haibin.huang@intel.com>
>>> Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
>>> ---
>>> src/conf/domain_capabilities.c | 11 +++++++++++
>>> src/conf/domain_capabilities.h | 22 ++++++++++++++++++++++
>>> src/libvirt_private.syms | 1 +
>>> 3 files changed, 34 insertions(+)
>>
>> [...]
>>
>>> diff --git a/src/conf/domain_capabilities.h b/src/conf/domain_capabilities.h
>>> index a526969cda..e4654a65cd 100644
>>> --- a/src/conf/domain_capabilities.h
>>> +++ b/src/conf/domain_capabilities.h
>>> @@ -193,6 +193,22 @@ struct _virSEVCapability {
>>> unsigned int max_es_guests;
>>> };
>>>
>>> +typedef struct _virSection virSection;
>>
>> This name feels a bit too generic and not much reusable. On the other
>> hand changing it feels pointless.
Ooops yes, of course. Been looking too much into these patches lately.
>
> Clearly it should be virSGXSection
Indeed, fixed.
Michal
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic