Received: by 2002:ab2:69cc:0:b0:1fd:c486:4f03 with SMTP id n12csp262993lqp; Tue, 11 Jun 2024 04:01:35 -0700 (PDT) X-Forwarded-Encrypted: i=3; AJvYcCU5ln/1FDiptuQadsckQr9hiY0xwZtrYqmN8b2W7gKkrD3w5fZ0hG5SoMXbnazpx688uw6ZoFzKWHnSz+mSxgKPQx2PcIq32deBVA8G6Q== X-Google-Smtp-Source: AGHT+IF2AcQsyLdue92Agw+9X4zbQ+pXnnvNW/Duuo3gqs+iYpROWtrj2Q1H0oFigS56XWFoXbPv X-Received: by 2002:a17:906:c58:b0:a6e:fb3a:b661 with SMTP id a640c23a62f3a-a6efb3ab6edmr457036966b.68.1718103695238; Tue, 11 Jun 2024 04:01:35 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1718103695; cv=pass; d=google.com; s=arc-20160816; b=IeyiH4Z4EEJx7mVWb32AVNhXLOFtTFysz7emZkyWbrnuEo8fEr2xB8DRE+lL1RbDHA QiW4wCHRhFXV+tNCY2sHJ1CzVZZf2vGRPHuTA4tjossbGqcCdpslpsbgnAEUwaXgHB5f XNXJZHGqtZM3ItbdqMF2ncxsVUI6lcT/9x6DAQI0V5uw8pwaec1WoJSIGnHa/DPlFTnv JZyAhSG/HMzosB+8MfjxeXOGlon+/kTouKnPHXytlyNGrYd1Hi+pdKFw3lneEe8PZ2ZC qr/4++nXzkf0G9rGxaNTSxjqkMLzOFgwu8YjHNcwFOv54hAz9E2sW+397c+O2ouuF4PO Yxww== 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:message-id :date:subject:cc:to:from:dkim-signature; bh=DaJUfAifrLY79J8y7mFq2uYCy/DQsT1bjfecVL6JVKQ=; fh=Y/y/ztjXR7aEronvE26r4mZ0eg4Xsop0N9w5JVKmVVc=; b=0WHYUckEGERog07jkU29wLDfddmabc316R3k4hAiVEZV27OFs640mPoC9gia2U8A72 SaeR8H9yTURh2jxG3Pjt0EUKeKaDwD1sEGLxwYxnKlxyfzCw8LGzCdHfi36KvQxi895f jIAulgP6oq79kf/Z4KO9PpyMwUiCLTgmpH467bo99emicPSxSICCIP0jO7mshnUj+NYa b89zkFEIM5LxDzjrwSGhq6RRK1Ofi8vjav5ANco2Vrf2FUo5vf6WBdd3Np077hxKuZhh Xk+P6vDjYGae0Uw9pZswDubXwS+jpPOKW8XqJpUJ/ENPU85jtVoJTpCzKHaZX/E2JfB3 p81Q==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b="QC/xfWhb"; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-kernel+bounces-209689-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) smtp.mailfrom="linux-kernel+bounces-209689-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from am.mirrors.kernel.org (am.mirrors.kernel.org. [2604:1380:4601:e00::3]) by mx.google.com with ESMTPS id a640c23a62f3a-a6f09ac085dsi340064066b.190.2024.06.11.04.01.35 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 11 Jun 2024 04:01:35 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel+bounces-209689-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b="QC/xfWhb"; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-kernel+bounces-209689-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) smtp.mailfrom="linux-kernel+bounces-209689-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org 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 am.mirrors.kernel.org (Postfix) with ESMTPS id EF1AD1F235E3 for ; Tue, 11 Jun 2024 11:01:34 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 50C6246525; Tue, 11 Jun 2024 11:01:05 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="QC/xfWhb" Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 6387B17B408; Tue, 11 Jun 2024 11:01:04 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1718103664; cv=none; b=L8SvItGSd+Nr+Nt8O4Hf4AA+yJXnZ7nVkupp7CDEe3phYU8x4+fzjpird5O/+Ijy/bPAjctbxaDWxuEzwtxNmKeMEBVNdff6rINSTpetbcmpsrHULDv2Jl10zH00KvTzLQM6sCxXsR4ZyfS6nHFWOiaO97OGs/aCIie4jfi3qVM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1718103664; c=relaxed/simple; bh=xq2U4bFBc0pabPUqJTbe1F2n2yJ1049NA855TclycPE=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=kUTtUkTZEITBoaR0zV6dtzPGb9OszyB2JipzIQ7TUuSm6QO6MqJHcDQtT7HVuy4Vuila9QE2eephUCL9JbA+CZQSkU77GMKlasqv44vfx8H/SrhpzHR3z12oiE6Sqs0yQYrOTfzqo9jUFz0sHgs5r8JkrMetr0xzMKP1I4glqbY= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=QC/xfWhb; arc=none smtp.client-ip=10.30.226.201 Received: by smtp.kernel.org (Postfix) with ESMTPSA id DB58AC2BD10; Tue, 11 Jun 2024 11:01:03 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1718103664; bh=xq2U4bFBc0pabPUqJTbe1F2n2yJ1049NA855TclycPE=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=QC/xfWhbffx+WVoSOaXHIrUnxXSkewXhaNi/hgRmiGvJI3/DD7NRSwnxWZcGj2P0s fZwVBMkKJPQvIV9uvSVKFGMo/zQJr4cq+NnXcAszq9OqX5Z9eR9kW61xhStU7C8jSS Xjyv6ECW6oPbALlEKfFzg6ExTj8YLyFAn55kvzxGtAVGVBC4ZCXBxgLaddEPKKi1wv SeBXJMNvZSWeuP78EFBAvdpOSnSE1aCiEJ2S7Km4dZNGj08/4JrN1qgX1m+t4EJE59 h9FN2MySUaisAa4ve/QoWoCW0O0Vfke5qBKLSSJuGq8mb6m2fs4PvRAK/0Q03B/y5l XSVSYe62Ibt1g== From: Andrii Nakryiko To: linux-fsdevel@vger.kernel.org, brauner@kernel.org, viro@zeniv.linux.org.uk, akpm@linux-foundation.org Cc: linux-kernel@vger.kernel.org, bpf@vger.kernel.org, gregkh@linuxfoundation.org, linux-mm@kvack.org, liam.howlett@oracle.com, surenb@google.com, rppt@kernel.org, Andrii Nakryiko Subject: [PATCH v4 1/7] fs/procfs: extract logic for getting VMA name constituents Date: Tue, 11 Jun 2024 04:00:49 -0700 Message-ID: <20240611110058.3444968-2-andrii@kernel.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240611110058.3444968-1-andrii@kernel.org> References: <20240611110058.3444968-1-andrii@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Extract generic logic to fetch relevant pieces of data to describe VMA name. This could be just some string (either special constant or user-provided), or a string with some formatted wrapping text (e.g., "[anon_shmem:]"), or, commonly, file path. seq_file-based logic has different methods to handle all three cases, but they are currently mixed in with extracting underlying sources of data. This patch splits this into data fetching and data formatting, so that data fetching can be reused later on. There should be no functional changes. Signed-off-by: Andrii Nakryiko --- fs/proc/task_mmu.c | 125 +++++++++++++++++++++++++-------------------- 1 file changed, 71 insertions(+), 54 deletions(-) diff --git a/fs/proc/task_mmu.c b/fs/proc/task_mmu.c index 5aceb3db7565..2cc6220222e2 100644 --- a/fs/proc/task_mmu.c +++ b/fs/proc/task_mmu.c @@ -239,6 +239,67 @@ static int do_maps_open(struct inode *inode, struct file *file, sizeof(struct proc_maps_private)); } +static void get_vma_name(struct vm_area_struct *vma, + const struct path **path, + const char **name, + const char **name_fmt) +{ + struct anon_vma_name *anon_name = vma->vm_mm ? anon_vma_name(vma) : NULL; + + *name = NULL; + *path = NULL; + *name_fmt = NULL; + + /* + * Print the dentry name for named mappings, and a + * special [heap] marker for the heap: + */ + if (vma->vm_file) { + /* + * If user named this anon shared memory via + * prctl(PR_SET_VMA ..., use the provided name. + */ + if (anon_name) { + *name_fmt = "[anon_shmem:%s]"; + *name = anon_name->name; + } else { + *path = file_user_path(vma->vm_file); + } + return; + } + + if (vma->vm_ops && vma->vm_ops->name) { + *name = vma->vm_ops->name(vma); + if (*name) + return; + } + + *name = arch_vma_name(vma); + if (*name) + return; + + if (!vma->vm_mm) { + *name = "[vdso]"; + return; + } + + if (vma_is_initial_heap(vma)) { + *name = "[heap]"; + return; + } + + if (vma_is_initial_stack(vma)) { + *name = "[stack]"; + return; + } + + if (anon_name) { + *name_fmt = "[anon:%s]"; + *name = anon_name->name; + return; + } +} + static void show_vma_header_prefix(struct seq_file *m, unsigned long start, unsigned long end, vm_flags_t flags, unsigned long long pgoff, @@ -262,17 +323,15 @@ static void show_vma_header_prefix(struct seq_file *m, static void show_map_vma(struct seq_file *m, struct vm_area_struct *vma) { - struct anon_vma_name *anon_name = NULL; - struct mm_struct *mm = vma->vm_mm; - struct file *file = vma->vm_file; + const struct path *path; + const char *name_fmt, *name; vm_flags_t flags = vma->vm_flags; unsigned long ino = 0; unsigned long long pgoff = 0; unsigned long start, end; dev_t dev = 0; - const char *name = NULL; - if (file) { + if (vma->vm_file) { const struct inode *inode = file_user_inode(vma->vm_file); dev = inode->i_sb->s_dev; @@ -283,57 +342,15 @@ show_map_vma(struct seq_file *m, struct vm_area_struct *vma) start = vma->vm_start; end = vma->vm_end; show_vma_header_prefix(m, start, end, flags, pgoff, dev, ino); - if (mm) - anon_name = anon_vma_name(vma); - /* - * Print the dentry name for named mappings, and a - * special [heap] marker for the heap: - */ - if (file) { + get_vma_name(vma, &path, &name, &name_fmt); + if (path) { seq_pad(m, ' '); - /* - * If user named this anon shared memory via - * prctl(PR_SET_VMA ..., use the provided name. - */ - if (anon_name) - seq_printf(m, "[anon_shmem:%s]", anon_name->name); - else - seq_path(m, file_user_path(file), "\n"); - goto done; - } - - if (vma->vm_ops && vma->vm_ops->name) { - name = vma->vm_ops->name(vma); - if (name) - goto done; - } - - name = arch_vma_name(vma); - if (!name) { - if (!mm) { - name = "[vdso]"; - goto done; - } - - if (vma_is_initial_heap(vma)) { - name = "[heap]"; - goto done; - } - - if (vma_is_initial_stack(vma)) { - name = "[stack]"; - goto done; - } - - if (anon_name) { - seq_pad(m, ' '); - seq_printf(m, "[anon:%s]", anon_name->name); - } - } - -done: - if (name) { + seq_path(m, path, "\n"); + } else if (name_fmt) { + seq_pad(m, ' '); + seq_printf(m, name_fmt, name); + } else if (name) { seq_pad(m, ' '); seq_puts(m, name); } -- 2.43.0