<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#ffffff">
<font face="Verdana">Well going back to the present build does not
work. Node is not created and and the error returns.<br>
I added the changes that you show in previous reply and get an
error shown below<br>
<br>
<small><small>kBuild: Compiling VBoxSVC -
/home/perry/compile/vbox/src/VBox/Main/src-server/linux/USBGetDevices.cpp<br>
/home/perry/compile/vbox/src/VBox/Main/src-server/linux/USBGetDevices.cpp:
In function ‘const USBDEVTREELOCATION*
USBProxyLinuxGetDeviceRoot(bool)’:<br>
/home/perry/compile/vbox/src/VBox/Main/src-server/linux/USBGetDevices.cpp:1447:
error: ‘pDevices’ was not declared in this scope<br>
</small></small><br>
Reverting and nodes do show up in /dev/vboxusb<br>
<br>
This works<br>
static const USBDEVTREELOCATION s_aTreeLocations[] =<br>
{<br>
{ "/proc/bus/usb", false }, <br>
{ "/dev/bus/usb", false }, <br>
{ "/dev/vboxusb", true }, <br>
{ "/dev/bus/usb", true }, <br>
};<br>
<br>
This does not work<br>
static const USBDEVTREELOCATION s_aTreeLocations[] =<br>
{<br>
{ "/dev/vboxusb", true },<br>
{ "/proc/bus/usb", false },<br>
};<br>
<br>
</font><br>
On 03/23/2011 03:30 PM, Michael Thayer wrote:
<blockquote
cite="mid:1300912225.2461.10.camel@michael-latitude-e6410"
type="cite">
<pre wrap="">On Wed, 2011-03-23 at 15:17 -0500, Perry Halbert wrote:
</pre>
<blockquote type="cite">
<pre wrap="">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.
</pre>
</blockquote>
<pre wrap="">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)
</pre>
<blockquote type="cite">
<pre wrap="">On 03/23/2011 03:14 PM, Michael Thayer wrote:
</pre>
<blockquote type="cite">
<pre wrap="">On Wed, 2011-03-23 at 15:06 -0500, Perry Halbert wrote:
</pre>
<blockquote type="cite">
<pre wrap="">Yes it had the directory. I don't know if it actually created it
</pre>
</blockquote>
</blockquote>
<pre wrap="">but
</pre>
<blockquote type="cite">
<blockquote type="cite">
<pre wrap="">when investigating it was there.
I can rebuild with the additions if you want for me to look for
anything
specific.
</pre>
</blockquote>
<pre wrap="">The Additions shouldn't play any part here. I will give you another
test though - try stopping all VirtualBox processes, plugging in a
</pre>
</blockquote>
<pre wrap="">USB
</pre>
<blockquote type="cite">
<pre wrap="">device (a node should be created in /dev/vboxusb for it) and
</pre>
</blockquote>
<pre wrap="">restarting
</pre>
<blockquote type="cite">
<pre wrap="">VirtualBox.
Regards,
Michael
</pre>
<blockquote type="cite">
<pre wrap="">On 03/23/2011 02:58 PM, Michael Thayer wrote:
</pre>
<blockquote type="cite">
<pre wrap="">Hello Perry,
On Wed, 2011-03-23 at 14:52 -0500, Perry Halbert wrote:
</pre>
<blockquote type="cite">
<pre wrap="">Changeset 36350 breaks the USB subset
trunk/src/VBox/Main/src-server/linux/USBGetDevices.cpp (modified)
</pre>
</blockquote>
</blockquote>
<pre wrap="">(1 diff)
</pre>
<blockquote type="cite">
<blockquote type="cite">
<pre wrap="">Host Ubuntu 10.10 x86_32
Removing the change fixes it.
</pre>
</blockquote>
<pre wrap="">Does the directory /dev/vboxusb exist on your system? Does
</pre>
</blockquote>
</blockquote>
</blockquote>
<pre wrap="">creating
</pre>
<blockquote type="cite">
<blockquote type="cite">
<pre wrap="">it
</pre>
<blockquote type="cite">
<pre wrap="">make a difference if not?
Regards,
Michael
</pre>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</body>
</html>