Received: by 10.223.185.116 with SMTP id b49csp5507846wrg; Tue, 27 Feb 2018 14:47:57 -0800 (PST) X-Google-Smtp-Source: AH8x2266Jy8DWIWFm2jCoOzKm/qnrl6LxQlCxUyHVV34oS4iBIOCbEwpVbrjmAiKgLTtrIZ0xLSd X-Received: by 10.98.224.65 with SMTP id f62mr15612792pfh.191.1519771677487; Tue, 27 Feb 2018 14:47:57 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1519771677; cv=none; d=google.com; s=arc-20160816; b=Fm77WoWW5GQNwo53B9l/dIHEOhiYb99RilJ6zef5Q6+l8uglhQ7XsYn1XMa8HZVoTw EmSPuZpj2VUi/P4G0Vx2ANM7Z3B5Gs7AiDL1168CnNo1XBnRA2rjmTVKACCr2o1A6+IV 6D1eXuxo4TM5I1lsTGIVD++IHOa4Bh6hQx5BZI4Wra8pXv0fzmwHVSo61e9jp/05ew7e fBTyevi0Hs8wA8tZWXKz0aoeJHuvxXmlCxTPwPDr5omDJBk7P0hITloE8gqeMZL8fxdQ Neb1FAtRkN1xlRPKh7B5eD5GtY7QijqNotpFrX430LYVrhjG8ETpBCsJRCxH31rw+1+o Z/Wg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:references:cms-type :content-transfer-encoding:content-language:in-reply-to:mime-version :user-agent:date:message-id:from:cc:to:subject:dkim-signature :dkim-filter:arc-authentication-results; bh=C8Da8lE7k8IVZcWWamc7eoC7UWmr/MkviteQZYyLKkg=; b=ooxL+bgqbfa2l4EP+Eg/N24RvZ3Mo5V1HY2Bb42rAkAKTT65gkmbk1O4zYfIminaRw aFAK9QEzhtbNvT66anOcVthe44q4BJZakyGpAbPsxYIQQ+BiXvJY8GlHCw+vPgtyO74m vo8DwEGyo3lE48sdrCjAk9doX9NEoaOuCtRrOBk9pUgx/2i/sITzCnKrLc7cC5cbnX8t 6VgbhRuQgF8ulAJDSmfXt3KbisTz0xeOkvk0Jv2S74fp/A2tyefRu7/mC/GFkUtU28CQ 5qaFkLXGw5i0JMGBjyqIIvbxb5DNecZl1VupePsgBzK0Grjq5yrKvH+92+8/lEnAAfH0 u1mw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@samsung.com header.s=mail20170921 header.b=RH/vCd8x; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=samsung.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id x4si119914pgo.278.2018.02.27.14.47.42; Tue, 27 Feb 2018 14:47:57 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@samsung.com header.s=mail20170921 header.b=RH/vCd8x; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=samsung.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752074AbeB0WqG (ORCPT + 99 others); Tue, 27 Feb 2018 17:46:06 -0500 Received: from mailout1.samsung.com ([203.254.224.24]:61749 "EHLO mailout1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751754AbeB0WqD (ORCPT ); Tue, 27 Feb 2018 17:46:03 -0500 Received: from epcas1p4.samsung.com (unknown [182.195.41.48]) by mailout1.samsung.com (KnoxPortal) with ESMTP id 20180227224602epoutp0174302c78af4f68e8d9745db7507ea6ff~XUC8CGEEk2553625536epoutp01b; Tue, 27 Feb 2018 22:46:02 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 mailout1.samsung.com 20180227224602epoutp0174302c78af4f68e8d9745db7507ea6ff~XUC8CGEEk2553625536epoutp01b DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=samsung.com; s=mail20170921; t=1519771562; bh=C8Da8lE7k8IVZcWWamc7eoC7UWmr/MkviteQZYyLKkg=; h=Subject:To:Cc:From:Date:In-reply-to:References:From; b=RH/vCd8xxrylKo2g9DEamP92728KtfosxtqOsfPSZ3kSoIVi0dhYUGof92gu1C74F Y8uQIV4nL0anNVScdZZAUp81J/Dqx/+Y4l7qQ3QQ5E3mmYYz82B6cKQJpmfnrpks5f 1eYSYtaWmLNdbj8shTDIxfviGKcykZAn1tOfkVOk= Received: from epsmges1p4.samsung.com (unknown [182.195.42.56]) by epcas1p1.samsung.com (KnoxPortal) with ESMTP id 20180227224601epcas1p1891776107515f21072c8b25578993463~XUC74vD0G3063230632epcas1p1i; Tue, 27 Feb 2018 22:46:01 +0000 (GMT) Received: from epcas1p1.samsung.com ( [182.195.41.45]) by epsmges1p4.samsung.com (Symantec Messaging Gateway) with SMTP id DC.9A.04180.9AFD59A5; Wed, 28 Feb 2018 07:46:01 +0900 (KST) Received: from epsmgms2p1new.samsung.com (unknown [182.195.42.142]) by epcas1p2.samsung.com (KnoxPortal) with ESMTP id 20180227224601epcas1p2d95ba77c214fd7d0a8c12ba29c595a86~XUC7saUa_0645006450epcas1p2o; Tue, 27 Feb 2018 22:46:01 +0000 (GMT) X-AuditID: b6c32a38-adbff70000001054-f6-5a95dfa92a34 Received: from epmmp2 ( [203.254.227.17]) by epsmgms2p1new.samsung.com (Symantec Messaging Gateway) with SMTP id E3.B6.03826.9AFD59A5; Wed, 28 Feb 2018 07:46:01 +0900 (KST) Received: from [106.116.151.61] by mmp2.samsung.com (Oracle Communications Messaging Server 7.0.5.31.0 64bit (built May 5 2014)) with ESMTPA id <0P4T001NQZ8L1LB0@mmp2.samsung.com>; Wed, 28 Feb 2018 07:46:01 +0900 (KST) Subject: Re: [PATCH] usbip: tools usbip_attach: Fix cryptic error messages To: Shuah Khan , valentina.manea.m@gmail.com, shuah@kernel.org, gregkh@linuxfoundation.org Cc: yuyang.du@intel.com, nobuo.iwata@fujixerox.co.jp, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org From: Krzysztof Opasiak Message-id: <8b7b3f6a-4075-e7e3-39ac-51ddb8f961a0@samsung.com> Date: Tue, 27 Feb 2018 23:45:57 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 MIME-version: 1.0 In-reply-to: <20180227222356.23245-1-shuahkh@osg.samsung.com> Content-type: text/plain; charset="utf-8"; format="flowed" Content-language: en-US Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFlrIKsWRmVeSWpSXmKPExsWy7bCmru7K+1OjDI6/MLdoXryezeLyrjls FouWtTJb3Oq4ymIx5eU6doupXz6wWLy7NJfd4vmq5YwOHB4f1x9l9tg56y67x+I9L5k8Nq3q ZPPYP3cNu8eWfqDY501yAexRXDYpqTmZZalF+nYJXBm//lxiK9jHWTHnwHn2Bsaz7F2MnBwS AiYSqzpOsnQxcnEICexglJjY9p0VwvnOKHGwdQYLTNW9nU9YQWwhgQ2MEk+aYiHs+4wS7/tt QGxhAS+JWS9+s4HYIgKlEge/7gHrZRbIl5i3Yw3QNg4ONgF9iXm7REHCvAJ2Ei+7joGNZBFQ lVj36i9Yq6hAhMTCqU8ZIWoEJX5Mvgc2hlPAVuLKwz+sECOtJJ79a4WyxSWaW29CrZKX2Lzm LTPI/RIC99kkOo8vgPrSRWL7r+VsELawxKvjW8DukRCQlrh01Baifh2jxOQL+1khnN2MEvv6 HjFDNFhL/Fk1kQ1iA5/Eu689rBDNvBIdbUIQJR4S045+gIaVo8SLGT1skECcwChx90E/6wRG uVlIHpqF5IlZSJ6YheSJBYwsqxjFUguKc9NTiw0LTPSKE3OLS/PS9ZLzczcxghOOlsUOxj3n fA4xCnAwKvHwZmRPjRJiTSwrrsw9xCjBwawkwlt9GyjEm5JYWZValB9fVJqTWnyIUZqDRUmc NyDAJUpIID2xJDU7NbUgtQgmy8TBKdXAmHvpguzxTL0jawtfpn38vuXG8zwGT4bKgphj7Y+V 9q/4Od/N4JNC65bZbmlVby7misau//U87t0zjfqn81Uv/yt/Yrs36vefOdyM35WyJFYkK79c 8p7tzx6miN+LhWwnL1SR330/OVOEX9Mz1Xqnxq7ksyueVG3OnrvqWsf82uTyFzpiXe+CFJVY ijMSDbWYi4oTAXpGpk00AwAA X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFvrNLMWRmVeSWpSXmKPExsVy+t9jQd2V96dGGcycqWnRvHg9m8XlXXPY LBYta2W2uNVxlcViyst17BZTv3xgsXh3aS67xfNVyxkdODw+rj/K7LFz1l12j8V7XjJ5bFrV yeaxf+4ado8t/UCxz5vkAtijuGxSUnMyy1KL9O0SuDJ+/bnEVrCPs2LOgfPsDYxn2bsYOTkk BEwk7u18wtrFyMUhJLCOUeLQ7QdMEM5DRonNZ96wgFQJC3hJzHrxmw3EFhEoleg43MQKYjML 5EssOHASqmESo0TXvr1AYzk42AT0JebtEgWp4RWwk3jZdQysnkVAVWLdq79gc0QFIiQ6V85n gagRlPgx+R6YzSlgK3Hl4R+o+WYSX14ehrLFJZpbb7JA2PISm9e8ZZ7AKDALSfssJC2zkLTM QtKygJFlFaNkakFxbnpusVGBYV5quV5xYm5xaV66XnJ+7iZGYJRsO6zVt4Px/pL4Q4wCHIxK PLwZ2VOjhFgTy4orcw8xSnAwK4nwVt8GCvGmJFZWpRblxxeV5qQWH2KU5mBREue9nXcsUkgg PbEkNTs1tSC1CCbLxMEp1cCY/KjXtty+4NjrLQ5F+5YstCuJynh14dKPLzvtHdaEXLnHkDX1 cVqAtBNvQPZZrsN2oVbGx5K0kjrmWBxWq+byfpd+6PdnPwkxY+20AH2Gneb3dVtC7O5ar1+s JWRk+F2YY9PR291/3ixvzgm3me9km3bBr0Q+PM/ilMjEz8JTdpjP0l18JUeJpTgj0VCLuag4 EQCDRLswjgIAAA== X-CMS-MailID: 20180227224601epcas1p2d95ba77c214fd7d0a8c12ba29c595a86 X-Msg-Generator: CA CMS-TYPE: 101P X-CMS-RootMailID: 20180227222421epcas2p23e030c5de1620f5976cf4f91a2685682 X-RootMTR: 20180227222421epcas2p23e030c5de1620f5976cf4f91a2685682 References: <20180227222356.23245-1-shuahkh@osg.samsung.com> Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 02/27/2018 11:23 PM, Shuah Khan wrote: > Attach device error message is cryptic and useless. Fix it to be > informative. > > Signed-off-by: Shuah Khan > --- > tools/usb/usbip/src/usbip_attach.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/tools/usb/usbip/src/usbip_attach.c b/tools/usb/usbip/src/usbip_attach.c > index 7f07b2d50f59..f60738735398 100644 > --- a/tools/usb/usbip/src/usbip_attach.c > +++ b/tools/usb/usbip/src/usbip_attach.c > @@ -195,7 +195,8 @@ static int attach_device(char *host, char *busid) > > rhport = query_import_device(sockfd, busid); > if (rhport < 0) { > - err("query"); > + err("Attach request for Device %s. Is this device exported?", > + busid); > return -1; > } The code itself is ok and you may put my: Reviewed-by: Krzysztof Opasiak but just because of my curiosity, there is a number of places in usbip tools where the same level of descriptiveness is used for error message. Why did you choose to fix this one not any other or all others?;) Best regards, -- Krzysztof Opasiak Samsung R&D Institute Poland Samsung Electronics