From: Conor Dooley <mail@conchuod.ie>
To: Anup Patel <anup@brainfault.org>,
Atish Patra <atishp@atishpatra.org>,
Paul Walmsley <paul.walmsley@sifive.com>,
Palmer Dabbelt <palmer@dabbelt.com>,
Albert Ou <aou@eecs.berkeley.edu>
Cc: Conor Dooley <conor.dooley@microchip.com>,
Guo Ren <guoren@kernel.org>,
Vincent Chen <vincent.chen@sifive.com>,
Xianting Tian <xianting.tian@linux.alibaba.com>,
Heiko Stuebner <heiko@sntech.de>,
Kefeng Wang <wangkefeng.wang@huawei.com>,
Tong Tiangen <tongtiangen@huawei.com>,
kvm@vger.kernel.org, kvm-riscv@lists.infradead.org,
linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org
Subject: [PATCH 0/4] misc warning cleanup in arch/risc-v
Date: Sun, 14 Aug 2022 15:12:34 +0100 [thread overview]
Message-ID: <20220814141237.493457-1-mail@conchuod.ie> (raw)
From: Conor Dooley <conor.dooley@microchip.com>
Hey all,
Couple fixes here for most of what's left of the {sparse,} warnings in
arch/riscv that are still in need of patches. Ben has sent patches
for the VDSO issue already (although they seem to need rework).
VDSO aside, With this patchset applied, we are left with:
- cpuinfo_ops missing prototype: this likely needs to go into an
asm-generic header & I'll send a separate patch for that.
- Complaints about an error in mm/init.c:
"error inarch/riscv/mm/init.c:819:2: error: "setup_vm() is <trunc>
I think this can be ignored.
- 600+ -Woverride-init warnings for syscall table setup where
overriding seems to be the whole point of the macro.
- Warnings about imported kvm core code.
- Flexible array member warnings that look like common KVM code
patterns
- An unexpected unlock in kvm_riscv_check_vcpu_requests that was added
intentionally:
https://lore.kernel.org/all/20220710151105.687193-1-apatel@ventanamicro.com/
Is it worth looking into whether that's a false positive or not?
Thanks,
Conor.
Conor Dooley (4):
riscv: kvm: vcpu_timer: fix unused variable warnings
riscv: kvm: move extern sbi_ext declarations to a header
riscv: signal: fix missing prototype warning
riscv: traps: add missing prototype
arch/riscv/include/asm/kvm_vcpu_sbi.h | 12 ++++++++++++
arch/riscv/include/asm/signal.h | 12 ++++++++++++
arch/riscv/include/asm/thread_info.h | 2 ++
arch/riscv/kernel/signal.c | 1 +
arch/riscv/kernel/traps.c | 3 ++-
arch/riscv/kvm/vcpu_sbi.c | 12 +-----------
arch/riscv/kvm/vcpu_timer.c | 4 ----
7 files changed, 30 insertions(+), 16 deletions(-)
create mode 100644 arch/riscv/include/asm/signal.h
--
2.37.1
next reply other threads:[~2022-08-14 14:12 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-08-14 14:12 Conor Dooley [this message]
2022-08-14 14:12 ` [PATCH 1/4] riscv: kvm: vcpu_timer: fix unused variable warnings Conor Dooley
2022-08-19 16:11 ` Anup Patel
2022-08-19 20:09 ` Atish Patra
2022-08-14 14:12 ` [PATCH 2/4] riscv: kvm: move extern sbi_ext declarations to a header Conor Dooley
2022-08-19 16:12 ` Anup Patel
2022-08-14 14:12 ` [PATCH 3/4] riscv: signal: fix missing prototype warning Conor Dooley
2022-08-14 14:12 ` [PATCH 4/4] riscv: traps: add missing prototype Conor Dooley
2022-08-18 23:01 ` [PATCH 0/4] misc warning cleanup in arch/risc-v Palmer Dabbelt
2022-08-19 11:34 ` Conor.Dooley
2022-08-19 16:14 ` Anup Patel
2022-08-25 23:42 ` Palmer Dabbelt
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=20220814141237.493457-1-mail@conchuod.ie \
--to=mail@conchuod.ie \
--cc=anup@brainfault.org \
--cc=aou@eecs.berkeley.edu \
--cc=atishp@atishpatra.org \
--cc=conor.dooley@microchip.com \
--cc=guoren@kernel.org \
--cc=heiko@sntech.de \
--cc=kvm-riscv@lists.infradead.org \
--cc=kvm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-riscv@lists.infradead.org \
--cc=palmer@dabbelt.com \
--cc=paul.walmsley@sifive.com \
--cc=tongtiangen@huawei.com \
--cc=vincent.chen@sifive.com \
--cc=wangkefeng.wang@huawei.com \
--cc=xianting.tian@linux.alibaba.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
all inboxes | Powered by JetHome®