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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id B2B89C433EF for ; Fri, 25 Feb 2022 15:51:56 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S242319AbiBYPw0 (ORCPT ); Fri, 25 Feb 2022 10:52:26 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59306 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232662AbiBYPwW (ORCPT ); Fri, 25 Feb 2022 10:52:22 -0500 Received: from netrider.rowland.org (netrider.rowland.org [192.131.102.5]) by lindbergh.monkeyblade.net (Postfix) with SMTP id 718371CF084 for ; Fri, 25 Feb 2022 07:51:49 -0800 (PST) Received: (qmail 1082425 invoked by uid 1000); 25 Feb 2022 10:51:48 -0500 Date: Fri, 25 Feb 2022 10:51:48 -0500 From: "stern@rowland.harvard.edu" To: "gregkh@linuxfoundation.org" Cc: "Zhang, Qiang1" , Tejun Heo , syzbot , "linux-kernel@vger.kernel.org" , "syzkaller-bugs@googlegroups.com" , "balbi@kernel.org" Subject: Re: [syzbot] KASAN: use-after-free Read in dev_uevent Message-ID: References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Feb 25, 2022 at 09:53:35AM +0100, gregkh@linuxfoundation.org wrote: > On Thu, Feb 24, 2022 at 09:06:13PM -0500, stern@rowland.harvard.edu wrote: > > On Thu, Feb 24, 2022 at 11:37:39PM +0100, gregkh@linuxfoundation.org wrote: > > > On Thu, Feb 24, 2022 at 04:23:26PM -0500, stern@rowland.harvard.edu wrote: > > > > Can you tell us how this should be fixed? > > > > > > It should be fixed by properly using the driver core to bind/unbind the > > > driver to devices like I mentioned previously :) > > > > This would involve creating a "gadget" bus_type (or should it be a > > device_type under the platform bus?) and registering the gadgets > > on it, right?. > > Yes. Or you can use the aux bus for this, which might be easier. > > > Similarly, the gadget drivers would be registered on > > this bus. I suppose we can control which drivers get bound to which > > gadgets with careful matching code. > > The aux bus might make this easier: > Documentation/driver-api/auxiliary_bus.rst Won't this end up changing the user-visible filenames and directories in sysfs for gadgets and gadget drivers? For instance, currently gadgets get registered under their UDC driver name, like "net2280" or "at91". If we put them on the aux bus then they will have to get registered under a name looking something like "udc.gadget.0", i.e., module name, generic device name, and ID number. We will be forced to use a generic device name because the aux bus does matching based on it, and we want every gadget driver to be able to match every UDC. We don't want some gadget drivers restricted to net2280 gadgets, others restricted to fotg210 gadgets, and so on. Alan Stern