Received: by 2002:a05:6a10:af89:0:0:0:0 with SMTP id iu9csp3668971pxb; Mon, 24 Jan 2022 14:56:49 -0800 (PST) X-Google-Smtp-Source: ABdhPJxW7TwkT5z4QFNzHDUFEQYvO5QEYiM66fhPQ3GCl17t4yFS7ZPkXWPwWXJzw8FDAvhvDGWB X-Received: by 2002:a05:6a00:2181:b0:4c1:3b84:b43f with SMTP id h1-20020a056a00218100b004c13b84b43fmr15992547pfi.50.1643065008907; Mon, 24 Jan 2022 14:56:48 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1643065008; cv=none; d=google.com; s=arc-20160816; b=xdHZfD6qsCAZ3Zx+Jl8TVPGNa/0lliXlUVVZEd+fhD4ri3S9uL/be/UBQw22iAFKtz JsB0aUFERP8uQzCqB7E0Cf1dUPWpeiaxtyjwHtpjTWj/mbzvIxfaeUI92UaPW6sQGVL3 eRIGJmdw5zdxwHfnkk6UFPInQUJq2X1qgcV7r46oeHxJ67eDFe+U5+TUuJRzbBB6amtI Y0/TwZmVNAX9inESsqYKoDacpAMhAXFbZQikVt1y0IoZNZ5mTrpoB85Gqk0Y4JDpYeY/ X0A/z2siyj3ERzEfsS0hl61ugVdK+dbRwAjSALVzbgRs+IBcXde01Pb1V3pjMrOp/Xo1 9ckw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :user-agent:references:in-reply-to:message-id:date:subject:cc:to :from:dkim-signature; bh=tnyc7t6b5olkme9de4jkIRrP5+R8nli6jcpxwf7CTIA=; b=ctIB9Ac4xzsLr5Me5DDS1YnL5ku83i4Wf5ukBuJcbJQFiIIJpGWFVhsB52bDAiapGx VLvHkrBV+r1iAtKcIr0mXX4WhuBa/btuqO/kC5EYmERrnoVHKGO65zR9D9/RzGduShj7 f5vIbxp4OnHZoXDkJ+YElj3qTni1AytpaO2j5BkKGAOAg3zHHqPZHeDn6yNBcLVwAfmJ usHP3leRHLY76/1eJEoPPbjGbdBvsOdKq9tBW/NdeCK/l2EwPhgmigwFjKpsQRTlGmol /Sq4Mqups4jt9h3WzZPKUGz3RlPaofsdNCVPgdFljeaLEQO9NNMkMX7XS/HylFFJ8hyx muOg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b=Olo1C9cV; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id k10si17034281pfc.213.2022.01.24.14.56.37; Mon, 24 Jan 2022 14:56:48 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b=Olo1C9cV; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1378844AbiAXWxi (ORCPT + 99 others); Mon, 24 Jan 2022 17:53:38 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58102 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1575134AbiAXVvE (ORCPT ); Mon, 24 Jan 2022 16:51:04 -0500 Received: from ams.source.kernel.org (ams.source.kernel.org [IPv6:2604:1380:4601:e00::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 813EBC09398A; Mon, 24 Jan 2022 12:33:51 -0800 (PST) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 49DD2B8122C; Mon, 24 Jan 2022 20:33:50 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 7DE72C340E5; Mon, 24 Jan 2022 20:33:48 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1643056429; bh=PIlHMzSJcbtlID3nr+KRYnPjFbUreuFKXZbjpwgQed0=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=Olo1C9cVOKExLkTnuojmxZjQWgNLmriQezdwzopFY/lSaWww70gzL8UsgBfu1yOHP q3tIuJ4v95Vq/pssr8aOSlpHkKgLDsfI3DZqbWHdpzL6qFgqTtuwP/AW3WmqhpKrEm EPwww2PDbYXqs4GH2bPZpY7i2jUQ5m1u1Ms/rX58= From: Greg Kroah-Hartman To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman , stable@vger.kernel.org, John Keeping , Pavankumar Kondeti , Sasha Levin Subject: [PATCH 5.15 482/846] usb: gadget: f_fs: Use stream_open() for endpoint files Date: Mon, 24 Jan 2022 19:39:59 +0100 Message-Id: <20220124184117.638999670@linuxfoundation.org> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20220124184100.867127425@linuxfoundation.org> References: <20220124184100.867127425@linuxfoundation.org> User-Agent: quilt/0.66 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Pavankumar Kondeti [ Upstream commit c76ef96fc00eb398c8fc836b0eb2f82bcc619dc7 ] Function fs endpoint file operations are synchronized via an interruptible mutex wait. However we see threads that do ep file operations concurrently are getting blocked for the mutex lock in __fdget_pos(). This is an uninterruptible wait and we see hung task warnings and kernel panic if hung_task_panic systcl is enabled if host does not send/receive the data for long time. The reason for threads getting blocked in __fdget_pos() is due to the file position protection introduced by the commit 9c225f2655e3 ("vfs: atomic f_pos accesses as per POSIX"). Since function fs endpoint files does not have the notion of the file position, switch to the stream mode. This will bypass the file position mutex and threads will be blocked in interruptible state for the function fs mutex. It should not affects user space as we are only changing the task state changes the task state from UNINTERRUPTIBLE to INTERRUPTIBLE while waiting for the USB transfers to be finished. However there is a slight change to the O_NONBLOCK behavior. Earlier threads that are using O_NONBLOCK are also getting blocked inside fdget_pos(). Now they reach to function fs and error code is returned. The non blocking behavior is actually honoured now. Reviewed-by: John Keeping Signed-off-by: Pavankumar Kondeti Link: https://lore.kernel.org/r/1636712682-1226-1-git-send-email-quic_pkondeti@quicinc.com Signed-off-by: Greg Kroah-Hartman Signed-off-by: Sasha Levin --- drivers/usb/gadget/function/f_fs.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/usb/gadget/function/f_fs.c b/drivers/usb/gadget/function/f_fs.c index aac343f7d7d3d..782d67c2c6e0d 100644 --- a/drivers/usb/gadget/function/f_fs.c +++ b/drivers/usb/gadget/function/f_fs.c @@ -614,7 +614,7 @@ static int ffs_ep0_open(struct inode *inode, struct file *file) file->private_data = ffs; ffs_data_opened(ffs); - return 0; + return stream_open(inode, file); } static int ffs_ep0_release(struct inode *inode, struct file *file) @@ -1154,7 +1154,7 @@ ffs_epfile_open(struct inode *inode, struct file *file) file->private_data = epfile; ffs_data_opened(epfile->ffs); - return 0; + return stream_open(inode, file); } static int ffs_aio_cancel(struct kiocb *kiocb) -- 2.34.1