From: Andrew Cooper <andrew.cooper3@citrix.com>
To: Xen-devel <xen-devel@lists.xenproject.org>
Cc: "Andrew Cooper" <andrew.cooper3@citrix.com>,
"Jan Beulich" <JBeulich@suse.com>,
"Roger Pau Monné" <roger.pau@citrix.com>, "Wei Liu" <wl@xen.org>,
"Stefano Stabellini" <sstabellini@kernel.org>,
"Julien Grall" <julien@xen.org>,
"Volodymyr Babchuk" <Volodymyr_Babchuk@epam.com>,
"Bertrand Marquis" <bertrand.marquis@arm.com>,
"Michal Orzel" <michal.orzel@amd.com>,
"Oleksii Kurochko" <oleksii.kurochko@gmail.com>,
"Shawn Anastasio" <sanastasio@raptorengineering.com>,
"consulting @ bugseng . com" <consulting@bugseng.com>,
"Simone Ballarin" <simone.ballarin@bugseng.com>,
"Federico Serafini" <federico.serafini@bugseng.com>,
"Nicola Vetrini" <nicola.vetrini@bugseng.com>
Subject: [PATCH 0/7] xen/bitops: Reduce the mess, starting with ffs()
Date: Wed, 13 Mar 2024 17:27:09 +0000 [thread overview]
Message-ID: <20240313172716.2325427-1-andrew.cooper3@citrix.com> (raw)
bitops.h is a mess. It has grown organtically over many years, and forces
unreasonable repsonsibilities out into the per-arch stubs.
Start cleaning it up with ffs() and friends. Across the board, this adds:
* Functioning bitops without arch-specific asm
* An option for arches to provide more optimal code generation
* Compile-time constant folding
* Testing at both compile time and during init that the basic operations
behave according to spec.
and the only reason this series isn't a net reduction in code alone is the
because of the testing infrastructure in patch 1.
This form is superior in many ways, including getting RISC-V support for free.
Testing:
https://gitlab.com/xen-project/people/andyhhp/xen/-/pipelines/1212269924
https://cirrus-ci.com/build/4939856296542208
Andrew Cooper (7):
xen/bitops: Cleanup ahead of rearrangements
xen/bitops: Implement ffs() in common logic
xen/bitops: Implement ffsl() in common logic
xen/bitops: Delete generic_ffs{,l}()
xen/bitops: Implement ffs64() in common logic
xen: Swap find_first_set_bit() for ffsl() - 1
xen/bitops: Delete find_first_set_bit()
xen/arch/arm/include/asm/bitops.h | 16 +---
xen/arch/ppc/include/asm/bitops.h | 11 ---
xen/arch/x86/guest/xen/xen.c | 4 +-
xen/arch/x86/hvm/dom0_build.c | 2 +-
xen/arch/x86/hvm/hpet.c | 8 +-
xen/arch/x86/include/asm/bitops.h | 53 +++++------
xen/arch/x86/include/asm/pt-contig-markers.h | 2 +-
xen/arch/x86/mm.c | 2 +-
xen/arch/x86/mm/p2m-pod.c | 4 +-
xen/common/Makefile | 1 +
xen/common/bitops.c | 70 ++++++++++++++
xen/common/page_alloc.c | 2 +-
xen/common/softirq.c | 2 +-
xen/drivers/passthrough/amd/iommu_map.c | 2 +-
xen/drivers/passthrough/iommu.c | 4 +-
xen/drivers/passthrough/x86/iommu.c | 4 +-
xen/include/xen/bitops.h | 97 +++++++++-----------
xen/include/xen/compiler.h | 3 +-
18 files changed, 160 insertions(+), 127 deletions(-)
create mode 100644 xen/common/bitops.c
base-commit: 03cf7ca23e0e876075954c558485b267b7d02406
--
2.30.2
next reply other threads:[~2024-03-13 17:27 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-03-13 17:27 Andrew Cooper [this message]
2024-03-13 17:27 ` [PATCH 1/7] xen/bitops: Cleanup ahead of rearrangements Andrew Cooper
2024-03-13 18:39 ` Shawn Anastasio
2024-03-13 23:06 ` Andrew Cooper
2024-03-14 13:59 ` Jan Beulich
2024-03-13 17:27 ` [PATCH 2/7] xen/bitops: Implement ffs() in common logic Andrew Cooper
2024-03-14 14:16 ` Jan Beulich
2024-03-14 16:23 ` Andrew Cooper
2024-03-14 16:35 ` Jan Beulich
2024-03-13 17:27 ` [PATCH 3/7] xen/bitops: Implement ffsl() " Andrew Cooper
2024-03-13 17:48 ` Andrew Cooper
2024-03-14 13:45 ` Andrew Cooper
2024-03-13 18:16 ` Andrew Cooper
2024-03-13 17:27 ` [PATCH 4/7] xen/bitops: Delete generic_ffs{,l}() Andrew Cooper
2024-03-13 17:27 ` [PATCH 5/7] xen/bitops: Implement ffs64() in common logic Andrew Cooper
2024-03-14 15:56 ` Jan Beulich
2024-03-13 17:27 ` [PATCH 6/7] xen: Swap find_first_set_bit() for ffsl() - 1 Andrew Cooper
2024-03-14 14:30 ` Jan Beulich
2024-03-14 16:48 ` Oleksii
2024-03-14 16:55 ` Jan Beulich
2024-03-14 18:47 ` Andrew Cooper
2024-03-14 18:51 ` Andrew Cooper
2024-03-18 9:13 ` Jan Beulich
2024-03-18 12:27 ` Andrew Cooper
2024-03-13 17:27 ` [PATCH 7/7] xen/bitops: Delete find_first_set_bit() Andrew Cooper
2024-03-14 15:59 ` Jan Beulich
2024-03-14 17:14 ` Andrew Cooper
2024-03-15 13:48 ` Andrew Cooper
2024-03-15 14:16 ` Jan Beulich
2024-03-14 14:45 ` [RISCV] [PATCH 0/7] xen/bitops: Reduce the mess, starting with ffs() Andrew Cooper
2024-03-14 15:33 ` Jan Beulich
2024-03-14 15:55 ` Andrew Cooper
2024-03-14 16:32 ` Oleksii
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=20240313172716.2325427-1-andrew.cooper3@citrix.com \
--to=andrew.cooper3@citrix.com \
--cc=JBeulich@suse.com \
--cc=Volodymyr_Babchuk@epam.com \
--cc=bertrand.marquis@arm.com \
--cc=consulting@bugseng.com \
--cc=federico.serafini@bugseng.com \
--cc=julien@xen.org \
--cc=michal.orzel@amd.com \
--cc=nicola.vetrini@bugseng.com \
--cc=oleksii.kurochko@gmail.com \
--cc=roger.pau@citrix.com \
--cc=sanastasio@raptorengineering.com \
--cc=simone.ballarin@bugseng.com \
--cc=sstabellini@kernel.org \
--cc=wl@xen.org \
--cc=xen-devel@lists.xenproject.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.