From: Jarmo Tiitto <jarmo.tiitto@gmail.com>
To: Sami Tolvanen <samitolvanen@google.com>,
Bill Wendling <wcw@google.com>, Kees Cook <keescook@chromium.org>,
Nathan Chancellor <nathan@kernel.org>,
Nick Desaulniers <ndesaulniers@google.com>,
clang-built-linux@googlegroups.com, linux-kernel@vger.kernel.org
Cc: Jarmo Tiitto <jarmo.tiitto@gmail.com>, morbo@google.com
Subject: [RFC PATCH 5/5] pgo: Cleanup code in pgo/fs.c
Date: Sat, 12 Jun 2021 06:24:26 +0300 [thread overview]
Message-ID: <20210612032425.11425-6-jarmo.tiitto@gmail.com> (raw)
In-Reply-To: <20210612032425.11425-1-jarmo.tiitto@gmail.com>
Cleanups to comments and punctuation.
Cleanup return path in pgo_module_init.
Signed-off-by: Jarmo Tiitto <jarmo.tiitto@gmail.com>
---
kernel/pgo/fs.c | 47 +++++++++++++++++++++++------------------------
1 file changed, 23 insertions(+), 24 deletions(-)
diff --git a/kernel/pgo/fs.c b/kernel/pgo/fs.c
index 98b982245b58..855d5e3050fa 100644
--- a/kernel/pgo/fs.c
+++ b/kernel/pgo/fs.c
@@ -294,7 +294,7 @@ static int prf_open(struct inode *inode, struct file *file)
int err = -EINVAL;
if (WARN_ON(!inode->i_private)) {
- /* bug: inode was not initialized by us */
+ /* Bug: inode was not initialized by us. */
return err;
}
@@ -302,7 +302,7 @@ static int prf_open(struct inode *inode, struct file *file)
if (!data)
return -ENOMEM;
- /* Get prf_object of this inode */
+ /* Get prf_object of this inode. */
data->core = inode->i_private;
/* Get initial buffer size. */
@@ -425,17 +425,17 @@ static void pgo_module_init(struct module *mod)
char fname[MODULE_NAME_LEN + 9]; /* +strlen(".profraw") */
unsigned long flags;
- /* add new prf_object entry for the module */
+ /* Add new prf_object entry for the module. */
po = kzalloc(sizeof(*po), GFP_KERNEL);
if (!po)
- goto out;
+ return; /* -ENOMEM */
po->mod_name = mod->name;
fname[0] = 0;
snprintf(fname, sizeof(fname), "%s.profraw", po->mod_name);
- /* setup prf_object sections */
+ /* Setup prf_object sections. */
po->data = mod->prf_data;
po->data_num = prf_get_count(mod->prf_data,
(char *)mod->prf_data + mod->prf_data_size, sizeof(po->data[0]));
@@ -452,20 +452,19 @@ static void pgo_module_init(struct module *mod)
po->vnds_num = prf_get_count(mod->prf_vnds,
(char *)mod->prf_vnds + mod->prf_vnds_size, sizeof(po->vnds[0]));
- /* create debugfs entry */
+ /* Create debugfs entry. */
po->file = debugfs_create_file(fname, 0600, directory, po, &prf_fops);
if (!po->file) {
pr_err("Failed to setup module pgo: %s", fname);
kfree(po);
- goto out;
- }
- /* finally enable profiling for the module */
- flags = prf_list_lock();
- list_add_tail_rcu(&po->link, &prf_list);
- prf_list_unlock(flags);
+ } else {
+ /* Finally enable profiling for the module. */
+ flags = prf_list_lock();
+ list_add_tail_rcu(&po->link, &prf_list);
+ prf_list_unlock(flags);
+ }
-out:
return;
}
@@ -477,33 +476,33 @@ static int pgo_module_notifier(struct notifier_block *nb, unsigned long event,
unsigned long flags;
if (event == MODULE_STATE_LIVE) {
- /* does the module have profiling info? */
+ /* Does the module have profiling info? */
if (mod->prf_data
&& mod->prf_cnts
&& mod->prf_names
&& mod->prf_vnds) {
- /* setup module profiling */
+ /* Setup module profiling. */
pgo_module_init(mod);
}
}
if (event == MODULE_STATE_GOING) {
- /* find the prf_object from the list */
+ /* Find the prf_object from the list. */
rcu_read_lock();
list_for_each_entry_rcu(po, &prf_list, link) {
if (strcmp(po->mod_name, mod->name) == 0)
goto out_unlock;
}
- /* no such module */
+ /* No such module. */
po = NULL;
out_unlock:
rcu_read_unlock();
if (po) {
- /* remove from profiled modules */
+ /* Remove from profiled modules. */
flags = prf_list_lock();
list_del_rcu(&po->link);
prf_list_unlock(flags);
@@ -511,7 +510,7 @@ static int pgo_module_notifier(struct notifier_block *nb, unsigned long event,
debugfs_remove(po->file);
po->file = NULL;
- /* cleanup memory */
+ /* Cleanup memory. */
kfree_rcu(po, rcu);
}
}
@@ -528,7 +527,7 @@ static int __init pgo_init(void)
{
unsigned long flags;
- /* Init profiler vmlinux core entry */
+ /* Init profiler vmlinux core entry. */
memset(&prf_vmlinux, 0, sizeof(prf_vmlinux));
prf_vmlinux.data = __llvm_prf_data_start;
prf_vmlinux.data_num = prf_get_count(__llvm_prf_data_start,
@@ -546,7 +545,7 @@ static int __init pgo_init(void)
prf_vmlinux.vnds_num = prf_get_count(__llvm_prf_vnds_start,
__llvm_prf_vnds_end, sizeof(__llvm_prf_vnds_start[0]));
- /* enable profiling */
+ /* Enable profiling. */
flags = prf_list_lock();
prf_vmlinux.mod_name = "vmlinux";
list_add_tail_rcu(&prf_vmlinux.link, &prf_list);
@@ -565,10 +564,10 @@ static int __init pgo_init(void)
&prf_reset_fops))
goto err_remove;
- /* register module notifer */
+ /* Register module notifer. */
register_module_notifier(&pgo_module_nb);
- /* show notice why the system slower: */
+ /* Show notice why the system slower: */
pr_notice("Clang PGO instrumentation is active.");
return 0;
@@ -581,7 +580,7 @@ static int __init pgo_init(void)
/* Remove debugfs entries. */
static void __exit pgo_exit(void)
{
- /* unsubscribe the notifier and do cleanup. */
+ /* Unsubscribe the notifier and do cleanup. */
unregister_module_notifier(&pgo_module_nb);
debugfs_remove_recursive(directory);
--
2.32.0
next prev parent reply other threads:[~2021-06-12 3:35 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-06-12 3:24 [RFC PATCH 0/5] pgo: Add PGO support for module profile data Jarmo Tiitto
2021-06-12 3:24 ` [RFC PATCH 1/5] pgo: Expose module sections for clang PGO instumentation Jarmo Tiitto
2021-06-12 3:24 ` [RFC PATCH 2/5] pgo: Make serializing functions to take prf_object Jarmo Tiitto
2021-06-12 3:24 ` [RFC PATCH 3/5] pgo: Wire up the new more generic code for modules Jarmo Tiitto
2021-06-14 15:55 ` Kees Cook
2021-06-14 19:08 ` jarmo.tiitto
2021-06-12 3:24 ` [RFC PATCH 4/5] pgo: Add module notifier machinery Jarmo Tiitto
2021-06-14 16:00 ` Kees Cook
2021-06-14 18:31 ` jarmo.tiitto
2021-06-12 3:24 ` Jarmo Tiitto [this message]
2021-06-14 15:50 ` [RFC PATCH 5/5] pgo: Cleanup code in pgo/fs.c Kees Cook
2021-06-14 16:05 ` [RFC PATCH 0/5] pgo: Add PGO support for module profile data Kees Cook
2021-06-14 21:57 ` Kees Cook
2021-06-14 22:27 ` jarmo.tiitto
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210612032425.11425-6-jarmo.tiitto@gmail.com \
--to=jarmo.tiitto@gmail.com \
--cc=clang-built-linux@googlegroups.com \
--cc=keescook@chromium.org \
--cc=linux-kernel@vger.kernel.org \
--cc=morbo@google.com \
--cc=nathan@kernel.org \
--cc=ndesaulniers@google.com \
--cc=samitolvanen@google.com \
--cc=wcw@google.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).