<feed xmlns='http://www.w3.org/2005/Atom'>
<title>linux-stable.git/tools/usb/usbip, branch linux-4.12.y</title>
<subtitle>Linux kernel stable tree</subtitle>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux-stable.git/'/>
<entry>
<title>usb: fix some references for /proc/bus/usb</title>
<updated>2017-04-18T14:54:19+00:00</updated>
<author>
<name>Mauro Carvalho Chehab</name>
<email>mchehab@s-opensource.com</email>
</author>
<published>2017-04-17T00:51:07+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux-stable.git/commit/?id=21470e32ca7f976bf131aa3c7b54019d07f7d821'/>
<id>21470e32ca7f976bf131aa3c7b54019d07f7d821</id>
<content type='text'>
Since when we got rid of usbfs, the /proc/bus/usb is now
elsewhere. Fix references for it.

Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@s-opensource.com&gt;
Acked-by: Serge Hallyn &lt;serge@hallyn.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Since when we got rid of usbfs, the /proc/bus/usb is now
elsewhere. Fix references for it.

Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@s-opensource.com&gt;
Acked-by: Serge Hallyn &lt;serge@hallyn.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>usb: usbip tool: Fix get_nports()</title>
<updated>2017-04-08T10:17:42+00:00</updated>
<author>
<name>Yuyang Du</name>
<email>yuyang.du@intel.com</email>
</author>
<published>2017-04-05T22:03:23+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux-stable.git/commit/?id=37e47d5cfeec1dcc8246401ce84f2780f99ce990'/>
<id>37e47d5cfeec1dcc8246401ce84f2780f99ce990</id>
<content type='text'>
The commit 0775a9cbc694e8c72 ("usbip: vhci extension: modifications
to vhci driver") introduced multiple controllers, and nports as a sys
file, and claimed to read the nports from it, but it didn't.

In addition, the get_nports() has been so wrong that even with 8 port
lines for instance, it gets 7 (I am guessing it is due to a '\n' mess).
Nevertheless, we fix it by reading the nports attribute.

Reviewed-by: Krzysztof Opasiak &lt;k.opasiak@samsung.com&gt;
Signed-off-by: Yuyang Du &lt;yuyang.du@intel.com&gt;
Acked-by: Shuah Khan &lt;shuahkh@osg.samsung.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The commit 0775a9cbc694e8c72 ("usbip: vhci extension: modifications
to vhci driver") introduced multiple controllers, and nports as a sys
file, and claimed to read the nports from it, but it didn't.

In addition, the get_nports() has been so wrong that even with 8 port
lines for instance, it gets 7 (I am guessing it is due to a '\n' mess).
Nevertheless, we fix it by reading the nports attribute.

Reviewed-by: Krzysztof Opasiak &lt;k.opasiak@samsung.com&gt;
Signed-off-by: Yuyang Du &lt;yuyang.du@intel.com&gt;
Acked-by: Shuah Khan &lt;shuahkh@osg.samsung.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>usbip: Fix implicit fallthrough warning</title>
<updated>2017-03-17T04:16:56+00:00</updated>
<author>
<name>Jonathan Dieter</name>
<email>jdieter@lesbg.com</email>
</author>
<published>2017-02-27T08:31:04+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux-stable.git/commit/?id=cfd6ed4537a9e938fa76facecd4b9cd65b6d1563'/>
<id>cfd6ed4537a9e938fa76facecd4b9cd65b6d1563</id>
<content type='text'>
GCC 7 now warns when switch statements fall through implicitly, and with
-Werror enabled in configure.ac, that makes these tools unbuildable.

We fix this by notifying the compiler that this particular case statement
is meant to fall through.

Reviewed-by: Peter Senna Tschudin &lt;peter.senna@gmail.com&gt;
Signed-off-by: Jonathan Dieter &lt;jdieter@lesbg.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
GCC 7 now warns when switch statements fall through implicitly, and with
-Werror enabled in configure.ac, that makes these tools unbuildable.

We fix this by notifying the compiler that this particular case statement
is meant to fall through.

Reviewed-by: Peter Senna Tschudin &lt;peter.senna@gmail.com&gt;
Signed-off-by: Jonathan Dieter &lt;jdieter@lesbg.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>usbip: Fix potential format overflow in userspace tools</title>
<updated>2017-03-17T04:16:56+00:00</updated>
<author>
<name>Jonathan Dieter</name>
<email>jdieter@lesbg.com</email>
</author>
<published>2017-02-27T08:31:03+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux-stable.git/commit/?id=e5dfa3f902b9a642ae8c6997d57d7c41e384a90b'/>
<id>e5dfa3f902b9a642ae8c6997d57d7c41e384a90b</id>
<content type='text'>
The usbip userspace tools call sprintf()/snprintf() and don't check for
the return value which can lead the paths to overflow, truncating the
final file in the path.

More urgently, GCC 7 now warns that these aren't checked with
-Wformat-overflow, and with -Werror enabled in configure.ac, that makes
these tools unbuildable.

This patch fixes these problems by replacing sprintf() with snprintf() in
one place and adding checks for the return value of snprintf().

Reviewed-by: Peter Senna Tschudin &lt;peter.senna@gmail.com&gt;
Signed-off-by: Jonathan Dieter &lt;jdieter@lesbg.com&gt;
Acked-by: Shuah Khan &lt;shuahkh@osg.samsung.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The usbip userspace tools call sprintf()/snprintf() and don't check for
the return value which can lead the paths to overflow, truncating the
final file in the path.

More urgently, GCC 7 now warns that these aren't checked with
-Wformat-overflow, and with -Werror enabled in configure.ac, that makes
these tools unbuildable.

This patch fixes these problems by replacing sprintf() with snprintf() in
one place and adding checks for the return value of snprintf().

Reviewed-by: Peter Senna Tschudin &lt;peter.senna@gmail.com&gt;
Signed-off-by: Jonathan Dieter &lt;jdieter@lesbg.com&gt;
Acked-by: Shuah Khan &lt;shuahkh@osg.samsung.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>tools: usb: usbip: Update README</title>
<updated>2017-01-10T16:00:42+00:00</updated>
<author>
<name>Krzysztof Opasiak</name>
<email>k.opasiak@samsung.com</email>
</author>
<published>2016-12-20T19:53:14+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux-stable.git/commit/?id=444d930998f87ce2b5803866ecd171fdbd2c85ec'/>
<id>444d930998f87ce2b5803866ecd171fdbd2c85ec</id>
<content type='text'>
Update README file:
- remove outdated parts
- clarify terminology and general structure
- add some description of vUDC

Signed-off-by: Krzysztof Opasiak &lt;k.opasiak@samsung.com&gt;
Acked-by: Shuah Khan &lt;shuahkh@osg.samsung.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Update README file:
- remove outdated parts
- clarify terminology and general structure
- add some description of vUDC

Signed-off-by: Krzysztof Opasiak &lt;k.opasiak@samsung.com&gt;
Acked-by: Shuah Khan &lt;shuahkh@osg.samsung.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>tools: usb: usbip: Add simple script to show how to setup vUDC</title>
<updated>2017-01-10T16:00:41+00:00</updated>
<author>
<name>Krzysztof Opasiak</name>
<email>k.opasiak@samsung.com</email>
</author>
<published>2016-12-12T18:55:45+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux-stable.git/commit/?id=a52d3ec54a62c6e5e2fb177e45a1976fbd8eb287'/>
<id>a52d3ec54a62c6e5e2fb177e45a1976fbd8eb287</id>
<content type='text'>
Add some simple script which creates a USB gadget using ConfigFS
and then exports it using vUDC.

This may be useful for people who just started playing with
USB/IP and vUDC as it shows exact steps how to setup all stuff.

Signed-off-by: Krzysztof Opasiak &lt;k.opasiak@samsung.com&gt;
Acked-by: Shuah Khan &lt;shuahkh@osg.samsung.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Add some simple script which creates a USB gadget using ConfigFS
and then exports it using vUDC.

This may be useful for people who just started playing with
USB/IP and vUDC as it shows exact steps how to setup all stuff.

Signed-off-by: Krzysztof Opasiak &lt;k.opasiak@samsung.com&gt;
Acked-by: Shuah Khan &lt;shuahkh@osg.samsung.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>usbip: add missing compile time generated files to .gitignore</title>
<updated>2016-12-06T07:37:41+00:00</updated>
<author>
<name>Shuah Khan</name>
<email>shuahkh@osg.samsung.com</email>
</author>
<published>2016-12-06T00:51:24+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux-stable.git/commit/?id=71158c28eb434f3fe740b1560bbd698d56f506ee'/>
<id>71158c28eb434f3fe740b1560bbd698d56f506ee</id>
<content type='text'>
Add the following files to .gitignore

compile
libsrc/libusbip_la-sysfs_utils.lo
libsrc/libusbip_la-usbip_device_driver.lo
libsrc/libusbip_la-usbip_host_common.lo

Signed-off-by: Shuah Khan &lt;shuahkh@osg.samsung.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Add the following files to .gitignore

compile
libsrc/libusbip_la-sysfs_utils.lo
libsrc/libusbip_la-usbip_device_driver.lo
libsrc/libusbip_la-usbip_host_common.lo

Signed-off-by: Shuah Khan &lt;shuahkh@osg.samsung.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>usbip: deletion of incorrect socket descriptor checking</title>
<updated>2016-11-07T09:19:49+00:00</updated>
<author>
<name>Nobuo Iwata</name>
<email>nobuo.iwata@fujixerox.co.jp</email>
</author>
<published>2016-09-30T05:44:04+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux-stable.git/commit/?id=220c61b6f73a1d6f28591aeedff51a668068361b'/>
<id>220c61b6f73a1d6f28591aeedff51a668068361b</id>
<content type='text'>
This patch removes checking of socket descriptor value in daemons.

It was checked to be less than FD_SETSIZE(1024 usually) but it's not
correct.

To be exact, the maximum value of descriptor comes from
rlimit(RLIMIT_NOFILE).

Following kernel code determines the value :
    get_unused_fd_flags() : fs/files.c
    __alloc_fd() : fs/files.c
    expand_files() : fs/files.c

The defalut (soft limit) is defines as INR_OPEN_CUR(1024) in
include/linux/fs.h which is referenced form INIT_RLIMS in
include/asm-generic/resource.h. The value may be modified with ulimt,
sysctl, security configuration and etc.

With the kernel code above, when socket() system call returns positive
value, the value must be within rlimit(RLIMIT_NOFILE). No extra
checking is needed when socket() returns positive.

Without 'usbip: vhci number of ports extension' patch set, there's no
practical problem because of number of USB port restriction. With the
patch set, the value of socket descriptor can exceed FD_SETSIZE(1024
usually) if the rlimit is changed.

Signed-off-by: Nobuo Iwata &lt;nobuo.iwata@fujixerox.co.jp&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This patch removes checking of socket descriptor value in daemons.

It was checked to be less than FD_SETSIZE(1024 usually) but it's not
correct.

To be exact, the maximum value of descriptor comes from
rlimit(RLIMIT_NOFILE).

Following kernel code determines the value :
    get_unused_fd_flags() : fs/files.c
    __alloc_fd() : fs/files.c
    expand_files() : fs/files.c

The defalut (soft limit) is defines as INR_OPEN_CUR(1024) in
include/linux/fs.h which is referenced form INIT_RLIMS in
include/asm-generic/resource.h. The value may be modified with ulimt,
sysctl, security configuration and etc.

With the kernel code above, when socket() system call returns positive
value, the value must be within rlimit(RLIMIT_NOFILE). No extra
checking is needed when socket() returns positive.

Without 'usbip: vhci number of ports extension' patch set, there's no
practical problem because of number of USB port restriction. With the
patch set, the value of socket descriptor can exceed FD_SETSIZE(1024
usually) if the rlimit is changed.

Signed-off-by: Nobuo Iwata &lt;nobuo.iwata@fujixerox.co.jp&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>usbip: adding names db to port operation</title>
<updated>2016-05-03T21:32:07+00:00</updated>
<author>
<name>Nobuo Iwata</name>
<email>nobuo.iwata@fujixerox.co.jp</email>
</author>
<published>2016-03-22T07:31:03+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux-stable.git/commit/?id=e66fa8b08fbd87f375f964f1eaa1f5dfab9dc0c4'/>
<id>e66fa8b08fbd87f375f964f1eaa1f5dfab9dc0c4</id>
<content type='text'>
Adding names database to port command.

BEFORE) 'unknown' for vendor and product string.

Imported USB devices
====================
Port 00: &lt;Port in Use&gt; at Low Speed(1.5Mbps)
       unknown vendor : unknown product (03f0:0224)
       3-1 -&gt; usbip://10.0.2.15:3240/5-1
           -&gt; remote bus/dev 005/002

AFTER) Most vendor string will be converted.

