From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-10.3 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,MAILING_LIST_MULTI, SPF_HELO_NONE,USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 9E8A2C48BC2 for ; Sun, 27 Jun 2021 14:18:40 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 7023F61C20 for ; Sun, 27 Jun 2021 14:18:40 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230260AbhF0OVB (ORCPT ); Sun, 27 Jun 2021 10:21:01 -0400 Received: from netrider.rowland.org ([192.131.102.5]:46291 "HELO netrider.rowland.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S229927AbhF0OVB (ORCPT ); Sun, 27 Jun 2021 10:21:01 -0400 Received: (qmail 625832 invoked by uid 1000); 27 Jun 2021 10:18:36 -0400 Date: Sun, 27 Jun 2021 10:18:36 -0400 From: Alan Stern To: Igor Kononenko Cc: Felipe Balbi , Greg Kroah-Hartman , openbmc@lists.ozlabs.org, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/6] usb:gadget:mass-storage: Improve the signature of SCSI handler function Message-ID: <20210627141836.GC624763@rowland.harvard.edu> References: <20210626211820.107310-1-i.kononenko@yadro.com> <20210626211820.107310-2-i.kononenko@yadro.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210626211820.107310-2-i.kononenko@yadro.com> User-Agent: Mutt/1.10.1 (2018-07-13) Precedence: bulk List-ID: X-Mailing-List: linux-usb@vger.kernel.org On Sun, Jun 27, 2021 at 12:18:14AM +0300, Igor Kononenko wrote: > SCSI command handlers currently have an ambiguous return value. This (I dislike very much this way of writing patch descriptions. Unless the reader has already looked at the email subject line and remembers that this patch affects the mass-storage gadget, he will think the sentence above is talking about command handlers in the SCSI core -- a completely different part of the kernel. When writing patch descriptions, please do not assume that the reader already knows what the patch is about.) > return value may indicate the length of the data written to the response > buffer and the command's processing status. Thus, the understanding of > command handling may be implicit. The return value is _not_ ambiguous. If the value is >= 0 then it is a data length, otherwise it is a status. Yes, this is implicit, but it is a very common pattern used throughout the kernel and everyone understands it. > After this patch, the output buffer's size will be set in the > 'data_size_to_handle' field of 'struct fsg_common', and the command > handler's return value indicates only the processing status. What is the reason for making this change? Does it fix any problems or prepare the way for any future patches? It seems like this is completely unnecessary. Alan Stern > Tested: By probing the USBGadget Mass-Storage on the YADRO VEGMAN > BMC(AST2500) sample, each SCSI command was sent through HOST->BMC; the > USBGadget MassStorage debug print showed all sent commands works > properly. > > Signed-off-by: Igor Kononenko