linux-kernel - Re: [RFC PATCH 11/16] xen/grant-table: make grant-table xenhost aware

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <3986345a-2f34-52cd-7d5f-3cd1d8b3267e@oracle.com>
Date:   Tue, 18 Jun 2019 19:25:58 -0700
From:   Ankur Arora <ankur.a.arora@...cle.com>
To:     Juergen Gross <jgross@...e.com>, linux-kernel@...r.kernel.org,
        xen-devel@...ts.xenproject.org
Cc:     pbonzini@...hat.com, boris.ostrovsky@...cle.com,
        konrad.wilk@...cle.com, sstabellini@...nel.org,
        joao.m.martins@...cle.com
Subject: Re: [RFC PATCH 11/16] xen/grant-table: make grant-table xenhost aware

On 6/17/19 2:36 AM, Juergen Gross wrote:
> On 09.05.19 19:25, Ankur Arora wrote:
>> Largely mechanical changes: the exported grant table symbols now take
>> xenhost_t * as a parameter. Also, move the grant table global state
>> inside xenhost_t.
>>
>> If there's more than one xenhost, then initialize both.
>>
>> Signed-off-by: Ankur Arora <ankur.a.arora@...cle.com>
>> ---
>>   arch/x86/xen/grant-table.c |  71 +++--
>>   drivers/xen/grant-table.c  | 611 +++++++++++++++++++++----------------
>>   include/xen/grant_table.h  |  72 ++---
>>   include/xen/xenhost.h      |  11 +
>>   4 files changed, 443 insertions(+), 322 deletions(-)
>>
>> diff --git a/include/xen/xenhost.h b/include/xen/xenhost.h
>> index 9e08627a9e3e..acee0c7872b6 100644
>> --- a/include/xen/xenhost.h
>> +++ b/include/xen/xenhost.h
>> @@ -129,6 +129,17 @@ typedef struct {
>>           const struct evtchn_ops *evtchn_ops;
>>           int **evtchn_to_irq;
>>       };
>> +
>> +    /* grant table private state */
>> +    struct {
>> +        /* private to drivers/xen/grant-table.c */
>> +        void *gnttab_private;
>> +
>> +        /* x86/xen/grant-table.c */
>> +        void *gnttab_shared_vm_area;
>> +        void *gnttab_status_vm_area;
>> +        void *auto_xlat_grant_frames;
> 
> Please use proper types here instead of void *. This avoids lots of
> casts. It is okay to just add anonymous struct definitions and keep the
> real struct layout local to grant table code.
Will fix.

Ankur

> 
> 
> Juergen