Imported USB devices
====================
Port 00: &lt;Port in Use&gt; at Low Speed(1.5Mbps)
       Hewlett-Packard : unknown product (03f0:0224)
       3-1 -&gt; usbip://10.0.2.15:3240/5-1
           -&gt; remote bus/dev 005/002

Signed-off-by: Nobuo Iwata &lt;nobuo.iwata@fujixerox.co.jp&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Adding names database to port command.

BEFORE) 'unknown' for vendor and product string.

Imported USB devices
====================
Port 00: &lt;Port in Use&gt; at Low Speed(1.5Mbps)
       unknown vendor : unknown product (03f0:0224)
       3-1 -&gt; usbip://10.0.2.15:3240/5-1
           -&gt; remote bus/dev 005/002

AFTER) Most vendor string will be converted.

Imported USB devices
====================
Port 00: &lt;Port in Use&gt; at Low Speed(1.5Mbps)
       Hewlett-Packard : unknown product (03f0:0224)
       3-1 -&gt; usbip://10.0.2.15:3240/5-1
           -&gt; remote bus/dev 005/002

Signed-off-by: Nobuo Iwata &lt;nobuo.iwata@fujixerox.co.jp&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>usbip: tools: Start using VUDC backend in usbip tools</title>
<updated>2016-04-26T22:19:50+00:00</updated>
<author>
<name>Igor Kotrasinski</name>
<email>i.kotrasinsk@samsung.com</email>
</author>
<published>2016-03-08T20:49:06+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux-stable.git/commit/?id=e0546fd8b748b19d8edd1550530da8ebad6e4b31'/>
<id>e0546fd8b748b19d8edd1550530da8ebad6e4b31</id>
<content type='text'>
Modify userspace tools to allow exporting and connecting to vudc.

