Received: by 2002:ac0:a5a7:0:0:0:0:0 with SMTP id m36-v6csp3185201imm; Mon, 16 Jul 2018 23:57:22 -0700 (PDT) X-Google-Smtp-Source: AAOMgpe/EYQdmDN9E1p2cOuqcB1jBZ/djlMmipLqVQ9jCM+kB34qsotijCWgOAW5rUpCjamZRJJx X-Received: by 2002:a62:2983:: with SMTP id p125-v6mr454058pfp.128.1531810642225; Mon, 16 Jul 2018 23:57:22 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1531810642; cv=none; d=google.com; s=arc-20160816; b=Gou9MXISkbt18QKDbwvVCxmej2I4lmnqD8y2ogr24AmwyCdIqRcgZFFH6xOhOzSSEq FJo3RAO8uljXpCVYJz4O7ce6SUap83CttHiE8SLK3f5Q6/lUm2N7ZXGmOrQ6r4E+FFzt spO+h/TCjG/rjTvSLnvaOAibSep8NyWQ1xizVV1w5pKbVud/fcfFrkvc2NERfSZqlRoB rqQgwCu4z85rzHKumeRvSPSDueGA45B6a45KG0NuDHNg1Auf6ebdNx7QGVw7YdhJKxNY PxxrTDbjuVpRUPgxLFt4Iz+KCKueOMvdlnRMU2Oy1lET2uH3qUMazBE/zj/xkmYIcFiO eeHw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:arc-authentication-results; bh=AVnuJ6N8ayL47oZLHuTw6K60vK5ZugfUXO7gh4QvkXA=; b=EMdSsp17uoLH2me8UUdIdBFxsCRqCV25baZpkA1YNR8rE9hr67oKC71VJilxMeY/DE EobYBJySbm6M8bJMHiWN4wZckppJVKuHVgV5QyHTP9bechOWtmpwzuMZWqn9BbkE2SVd FNbQ/ipDup5hXW7jijc/Nl9l2s+QsWTOiaQABt4rIqB51AIjtIrSz4DLPIF9UURUVMgS GQJoP2bsOootdOLhpePQ0hcrgiyrFsRc9GgtRK6cwNPeEfRb6hJexaCOomfQgz6FacC3 0XqM5qv9j8xRZ2iB6daTRmaGzJTqsFKUgLvTN0BNGcJOvmPVQ9pPv55KizXSRqqzXNKU m4tQ== 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 Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id f8-v6si277238plb.381.2018.07.16.23.57.07; Mon, 16 Jul 2018 23:57:22 -0700 (PDT) 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 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728205AbeGQH06 (ORCPT + 99 others); Tue, 17 Jul 2018 03:26:58 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:44376 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727481AbeGQH06 (ORCPT ); Tue, 17 Jul 2018 03:26:58 -0400 Received: from localhost (LFbn-1-12247-202.w90-92.abo.wanadoo.fr [90.92.61.202]) by mail.linuxfoundation.org (Postfix) with ESMTPSA id 61B0ED0E; Tue, 17 Jul 2018 06:55:52 +0000 (UTC) Date: Tue, 17 Jul 2018 08:55:48 +0200 From: Greg Kroah-Hartman To: Todd Poynor Cc: devel@driverdev.osuosl.org, Zhongze Hu , John Joseph , lkml , Simon Que , Rob Springer , Guenter Roeck , Todd Poynor , Dmitry Torokhov Subject: Re: [PATCH 05/18] staging: gasket: sysfs mapping creation fixups Message-ID: <20180717065548.GA10961@kroah.com> References: <20180714055816.223754-1-toddpoynor@gmail.com> <20180714055816.223754-6-toddpoynor@gmail.com> <20180714063224.GD20451@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Jul 16, 2018 at 02:14:27PM -0700, Todd Poynor wrote: > On Fri, Jul 13, 2018 at 11:32 PM, Greg Kroah-Hartman > wrote: > > On Fri, Jul 13, 2018 at 10:58:03PM -0700, Todd Poynor wrote: > >> From: Todd Poynor > >> > >> Return EBUSY for attempt to create a mapping already in use. > > > > Why? > > The existing code returns EINVAL which often means something bogus was > requested, whereas EBUSY is sometimes used for requesting something > already in use. If that's not an error typically returned in this > situation I'm happy to drop it. My "why" was rhetorical, please spell this out in the changelog text :) thanks, greg k-h