Re: [PATCH v6 03/10] Add SpiceUsbDeviceManager parameter to device comparison functions

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

 



> On 4 Feb 2016, at 21:58 PM, Jonathon Jongsma <jjongsma@xxxxxxxxxx> wrote:
> 
> It doesn't seem that these new parameters are used until patch #7, unless I
> missed something. So I'd re-order the patch series so that this comes right
> before it's used (or even merge them together?).

I’d prefer to keep these changes in a separate patch to make history more clear. Reordering the series.
Thanks!

> 
> Acked-by: Jonathon Jongsma <jjongsma@xxxxxxxxxx>
> 
> 
> On Thu, 2015-10-29 at 17:26 +0200, Dmitry Fleytman wrote:
>> From: Christophe Fergeau <cfergeau@xxxxxxxxxx>
>> 
>> This additional parameter is currently unused, but this is in
>> preparation for the next commits.
>> ---
>> src/usb-device-manager.c | 40 ++++++++++++++++++++++------------------
>> 1 file changed, 22 insertions(+), 18 deletions(-)
>> 
>> diff --git a/src/usb-device-manager.c b/src/usb-device-manager.c
>> index d261dce..4930613 100644
>> --- a/src/usb-device-manager.c
>> +++ b/src/usb-device-manager.c
>> @@ -184,8 +184,9 @@ static guint8 spice_usb_device_get_state(SpiceUsbDevice
>> *device);
>> static void  spice_usb_device_set_state(SpiceUsbDevice *device, guint8 s);
>> #endif
>> 
>> -static gboolean spice_usb_device_equal_libdev(SpiceUsbDevice *device,
>> -                                              libusb_device *libdev);
>> +static gboolean spice_usb_manager_device_equal_libdev(SpiceUsbDeviceManager
>> *manager,
>> +                                                      SpiceUsbDevice *device,
>> +                                                      libusb_device *libdev);
>> static libusb_device *
>> spice_usb_device_manager_device_to_libdev(SpiceUsbDeviceManager *self,
>>                                           SpiceUsbDevice *device);
>> @@ -641,7 +642,8 @@ static void
>> spice_usb_device_manager_class_init(SpiceUsbDeviceManagerClass *klas
>> 
>> #ifdef USE_GUDEV
>> static gboolean spice_usb_device_manager_get_udev_bus_n_address(
>> -    GUdevDevice *udev, int *bus, int *address)
>> +    SpiceUsbDeviceManager *manager, GUdevDevice *udev,
>> +    int *bus, int *address)
>> {
>>     const gchar *bus_str, *address_str;
>> 
>> @@ -794,7 +796,7 @@ static void
>> spice_usb_device_manager_auto_connect_cb(GObject      *gobject,
>> 
>> #ifndef G_OS_WIN32 /* match functions for Linux -- match by bus.addr */
>> static gboolean
>> -spice_usb_device_manager_device_match(SpiceUsbDevice *device,
>> +spice_usb_device_manager_device_match(SpiceUsbDeviceManager *self,
>> SpiceUsbDevice *device,
>>                                       const int bus, const int address)
>> {
>>     return (spice_usb_device_get_busnum(device) == bus &&
>> @@ -803,7 +805,7 @@ spice_usb_device_manager_device_match(SpiceUsbDevice
>> *device,
>> 
>> #ifdef USE_GUDEV
>> static gboolean
>> -spice_usb_device_manager_libdev_match(libusb_device *libdev,
>> +spice_usb_device_manager_libdev_match(SpiceUsbDeviceManager *self,
>> libusb_device *libdev,
>>                                       const int bus, const int address)
>> {
>>     return (libusb_get_bus_number(libdev) == bus &&
>> @@ -813,7 +815,7 @@ spice_usb_device_manager_libdev_match(libusb_device
>> *libdev,
>> 
>> #else /* Win32 -- match functions for Windows -- match by vid:pid */
>> static gboolean
>> -spice_usb_device_manager_device_match(SpiceUsbDevice *device,
>> +spice_usb_device_manager_device_match(SpiceUsbDeviceManager *self,
>> SpiceUsbDevice *device,
>>                                       const int vid, const int pid)
>> {
>>     return (spice_usb_device_get_vid(device) == vid &&
>> @@ -821,7 +823,7 @@ spice_usb_device_manager_device_match(SpiceUsbDevice
>> *device,
>> }
>> 
>> static gboolean
>> -spice_usb_device_manager_libdev_match(libusb_device *libdev,
>> +spice_usb_device_manager_libdev_match(SpiceUsbDeviceManager *self,
>> libusb_device *libdev,
>>                                       const int vid, const int pid)
>> {
>>     int vid2, pid2;
>> @@ -843,7 +845,7 @@ spice_usb_device_manager_find_device(SpiceUsbDeviceManager
>> *self,
>> 
>>     for (i = 0; i < priv->devices->len; i++) {
>>         curr = g_ptr_array_index(priv->devices, i);
>> -        if (spice_usb_device_manager_device_match(curr, bus, address)) {
>> +        if (spice_usb_device_manager_device_match(self, curr, bus, address))
>> {
>>             device = curr;
>>             break;
>>         }
>> @@ -946,7 +948,7 @@ static void
>> spice_usb_device_manager_add_udev(SpiceUsbDeviceManager  *self,
>>     if (!devtype || strcmp(devtype, "usb_device"))
>>         return;
>> 
>> -    if (!spice_usb_device_manager_get_udev_bus_n_address(udev, &bus,
>> &address)) {
>> +    if (!spice_usb_device_manager_get_udev_bus_n_address(self, udev, &bus,
>> &address)) {
>>         g_warning("USB device without bus number or device address");
>>         return;
>>     }
>> @@ -967,7 +969,7 @@ static void
>> spice_usb_device_manager_add_udev(SpiceUsbDeviceManager  *self,
>>         libusb_get_device_list(priv->context, &dev_list);
>> 
>>     for (i = 0; dev_list && dev_list[i]; i++) {
>> -        if (spice_usb_device_manager_libdev_match(dev_list[i], bus, address))
>> {
>> +        if (spice_usb_device_manager_libdev_match(self, dev_list[i], bus,
>> address)) {
>>             libdev = dev_list[i];
>>             break;
>>         }
>> @@ -988,7 +990,7 @@ static void
>> spice_usb_device_manager_remove_udev(SpiceUsbDeviceManager  *self,
>> {
>>     int bus, address;
>> 
>> -    if (!spice_usb_device_manager_get_udev_bus_n_address(udev, &bus,
>> &address))
>> +    if (!spice_usb_device_manager_get_udev_bus_n_address(self, udev, &bus,
>> &address))
>>         return;
>> 
>>     spice_usb_device_manager_remove_dev(self, bus, address);
>> @@ -1291,7 +1293,7 @@ static SpiceUsbredirChannel
>> *spice_usb_device_manager_get_channel_for_dev(
>>     for (i = 0; i < priv->channels->len; i++) {
>>         SpiceUsbredirChannel *channel = g_ptr_array_index(priv->channels, i);
>>         libusb_device *libdev = spice_usbredir_channel_get_device(channel);
>> -        if (spice_usb_device_equal_libdev(device, libdev))
>> +        if (spice_usb_manager_device_equal_libdev(manager, device, libdev))
>>             return channel;
>>     }
>> #endif
>> @@ -1863,8 +1865,9 @@ static void spice_usb_device_unref(SpiceUsbDevice
>> *device)
>> 
>> #ifndef G_OS_WIN32 /* Linux -- directly compare libdev */
>> static gboolean
>> -spice_usb_device_equal_libdev(SpiceUsbDevice *device,
>> -                              libusb_device  *libdev)
>> +spice_usb_manager_device_equal_libdev(SpiceUsbDeviceManager *manager,
>> +                                      SpiceUsbDevice *device,
>> +                                      libusb_device  *libdev)
>> {
>>     SpiceUsbDeviceInfo *info = (SpiceUsbDeviceInfo *)device;
>> 
>> @@ -1875,8 +1878,9 @@ spice_usb_device_equal_libdev(SpiceUsbDevice *device,
>> }
>> #else /* Windows -- compare vid:pid of device and libdev */
>> static gboolean
>> -spice_usb_device_equal_libdev(SpiceUsbDevice *device,
>> -                              libusb_device  *libdev)
>> +spice_usb_manager_device_equal_libdev(SpiceUsbDeviceManager *manager,
>> +                                      SpiceUsbDevice *device,
>> +                                      libusb_device  *libdev)
>> {
>>     int vid, pid;
>> 
>> @@ -1885,7 +1889,7 @@ spice_usb_device_equal_libdev(SpiceUsbDevice *device,
>> 
>>     vid = spice_usb_device_get_vid(device);
>>     pid = spice_usb_device_get_pid(device);
>> -    return spice_usb_device_manager_libdev_match(libdev, vid, pid);
>> +    return spice_usb_device_manager_libdev_match(manager, libdev, vid, pid);
>> }
>> #endif
>> 
>> @@ -1918,7 +1922,7 @@
>> spice_usb_device_manager_device_to_libdev(SpiceUsbDeviceManager *self,
>>         return NULL;
>> 
>>     for (i = 0; (d = devlist[i]) != NULL; i++) {
>> -        if (spice_usb_device_equal_libdev(device, d)) {
>> +        if (spice_usb_manager_device_equal_libdev(self, device, d)) {
>>             libusb_ref_device(d);
>>             break;
>>         }

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]