linux-kernel - Re: [PATCH resend v2 2/3] virt: Add vboxguest VMMDEV communication code

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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <504eda6e-98b2-0ec4-5293-7998595fe9d3@redhat.com>
Date:   Wed, 29 Nov 2017 14:17:25 +0100
From:   Hans de Goede <hdegoede@...hat.com>
To:     Larry Finger <Larry.Finger@...inger.net>,
        Arnd Bergmann <arnd@...db.de>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Cc:     Michael Thayer <michael.thayer@...cle.com>,
        "Knut St . Osmundsen" <knut.osmundsen@...cle.com>,
        Christoph Hellwig <hch@...radead.org>,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH resend v2 2/3] virt: Add vboxguest VMMDEV communication
 code

Hi,

On 27-11-17 20:46, Larry Finger wrote:
> On 11/26/2017 09:12 AM, Hans de Goede wrote:
>> This commits adds a header describing the hardware interface for the
>> Virtual Box Guest PCI device used in Virtual Box virtual machines and
>> utility functions for talking to the Virtual Box hypervisor over this
>> interface.
>>
>> These utility functions will used both by the vboxguest driver for the
>> PCI device which offers the /dev/vboxguest ioctl API and by the vboxfs
>> driver which offers folder sharing support.
>>
>> Signed-off-by: Hans de Goede <hdegoede@...hat.com>
> 
> Reviewed-by: Larry Finger <Larry.Finger.net>
> 
> This patch has only minor defects. Please see the inline comments.
> 
>> ---
>> Changes in v2:
>> -Change all uapi headers to kernel coding style: Drop struct and enum typedefs
>>   make type and struct-member names all lowercase, enum values all uppercase.
>> -Remove unused struct type declarations from some headers (shaving of another
>>   1000 lines)
>> -Remove or fixup doxygen style comments
>> -Get rid of CHECK macros, use a function taking in_ and out_size args instead
>> -Some other small codyingstyle fixes
>> -Split into multiple patches

<snip>

>> +
>> +/**
>> + * Translates linear address types to page list direction flags.
>> + *
>> + * Return: page list flags.
>> + * @type:  The type.
>> + */
>> +static u32 hgcm_call_linear_addr_type_to_pagelist_flags(
>> +    enum vmmdev_hgcm_function_parameter_type type)
>> +{
>> +    switch (type) {
>> +    default:
>> +        WARN_ON(1);
> 
> Do you intend a fall through here? If so, please add a comment to that effect.

Yes, comment added for v3.

>> +    case VMMDEV_HGCM_PARM_TYPE_LINADDR:
>> +    case VMMDEV_HGCM_PARM_TYPE_LINADDR_KERNEL:
>> +        return VMMDEV_HGCM_F_PARM_DIRECTION_BOTH;
>> +
>> +    case VMMDEV_HGCM_PARM_TYPE_LINADDR_IN:
>> +    case VMMDEV_HGCM_PARM_TYPE_LINADDR_KERNEL_IN:
>> +        return VMMDEV_HGCM_F_PARM_DIRECTION_TO_HOST;
>> +
>> +    case VMMDEV_HGCM_PARM_TYPE_LINADDR_OUT:
>> +    case VMMDEV_HGCM_PARM_TYPE_LINADDR_KERNEL_OUT:
>> +        return VMMDEV_HGCM_F_PARM_DIRECTION_FROM_HOST;
>> +    }
>> +}

Regards,

Hans