Received: by 2002:a05:7412:d1aa:b0:fc:a2b0:25d7 with SMTP id ba42csp927433rdb; Tue, 30 Jan 2024 02:59:59 -0800 (PST) X-Google-Smtp-Source: AGHT+IGlMo9iBtmFO/8Ez1HEPxvOMtT2tARARj+NYerrcf58yl5TJuefAFV1e/QxLdbML8N5sgTJ X-Received: by 2002:ac8:5cd0:0:b0:42a:8455:6a8b with SMTP id s16-20020ac85cd0000000b0042a84556a8bmr8029534qta.115.1706612399329; Tue, 30 Jan 2024 02:59:59 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1706612399; cv=pass; d=google.com; s=arc-20160816; b=EK+dM7Muh36JC76oC49RV31D3k0v6XtjIP4FU9R4CGDVhmqzu1QnC9k3R24jJ2cSXg BndrY/11L2nktww/4guspTbFaC3DNE8MzIOwU5JlEceAJbgD4EAZzk8Tp2qm7i4I33+d vuBXLvfjjDQ0djnIt3HRn8lKOr19I2VIHMBFbDZtKRcIiINRVqtxDsOY1Qm6xNocpyXn lTQ0DZDqibFtBHvl87iyNS/MFRSjVk0l8YGP/H4FJl4qrMgbNuoO+2r3WkQF6CimY+p/ DIIv/In1C/qIze7VhdcigLvlx+jAm5bjiZ81Xye3LxYz+OTC2jjqRTkByMSUSM7LwVLx 69Ww== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:references:in-reply-to:date :subject:cc:to:from:message-id:dkim-signature; bh=zb2voQHdwFouq3+bHsSfj3f+sdaaJqRfRDqSUKWcVqA=; fh=wzsnDibVg0IdSOrFghAlafOikx+rTH+My02szoy7Pec=; b=vUR5CyB2IMNLXBf5xyg7iKYv56GGQHnTEL1Dr7pechu1j6xF4+/iKi6d/c2doH5pJA UIqJQC4QjY7jE+PMF/AGmpwRheyaahgrtxy4dEZ2kpK6J1ixgLmdTK6MyTkGC8obYVIQ nyB/uO9L/wFZ7EgemZghlrwxFDFTdZYQCFDPA7p1//UBgHzorD+NT91yKE3TsdepCOhD LnfrMRjM0dicUDj4oo+P14wXwzZjo2I7FWQbzBnW/GFgnIWy0BPPo/xJsj5D1D4z8vc7 7o+nB8eBoXv5/6teh/VWaK26NN72If3WxgnbNAT1fVyGdyR8qLQwP9hCTo5+z02tFl8X PlmA== ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@qq.com header.s=s201512 header.b=Sq289lLa; arc=pass (i=1 spf=pass spfdomain=qq.com dkim=pass dkdomain=qq.com dmarc=pass fromdomain=qq.com); spf=pass (google.com: domain of linux-kernel+bounces-44441-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-44441-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=qq.com Return-Path: Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [2604:1380:45d1:ec00::1]) by mx.google.com with ESMTPS id n19-20020a05622a041300b0042976f7ddefsi9823731qtx.653.2024.01.30.02.59.59 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 30 Jan 2024 02:59:59 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-44441-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1; Authentication-Results: mx.google.com; dkim=pass header.i=@qq.com header.s=s201512 header.b=Sq289lLa; arc=pass (i=1 spf=pass spfdomain=qq.com dkim=pass dkdomain=qq.com dmarc=pass fromdomain=qq.com); spf=pass (google.com: domain of linux-kernel+bounces-44441-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-44441-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=qq.com Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ny.mirrors.kernel.org (Postfix) with ESMTPS id 03CCA1C245C0 for ; Tue, 30 Jan 2024 10:59:59 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 82B56664AB; Tue, 30 Jan 2024 10:59:52 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=qq.com header.i=@qq.com header.b="Sq289lLa" Received: from out203-205-251-72.mail.qq.com (out203-205-251-72.mail.qq.com [203.205.251.72]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 82DDD65BC9 for ; Tue, 30 Jan 2024 10:59:48 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=203.205.251.72 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706612391; cv=none; b=YvrOdCTs2l6/tzpQhvzPs6GPG9dC2rtFsXTsFB3CeIA38A4ry7KOV3MXtEGCtmkj8aFtp/vb18OXff8PLl3g1DQm1fCTqWo/xYHnxP3oZxTwtHKRqHgsODzE2xvCKWbEBz9WT/N9SIdUy/+sAIs1HeKOp1QRkNmpN2QCZYj9EXM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706612391; c=relaxed/simple; bh=GB2IybzwCWFKvSFD6TdEeL9jOEQsn9lbMsabAy/BvEk=; h=Message-ID:From:To:Cc:Subject:Date:In-Reply-To:References: MIME-Version; b=XBEPjpbm7nZNag/sAsGYnt21Xg+1lrrMvY5WKjNBFi2x9WvKZzBArvpyjYxa/StlzE5sDWNYSSf93doklkCFyAsMk904gDiNar5orhuf+RonCKxx+p85jIrpPkufwFOSo1sqYaETYWpjV5ElCl71FKf5h4wrYU0HTO7aIt11VVY= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=qq.com; spf=pass smtp.mailfrom=qq.com; dkim=pass (1024-bit key) header.d=qq.com header.i=@qq.com header.b=Sq289lLa; arc=none smtp.client-ip=203.205.251.72 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=qq.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=qq.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qq.com; s=s201512; t=1706612386; bh=zb2voQHdwFouq3+bHsSfj3f+sdaaJqRfRDqSUKWcVqA=; h=From:To:Cc:Subject:Date:In-Reply-To:References; b=Sq289lLa+imgpUMSabocPeFmYCs8ajjrx9rfz9wUnOxJvW3eUxEGWmSdFWV7Ou7KI Y8JDMbtktbsfgJF2M+oEHBTm7Ze0Dn1UayI/XeAn4JFaflefhXHR0S4/d10g4Drl3p JmVcp4yEAq0u3Qf/nJ6f2F8aHYH4v+lsiwENc/mE= Received: from pek-lxu-l1.wrs.com ([111.198.225.215]) by newxmesmtplogicsvrszb9-0.qq.com (NewEsmtp) with SMTP id D6B15CE0; Tue, 30 Jan 2024 18:53:43 +0800 X-QQ-mid: xmsmtpt1706612023tdu1foa1s Message-ID: X-QQ-XMAILINFO: MgH8QHUHzFh7ennCN7xTm/B9Ev1wpdoKig/6PQiyPTfqIYNHA7Q660oAk2Ihuy 3UMmkJS3oHraVGekR/msbScZsAOdar30Nm4bYsekg0DQXUThgwgAzrGJeosSg+Q8I5n/rPjcOprn VeoJoZkF7C46zJR6qNCyOSAcj+ZjgKqw5VyVXLiTT7jgaqnCD3z8IDkqnRstkF/Gacg1mf5e4oww r0SHS23soSOts2K5qno8sZyN6JKbN8/ThXvgrzfPxlnRLWPLGP2zWWnWppTUZNmQ4XQ7KihzJcck 5mxHLKaOhA+cM7/V9E5RVWDdOiyg0KazKHlDf0zYq+1tAIjKHVoHn7ti92nZ/iuVieVdd7cGhEvJ W0+0BGMbGVeGuKO9C5QH06puPzljTzbjaK+SzuBbKFjvnE08i+iFKH7nYW+z/0+HCdfpUjZMJFQb TEgzvq99zfyS94/uKbgqL+VnPaw4dgHeogTjMTZlmoW0EjE/orbZ6Hg9MFimcOZgUpn4VjlrFoD0 qC+zIDhtWQSyQJ1bYFCtSWkypXFXoCO3WXh+u66vkZRpd75QqdM1QfTRPi+pkHluZbHIAHFGBvZu b9WkUCJG9ge+Hlc9oNEbYR3LoVmNziXYlvjd1ynUedZKbeuqRwLa3zvyrMlwlwP/M46EPEmvNv5a pETI1dQX1+VjpPgv475tI8lnQ7HxkfNg8cxo9gFeb86l+y1pno+Yz3OlfcMlTL+efbAIxFwkHv8l xJYXNkv1H3+M22AKuOsXkEX5lfLdOCbTZzFvVF6zT1LkXi3e/Ca6a3FxN0LhDf38DD0RtUen1xKV Bcvy8nCvosgihS6YEuyQbGudK3i5RTp9yOPHNhjqL+EY411esO6E9MG3NQeY/lKfv8UKav/j4+3W DVRwj1cIF+K5VYF3gKvbRRgqKy9p1sVrrohpqgpE+Hu1UAyL0Iofxo5wX8HEir2JORb4cHkY4dRu xrbDqTr5Zw0UQhBihvlSua4ZKq2TEe X-QQ-XMRINFO: Nq+8W0+stu50PRdwbJxPCL0= From: Edward Adam Davis To: syzbot+2373f6be3e6de4f92562@syzkaller.appspotmail.com Cc: linux-kernel@vger.kernel.org, syzkaller-bugs@googlegroups.com Subject: Re: [syzbot] [block?] [trace?] INFO: task hung in blk_trace_remove (2) Date: Tue, 30 Jan 2024 18:53:43 +0800 X-OQ-MSGID: <20240130105342.3944585-2-eadavis@qq.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <0000000000002b1fc7060fca3adf@google.com> References: <0000000000002b1fc7060fca3adf@google.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit please test task hung in blk_trace_remove #syz test https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master diff --git a/kernel/trace/blktrace.c b/kernel/trace/blktrace.c index d5d94510afd3..f3d02bf98a80 100644 --- a/kernel/trace/blktrace.c +++ b/kernel/trace/blktrace.c @@ -330,6 +330,13 @@ static void blk_trace_free(struct request_queue *q, struct blk_trace *bt) kfree(bt); } +static void blk_trace_free_rcu(struct blk_trace *bt) +{ + free_percpu(bt->sequence); + free_percpu(bt->msg_data); + kfree(bt); +} + static void get_probe_ref(void) { mutex_lock(&blk_probe_mutex); @@ -377,12 +384,36 @@ static int blk_trace_stop(struct blk_trace *bt) return 0; } +static void blk_trace_rcu_free(struct rcu_head *rcu) +{ + struct blk_trace *bt; + + bt = container_of(rcu, struct blk_trace, rcu); + if (bt) { + blk_trace_free_rcu(bt); + } +} + static void blk_trace_cleanup(struct request_queue *q, struct blk_trace *bt) { blk_trace_stop(bt); - synchronize_rcu(); - blk_trace_free(q, bt); + if (!bt->dir) + bt->debugfs_dir = q->debugfs_dir; + mutex_unlock(&q->debugfs_mutex); + relay_close(bt->rchan); + /* + * If 'bt->dir' is not set, then both 'dropped' and 'msg' are created + * under 'q->debugfs_dir', thus lookup and remove them. + */ + if (!bt->dir) { + debugfs_lookup_and_remove("dropped", bt->debugfs_dir); + debugfs_lookup_and_remove("msg", bt->debugfs_dir); + } else { + debugfs_remove(bt->dir); + } put_probe_ref(); + call_rcu(&bt->rcu, blk_trace_rcu_free); + mutex_lock(&q->debugfs_mutex); } static int __blk_trace_remove(struct request_queue *q) diff --git a/include/linux/blktrace_api.h b/include/linux/blktrace_api.h index 122c62e561fc..4920c201bd12 100644 --- a/include/linux/blktrace_api.h +++ b/include/linux/blktrace_api.h @@ -26,6 +26,8 @@ struct blk_trace { struct dentry *dir; struct list_head running_list; atomic_t dropped; + struct dentry *debugfs_dir; + struct rcu_head rcu; }; extern int blk_trace_ioctl(struct block_device *, unsigned, char __user *);