[Spice-devel] [PATCH v6 03/10] Add SpiceUsbDeviceManager parameter to device comparison functions
Dmitry Fleytman
dmitry at daynix.com
Thu Feb 11 13:54:42 UTC 2016
> On 4 Feb 2016, at 21:58 PM, Jonathon Jongsma <jjongsma at redhat.com> 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 at redhat.com>
>
>
> On Thu, 2015-10-29 at 17:26 +0200, Dmitry Fleytman wrote:
>> From: Christophe Fergeau <cfergeau at redhat.com>
>>
>> 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;
>> }
More information about the Spice-devel
mailing list