[vbox-dev] Changeset 36350 from SVN breaks USB subset
Perry Halbert
phalbert at cox.net
Fri Mar 25 12:59:27 GMT 2011
SVN updated to r36422 and no problems with the USB subset in Ubuntu 10.10.
Looks like you have fixed the issue my friend.
I install from SVN usually the day you release it so I can test it all
the way to packing and installing.
Testing on Ubuntu, Debian, openSUSE, Fedora, and Solaris.
On 03/25/2011 05:22 AM, Michael Thayer wrote:
> Perry,
>
> I hope that current HEAD will work for you again. If you give it a try
> please let me know!
>
> Regards,
>
> Michael
>
> On Thu, 2011-03-24 at 09:01 -0500, Perry Halbert wrote:
>> Michael,
>>
>> It looks like the second patch you sent actually seems to fix the
>> problem. I had failed to reverse the first patch before I ran the
>> second one and it looks like it got confused along with me. I will
>> check further today and see what happens.
>>
>>
>>
>> On 03/23/2011 07:52 PM, Perry Halbert wrote:
>>> Well thinking that it was some left over src code I purged the
>> build
>>> environment and rebuilt using 1) the latest from SVN and then 2)
>> with
>>> you latest patch. Neither seemed to make a difference. I will see
>> if
>>> I can tweak it some more tomorrow and let you know but I suspect
>> the
>>> negations are causing the problem,especially since I can put them
>> back
>>> and all is happy again. Seems something else is looking for these
>>> removed snippets somewhere and are not at all happy that they are
>> missing.
>>> Oh happy days, right :-)
>>>
>>>
>>>
>>> On 03/23/2011 04:58 PM, Michael Thayer wrote:
>>>> On Wed, 2011-03-23 at 16:52 -0500, Perry Halbert wrote:
>>>>> Sorry that doesn't get it either.
>>>>> I read that link a few days ago but remember I had no problems
>> until
>>>>> the
>>>>> changes in 36350 today.
>>>> I take it it builds correctly for you this time though. Did you
>> check
>>>> whether the device nodes are correctly created under /dev/vboxusb
>> when
>>>> you plug in devices? And you might also check which of the two
>> entries
>>>> which I removed from that structure was the important one. I
>> presume it
>>>> was
>>>> { "/dev/bus/usb", true }
>>>>
>>>> Regards,
>>>>
>>>> Michael
>>>>
>>>>> On 03/23/2011 04:34 PM, Michael Thayer wrote:
>>>>>> On Wed, 2011-03-23 at 16:15 -0500, Perry Halbert wrote:
>>>>>>> Well going back to the present build does not work. Node is not
>>>>>>> created
>>>>>>> and and the error returns.
>>>>>>> I added the changes that you show in previous reply and get an
>>>>> error
>>>>>>> shown below
>>>>>> Sorry about that! Here is the correct patch.
>>>>>>
>>>>>> By the way, see also [http://www.virtualbox.org/ticket/8502 ].
>>>>>>
>>>>>> Regards,
>>>>>>
>>>>>> Michael
>>>>>>
>>>>>> Index: src/VBox/Main/src-server/linux/USBGetDevices.cpp
>>>>>>
>> ===================================================================
>>>>>> --- src/VBox/Main/src-server/linux/USBGetDevices.cpp (revision
>>>>> 70699)
>>>>>> +++ src/VBox/Main/src-server/linux/USBGetDevices.cpp (working
>> copy)
>>>>>> @@ -1442,17 +1442,7 @@
>>>>>> if ( fHaveInotify
>>>>>> && !pcBestSysfs
>>>>>> &&
>>>>> RTPathExists(s_aTreeLocations[i].szDevicesRoot))
>>>>>> - {
>>>>>> - PUSBDEVICE pDevices;
>>>>>> -
>>>>>> - pDevices =
>>>>> getDevicesFromSysfs(s_aTreeLocations[i].szDevicesRoot,
>>>>>> - true);
>>>>>> - if (pDevices)
>>>>>> - {
>>>>>> - pcBestSysfs =&s_aTreeLocations[i];
>>>>>> - deviceListFree(&pDevices);
>>>>>> - }
>>>>>> - }
>>>>>> + pcBestSysfs =&s_aTreeLocations[i];
>>>>>> }
>>>>>> if (pcBestUsbfs&& !fPreferSysfs)
>>>>>> return pcBestUsbfs;
>>>>>>
>>>>>>
>>>>>>
>>>>>>> Reverting and nodes do show up in /dev/vboxusb
>>>>>>>
>>>>>>> This works
>>>>>>> static const USBDEVTREELOCATION s_aTreeLocations[] =
>>>>>>> {
>>>>>>> { "/proc/bus/usb", false },
>>>>>>> { "/dev/bus/usb", false },
>>>>>>> { "/dev/vboxusb", true },
>>>>>>> { "/dev/bus/usb", true },
>>>>>>> };
>>>>>>>
>>>>>>> This does not work
>>>>>>> static const USBDEVTREELOCATION s_aTreeLocations[] =
>>>>>>> {
>>>>>>> { "/dev/vboxusb", true },
>>>>>>> { "/proc/bus/usb", false },
>>>>>>> };
>>>>>>>
>>>>>>>
>>>>>>> On 03/23/2011 03:30 PM, Michael Thayer wrote:
>>>>>>>> On Wed, 2011-03-23 at 15:17 -0500, Perry Halbert wrote:
>>>>>>>>> By additions this time I was talking about the additional
>> changes
>>>>>>> to
>>>>>>>>> the
>>>>>>>>> file.
>>>>>>>>>
>>>>>>>>> Give me a bit and I will see what happens. Takes about 10
>>>>> minutes
>>>>>>> to
>>>>>>>>> get back to a clean build.
>>>>>>>> If that makes a difference you might try with the following
>> patch.
>>>>>>>> Regards,
>>>>>>>>
>>>>>>>> Michael
>>>>>>>>
>>>>>>>> Index: src/VBox/Main/src-server/linux/USBGetDevices.cpp
>>>>>>>>
>> ===================================================================
>>>>>>>> --- src/VBox/Main/src-server/linux/USBGetDevices.cpp (revision
>>>>>>> 70699)
>>>>>>>> +++ src/VBox/Main/src-server/linux/USBGetDevices.cpp (working
>>>>> copy)
>>>>>>>> @@ -1443,15 +1443,8 @@
>>>>>>>> && !pcBestSysfs
>>>>>>>> &&
>>>>>>> RTPathExists(s_aTreeLocations[i].szDevicesRoot))
>>>>>>>> {
>>>>>>>> - PUSBDEVICE pDevices;
>>>>>>>> -
>>>>>>>> - pDevices =
>>>>>>> getDevicesFromSysfs(s_aTreeLocations[i].szDevicesRoot,
>>>>>>>> - true);
>>>>>>>> - if (pDevices)
>>>>>>>> - {
>>>>>>>> - pcBestSysfs =&s_aTreeLocations[i];
>>>>>>>> - deviceListFree(&pDevices);
>>>>>>>> - }
>>>>>>>> + pcBestSysfs =&s_aTreeLocations[i];
>>>>>>>> + deviceListFree(&pDevices);
>>>>>>>> }
>>>>>>>> }
>>>>>>>> if (pcBestUsbfs&& !fPreferSysfs)
>>>>>>>>
>>>>>>>>
>>>>>>>>> On 03/23/2011 03:14 PM, Michael Thayer wrote:
>>>>>>>>>> On Wed, 2011-03-23 at 15:06 -0500, Perry Halbert wrote:
>>>>>>>>>>> Yes it had the directory. I don't know if it actually
>> created
>>>>> it
>>>>>>>>> but
>>>>>>>>>>> when investigating it was there.
>>>>>>>>>>>
>>>>>>>>>>> I can rebuild with the additions if you want for me to look
>> for
>>>>>>>>>>> anything
>>>>>>>>>>> specific.
>>>>>>>>>> The Additions shouldn't play any part here. I will give you
>>>>>>> another
>>>>>>>>>> test though - try stopping all VirtualBox processes, plugging
>> in
>>>>> a
>>>>>>>>> USB
>>>>>>>>>> device (a node should be created in /dev/vboxusb for it) and
>>>>>>>>> restarting
>>>>>>>>>> VirtualBox.
>>>>>>>>>>
>>>>>>>>>> Regards,
>>>>>>>>>>
>>>>>>>>>> Michael
>>>>>>>>>>
>>>>>>>>>>> On 03/23/2011 02:58 PM, Michael Thayer wrote:
>>>>>>>>>>>> Hello Perry,
>>>>>>>>>>>>
>>>>>>>>>>>> On Wed, 2011-03-23 at 14:52 -0500, Perry Halbert wrote:
>>>>>>>>>>>>> Changeset 36350 breaks the USB subset
>>>>>>>>>>>>> trunk/src/VBox/Main/src-server/linux/USBGetDevices.cpp
>>>>>>> (modified)
>>>>>>>>>>> (1 diff)
>>>>>>>>>>>>> Host Ubuntu 10.10 x86_32
>>>>>>>>>>>>> Removing the change fixes it.
>>>>>>>>>>>> Does the directory /dev/vboxusb exist on your system? Does
>>>>>>>>> creating
>>>>>>>>>>> it
>>>>>>>>>>>> make a difference if not?
>>>>>>>>>>>>
>>>>>>>>>>>> Regards,
>>>>>>>>>>>>
>>>>>>>>>>>> Michael
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.virtualbox.org/pipermail/vbox-dev/attachments/20110325/b66275b2/attachment.html>
More information about the vbox-dev
mailing list