Received: by 10.223.185.116 with SMTP id b49csp5511235wrg; Tue, 27 Feb 2018 14:52:45 -0800 (PST) X-Google-Smtp-Source: AH8x224eKUIbg8tac4pyXFHH2fEvC/N5iQT+4jS2fz/r7Qbj1hJnxgElqaYPHHcFEYlmSlpVbKF7 X-Received: by 10.99.168.75 with SMTP id i11mr12332802pgp.420.1519771965408; Tue, 27 Feb 2018 14:52:45 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1519771965; cv=none; d=google.com; s=arc-20160816; b=XUo3zT40Na2DGCG044ZA0TMq30GbpJPL2n9AMw3JDDRDRvPhp8U38OfNF9zOwkDm4g qkWzFamNUI9bubYzikV8oY9WRrgNhPwJ5kgeRiFM2+jeIlYW6snhmxxxJPA+w2O/6jpb xJohUQd7I9Y6qWVAlrAjAXRgcenTIJ5iAH+5I8O6cktg0GIbPCIvYrMns9/AMK3EU3gA 9Ek/H+P7mOp3gFwBSxvbZCpZWk3/101t/7m9hcn0bOUwv1w+YfcBVpYHEaiklI5fBDLT 0CfuL/HKbigP+Lx5JOayfsCPuD0/hMsnxqBmLIsjraVBuw/ttaOCuoq01mqgoytCS0ZA HCcQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject:arc-authentication-results; bh=pnK7uY0mrwXrVnOKtN2hGxocpcY/+CUkrwnu2NuhQuI=; b=tlO5/SS1UPxe3XGcpwrdcw+2UKVTKOC4pc81MVYkGGvtR0kI0zJA21yA3uHJvsIBPX QMc4i7FNQv3vdLpkHphj8Dk+wRzLpbXtygrFMrEHc79esNvycXsslTvoYU62wi6k/Ffi Qemggq1hljK9qz5UgFlHhImCnGCEGgm+74zMvYG4zAF2Ru6szCjeD83fkpww7cJ1FSGu ALXQzMZWmE+Ml24x7A4AljGUhsldY8aY+76QG5CyJVQk1/bHMFiYnd8S1hlY1GIWW0m7 E3cXp13uoHzYAtFG/KzmORa8vHjrCigwOBjBT+HRI3gl+9Cz+lzrNmSADWZL2B2asrwm 5cVw== ARC-Authentication-Results: i=1; mx.google.com; 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=fail (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 u16si157565pfl.299.2018.02.27.14.52.30; Tue, 27 Feb 2018 14:52:45 -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; 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=fail (p=NONE sp=NONE dis=NONE) header.from=samsung.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751960AbeB0Wvs (ORCPT + 99 others); Tue, 27 Feb 2018 17:51:48 -0500 Received: from osg.samsung.com ([64.30.133.232]:64414 "EHLO osg.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751633AbeB0Wvr (ORCPT ); Tue, 27 Feb 2018 17:51:47 -0500 Received: from localhost (localhost [127.0.0.1]) by osg.samsung.com (Postfix) with ESMTP id A3C6F2A9F8; Tue, 27 Feb 2018 14:51:46 -0800 (PST) X-Virus-Scanned: Debian amavisd-new at dev.s-opensource.com Received: from osg.samsung.com ([127.0.0.1]) by localhost (localhost [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id v6pT2QN8bnGN; Tue, 27 Feb 2018 14:51:45 -0800 (PST) Received: from [192.168.1.87] (c-24-9-64-241.hsd1.co.comcast.net [24.9.64.241]) by osg.samsung.com (Postfix) with ESMTPSA id 94D4F2A9EE; Tue, 27 Feb 2018 14:51:44 -0800 (PST) Subject: Re: [PATCH] usbip: tools usbip_attach: Fix cryptic error messages To: Krzysztof Opasiak , 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, Shuah Khan References: <20180227222356.23245-1-shuahkh@osg.samsung.com> <8b7b3f6a-4075-e7e3-39ac-51ddb8f961a0@samsung.com> From: Shuah Khan Message-ID: <86353881-6807-7ca4-960e-2d1f8a428658@osg.samsung.com> Date: Tue, 27 Feb 2018 15:51:43 -0700 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: <8b7b3f6a-4075-e7e3-39ac-51ddb8f961a0@samsung.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 02/27/2018 03:45 PM, Krzysztof Opasiak wrote: > > > 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, Yes there are several very cryptic and useless error message all over the place in usbip tools that could use refinement. This just happens to be the one that I seem to run into very frequently. It frustrated me enough that I decided to fix this one first. :) thanks, -- Shuah