[Top][All Lists]

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

Re: [Qemu-devel] [PATCH v3 2/2] usb-mtp: added object properties

From: Bandan Das
Subject: Re: [Qemu-devel] [PATCH v3 2/2] usb-mtp: added object properties
Date: Tue, 15 Nov 2016 16:55:14 -0500
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux)

Gerd Hoffmann <address@hidden> writes:

>   Hi,
>> and object properties. Most object properties are data that we already
>> have, except for the unique persistant object identifier. Windows
>> +        /* Should be persistant between sessions,
>> +         * but using our objedt ID is "good enough"
>> +         * for now */
>> +        usb_mtp_add_u64(d, 0x0000000000000000);
>> +        usb_mtp_add_u64(d, o->handle);
>> +        break;
> Didn't I mention for an older version already that the inode number
> would be a better fit?  The inode number is persistent and unique per
> filesystem.  While being at it we might also add the device id, to
> handle the (unlikely) case that the mtp device spans multiple
> filesystems.

Seems Isaac is busy with school and stuff. I will pick these up and post
a follow up soon.


> We stat() the file already and cache the result, so this is as simple as
> using o->stat.st_{dev,ino}.  See "man 2 stat" for details.
> Other than that the patch series looks fine.
> Sorry for the delay.  Now that vacation time, kvm forum and qemu 2.7
> freeze are over things should go faster for v4.
> cheers,
>   Gerd

reply via email to

[Prev in Thread] Current Thread [Next in Thread]