This commit is a result of cooperation between Samsung R&amp;D Institute
Poland and Open Operating Systems Student Society at University
of Warsaw (O2S3@UW) consisting of:

   Igor Kotrasinski &lt;ikotrasinsk@gmail.com&gt;
   Karol Kosik &lt;karo9@interia.eu&gt;
   Ewelina Kosmider &lt;3w3lfin@gmail.com&gt;
   Dawid Lazarczyk &lt;lazarczyk.dawid@gmail.com&gt;
   Piotr Szulc &lt;ps347277@students.mimuw.edu.pl&gt;

Tutor and project owner:
   Krzysztof Opasiak &lt;k.opasiak@samsung.com&gt;

Signed-off-by: Igor Kotrasinski &lt;i.kotrasinsk@samsung.com&gt;
Signed-off-by: Ewelina Kosmider &lt;3w3lfin@gmail.com&gt;
[Various bug fixes and improvements]
Signed-off-by: Krzysztof Opasiak &lt;k.opasiak@samsung.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Modify userspace tools to allow exporting and connecting to vudc.

This commit is a result of cooperation between Samsung R&amp;D Institute
Poland and Open Operating Systems Student Society at University
of Warsaw (O2S3@UW) consisting of:

   Igor Kotrasinski &lt;ikotrasinsk@gmail.com&gt;
   Karol Kosik &lt;karo9@interia.eu&gt;
   Ewelina Kosmider &lt;3w3lfin@gmail.com&gt;
   Dawid Lazarczyk &lt;lazarczyk.dawid@gmail.com&gt;
   Piotr Szulc &lt;ps347277@students.mimuw.edu.pl&gt;

Tutor and project owner:
   Krzysztof Opasiak &lt;k.opasiak@samsung.com&gt;

Signed-off-by: Igor Kotrasinski &lt;i.kotrasinsk@samsung.com&gt;
Signed-off-by: Ewelina Kosmider &lt;3w3lfin@gmail.com&gt;
[Various bug fixes and improvements]
Signed-off-by: Krzysztof Opasiak &lt;k.opasiak@samsung.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
</feed>
