Re: [PATCH 03/20] drivers: prefer unsigned int for flags

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 07/06/2011 09:07 PM, Daniel Veillard wrote:
> On Wed, Jul 06, 2011 at 05:23:46PM -0600, Eric Blake wrote:
>> Now that the public APIs always use unsigned flags, the internal
>> driver callbacks might as well do likewise.
>>

>>  struct remote_nwfilter_get_xml_desc_args {
>>          remote_nonnull_nwfilter    nwfilter;
>> -        int                        flags;
>> +        u_int                      flags;
>>  };
>>  struct remote_nwfilter_get_xml_desc_ret {
>>          remote_nonnull_string      xml;
> 
>   Can we triple check that the remote stuff really allocate the same
> space for int and u_int, that's my only doubt in that change,

We've done that exact type of change before - see commit 1ff2b6f.

> 
>   ACK,

I've pushed 1-5, and am slowly working on rebasing the remaining patches
to deal with Matthias' observation about read-only connections (my
debugging session proved that we _do_ pass a flag to all the driver open
functions).

-- 
Eric Blake   eblake@xxxxxxxxxx    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]