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=-2.5 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,UNPARSEABLE_RELAY,USER_AGENT_SANE_1 autolearn=no 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 51BDFC0651F for ; Thu, 4 Jul 2019 16:33:04 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 3402E20828 for ; Thu, 4 Jul 2019 16:33:04 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727278AbfGDQdC (ORCPT ); Thu, 4 Jul 2019 12:33:02 -0400 Received: from eddie.linux-mips.org ([148.251.95.138]:37140 "EHLO cvs.linux-mips.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726024AbfGDQdC (ORCPT ); Thu, 4 Jul 2019 12:33:02 -0400 Received: (from localhost user: 'ladis' uid#1021 fake: STDIN (ladis@eddie.linux-mips.org)) by eddie.linux-mips.org id S23991063AbfGDQc5SlTW5 (ORCPT ); Thu, 4 Jul 2019 18:32:57 +0200 Date: Thu, 4 Jul 2019 18:32:56 +0200 From: Ladislav Michl To: linux-usb@vger.kernel.org Cc: Felipe Balbi , Greg Kroah-Hartman Subject: Re: [PATCH 0/3] usb: gadget: u_serial: Fix and cleanup Message-ID: <20190704163256.GA7834@lenoch> References: <20190703163355.GA28579@lenoch> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190703163355.GA28579@lenoch> User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-usb-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-usb@vger.kernel.org On Wed, Jul 03, 2019 at 06:33:55PM +0200, Ladislav Michl wrote: > Following patchset makes console work (patch 1) for at AT91SAM9G20 board > connected to xhci_hcd and does some cleanup. > Tested with "console=ttyS0,115200n8 console=ttyGS0,115200n8" on kernel > command line and following inittab: > console::respawn:/sbin/getty -L 115200 ttyS0 vt100 > console::respawn:/sbin/getty -L 115200 ttyGS0 vt100 > > There are issues remaining: > > - first usb disconnect works while each next triggers WARN_ON in gs_close: > ------------[ cut here ]------------ > WARNING: CPU: 0 PID: 501 at drivers/usb/gadget/function/u_serial.c:706 gs_close+0x3c/0x1e4 > Modules linked in: > CPU: 0 PID: 501 Comm: getty Not tainted 5.2.0-rc7 #44 > Hardware name: Atmel AT91SAM9 > [] (unwind_backtrace) from [] (show_stack+0x10/0x18) > [] (show_stack) from [] (dump_stack+0x18/0x24) > [] (dump_stack) from [] (__warn+0xcc/0xe4) > [] (__warn) from [] (warn_slowpath_null+0x38/0x48) > [] (warn_slowpath_null) from [] (gs_close+0x3c/0x1e4) > [] (gs_close) from [] (tty_release+0x1d4/0x460) > [] (tty_release) from [] (__fput+0xe4/0x1b0) > [] (__fput) from [] (task_work_run+0x8c/0xa8) > [] (task_work_run) from [] (do_exit+0x354/0x814) > [] (do_exit) from [] (do_group_exit+0x54/0xb8) > [] (do_group_exit) from [] (get_signal+0x18c/0x658) > [] (get_signal) from [] (do_work_pending+0xe0/0x44c) > [] (do_work_pending) from [] (slow_work_pending+0xc/0x20) > Exception stack(0xc3797fb0 to 0xc3797ff8) > 7fa0: 00000000 beb87d0c 00000001 00000000 > 7fc0: 0009a150 00000000 00099c04 00000003 0009a198 0007e049 00099bd4 0009a1e4 > 7fe0: b6e3f000 beb87cd8 00018210 b6dbcc40 60000010 00000000 > ---[ end trace 70af570fde0de49b ]--- This one is explained in drivers/tty/serial/ip22zilog.c line 751 (see also __tty_hangup) And it also explains why patch 2 in this serie is actually wrong. Will send v2. > - init (both busybox' and systemd) waits for usb host to be plugged in, > otherwise boot is stuck and continues after host is connected. > > Will investigate those two later, however comments and suggestions > to the following patches are appreciated. > > Ladislav Michl (3): > usb: gadget: u_serial: Fix console_req complete event race > usb: gadget: u_serial: Remove console specific alloc/free req > functions > usb: gadget: u_serial: Use bool for req_busy > > drivers/usb/gadget/function/u_serial.c | 88 +++++++++----------------- > 1 file changed, 30 insertions(+), 58 deletions(-) > > -- > 2.20.1 >