From: Alexandru Elisei <alexandru.elisei@arm.com> To: Andre Przywara <andre.przywara@arm.com>, Will Deacon <will@kernel.org>, Julien Thierry <julien.thierry.kdev@gmail.com> Cc: kvm@vger.kernel.org, kvmarm@lists.cs.columbia.edu, Marc Zyngier <maz@kernel.org>, Sami Mujawar <sami.mujawar@arm.com> Subject: Re: [PATCH kvmtool v2 01/22] ioport: Remove ioport__setup_arch() Date: Tue, 2 Mar 2021 18:02:49 +0000 [thread overview] Message-ID: <e4dc0a95-f0ee-7f98-1e96-12f30dc447bd@arm.com> (raw) In-Reply-To: <20210225005915.26423-2-andre.przywara@arm.com> Hi Andre, On 2/25/21 12:58 AM, Andre Przywara wrote: > Since x86 had a special need for registering tons of special I/O ports, > we had an ioport__setup_arch() callback, to allow each architecture > to do the same. As it turns out no one uses it beside x86, so we remove > that unnecessary abstraction. > > The generic function was registered via a device_base_init() call, so > we just do the same for the x86 specific function only, and can remove > the unneeded ioport__setup_arch(). Looks good, I did a compile test for arm64 and x86, and I grepped the kvmtool directory for ioport__setup_arch. x86 is the only user left: Reviewed-by: Alexandru Elisei <alexandru.elisei@arm.com> Thanks, Alex > > Signed-off-by: Andre Przywara <andre.przywara@arm.com> > --- > arm/ioport.c | 5 ----- > include/kvm/ioport.h | 1 - > ioport.c | 6 ------ > mips/kvm.c | 5 ----- > powerpc/ioport.c | 6 ------ > x86/ioport.c | 3 ++- > 6 files changed, 2 insertions(+), 24 deletions(-) > > diff --git a/arm/ioport.c b/arm/ioport.c > index 2f0feb9a..24092c9d 100644 > --- a/arm/ioport.c > +++ b/arm/ioport.c > @@ -1,11 +1,6 @@ > #include "kvm/ioport.h" > #include "kvm/irq.h" > > -int ioport__setup_arch(struct kvm *kvm) > -{ > - return 0; > -} > - > void ioport__map_irq(u8 *irq) > { > *irq = irq__alloc_line(); > diff --git a/include/kvm/ioport.h b/include/kvm/ioport.h > index 039633f7..d0213541 100644 > --- a/include/kvm/ioport.h > +++ b/include/kvm/ioport.h > @@ -35,7 +35,6 @@ struct ioport_operations { > enum irq_type)); > }; > > -int ioport__setup_arch(struct kvm *kvm); > void ioport__map_irq(u8 *irq); > > int __must_check ioport__register(struct kvm *kvm, u16 port, struct ioport_operations *ops, > diff --git a/ioport.c b/ioport.c > index 844a832d..a6972179 100644 > --- a/ioport.c > +++ b/ioport.c > @@ -221,12 +221,6 @@ out: > return !kvm->cfg.ioport_debug; > } > > -int ioport__init(struct kvm *kvm) > -{ > - return ioport__setup_arch(kvm); > -} > -dev_base_init(ioport__init); > - > int ioport__exit(struct kvm *kvm) > { > ioport__unregister_all(); > diff --git a/mips/kvm.c b/mips/kvm.c > index 26355930..e110e5d5 100644 > --- a/mips/kvm.c > +++ b/mips/kvm.c > @@ -100,11 +100,6 @@ void kvm__irq_trigger(struct kvm *kvm, int irq) > die_perror("KVM_IRQ_LINE ioctl"); > } > > -int ioport__setup_arch(struct kvm *kvm) > -{ > - return 0; > -} > - > bool kvm__arch_cpu_supports_vm(void) > { > return true; > diff --git a/powerpc/ioport.c b/powerpc/ioport.c > index 0c188b61..a5cff4ee 100644 > --- a/powerpc/ioport.c > +++ b/powerpc/ioport.c > @@ -12,12 +12,6 @@ > > #include <stdlib.h> > > -int ioport__setup_arch(struct kvm *kvm) > -{ > - /* PPC has no legacy ioports to set up */ > - return 0; > -} > - > void ioport__map_irq(u8 *irq) > { > } > diff --git a/x86/ioport.c b/x86/ioport.c > index 7ad7b8f3..a8d2bb1a 100644 > --- a/x86/ioport.c > +++ b/x86/ioport.c > @@ -69,7 +69,7 @@ void ioport__map_irq(u8 *irq) > { > } > > -int ioport__setup_arch(struct kvm *kvm) > +static int ioport__setup_arch(struct kvm *kvm) > { > int r; > > @@ -150,3 +150,4 @@ int ioport__setup_arch(struct kvm *kvm) > > return 0; > } > +dev_base_init(ioport__setup_arch);
WARNING: multiple messages have this Message-ID (diff)
From: Alexandru Elisei <alexandru.elisei@arm.com> To: Andre Przywara <andre.przywara@arm.com>, Will Deacon <will@kernel.org>, Julien Thierry <julien.thierry.kdev@gmail.com> Cc: Marc Zyngier <maz@kernel.org>, Sami Mujawar <sami.mujawar@arm.com>, kvmarm@lists.cs.columbia.edu, kvm@vger.kernel.org Subject: Re: [PATCH kvmtool v2 01/22] ioport: Remove ioport__setup_arch() Date: Tue, 2 Mar 2021 18:02:49 +0000 [thread overview] Message-ID: <e4dc0a95-f0ee-7f98-1e96-12f30dc447bd@arm.com> (raw) In-Reply-To: <20210225005915.26423-2-andre.przywara@arm.com> Hi Andre, On 2/25/21 12:58 AM, Andre Przywara wrote: > Since x86 had a special need for registering tons of special I/O ports, > we had an ioport__setup_arch() callback, to allow each architecture > to do the same. As it turns out no one uses it beside x86, so we remove > that unnecessary abstraction. > > The generic function was registered via a device_base_init() call, so > we just do the same for the x86 specific function only, and can remove > the unneeded ioport__setup_arch(). Looks good, I did a compile test for arm64 and x86, and I grepped the kvmtool directory for ioport__setup_arch. x86 is the only user left: Reviewed-by: Alexandru Elisei <alexandru.elisei@arm.com> Thanks, Alex > > Signed-off-by: Andre Przywara <andre.przywara@arm.com> > --- > arm/ioport.c | 5 ----- > include/kvm/ioport.h | 1 - > ioport.c | 6 ------ > mips/kvm.c | 5 ----- > powerpc/ioport.c | 6 ------ > x86/ioport.c | 3 ++- > 6 files changed, 2 insertions(+), 24 deletions(-) > > diff --git a/arm/ioport.c b/arm/ioport.c > index 2f0feb9a..24092c9d 100644 > --- a/arm/ioport.c > +++ b/arm/ioport.c > @@ -1,11 +1,6 @@ > #include "kvm/ioport.h" > #include "kvm/irq.h" > > -int ioport__setup_arch(struct kvm *kvm) > -{ > - return 0; > -} > - > void ioport__map_irq(u8 *irq) > { > *irq = irq__alloc_line(); > diff --git a/include/kvm/ioport.h b/include/kvm/ioport.h > index 039633f7..d0213541 100644 > --- a/include/kvm/ioport.h > +++ b/include/kvm/ioport.h > @@ -35,7 +35,6 @@ struct ioport_operations { > enum irq_type)); > }; > > -int ioport__setup_arch(struct kvm *kvm); > void ioport__map_irq(u8 *irq); > > int __must_check ioport__register(struct kvm *kvm, u16 port, struct ioport_operations *ops, > diff --git a/ioport.c b/ioport.c > index 844a832d..a6972179 100644 > --- a/ioport.c > +++ b/ioport.c > @@ -221,12 +221,6 @@ out: > return !kvm->cfg.ioport_debug; > } > > -int ioport__init(struct kvm *kvm) > -{ > - return ioport__setup_arch(kvm); > -} > -dev_base_init(ioport__init); > - > int ioport__exit(struct kvm *kvm) > { > ioport__unregister_all(); > diff --git a/mips/kvm.c b/mips/kvm.c > index 26355930..e110e5d5 100644 > --- a/mips/kvm.c > +++ b/mips/kvm.c > @@ -100,11 +100,6 @@ void kvm__irq_trigger(struct kvm *kvm, int irq) > die_perror("KVM_IRQ_LINE ioctl"); > } > > -int ioport__setup_arch(struct kvm *kvm) > -{ > - return 0; > -} > - > bool kvm__arch_cpu_supports_vm(void) > { > return true; > diff --git a/powerpc/ioport.c b/powerpc/ioport.c > index 0c188b61..a5cff4ee 100644 > --- a/powerpc/ioport.c > +++ b/powerpc/ioport.c > @@ -12,12 +12,6 @@ > > #include <stdlib.h> > > -int ioport__setup_arch(struct kvm *kvm) > -{ > - /* PPC has no legacy ioports to set up */ > - return 0; > -} > - > void ioport__map_irq(u8 *irq) > { > } > diff --git a/x86/ioport.c b/x86/ioport.c > index 7ad7b8f3..a8d2bb1a 100644 > --- a/x86/ioport.c > +++ b/x86/ioport.c > @@ -69,7 +69,7 @@ void ioport__map_irq(u8 *irq) > { > } > > -int ioport__setup_arch(struct kvm *kvm) > +static int ioport__setup_arch(struct kvm *kvm) > { > int r; > > @@ -150,3 +150,4 @@ int ioport__setup_arch(struct kvm *kvm) > > return 0; > } > +dev_base_init(ioport__setup_arch); _______________________________________________ kvmarm mailing list kvmarm@lists.cs.columbia.edu https://lists.cs.columbia.edu/mailman/listinfo/kvmarm
next prev parent reply other threads:[~2021-03-03 7:23 UTC|newest] Thread overview: 88+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-02-25 0:58 [PATCH kvmtool v2 00/22] Unify I/O port and MMIO trap handling Andre Przywara 2021-02-25 0:58 ` Andre Przywara 2021-02-25 0:58 ` [PATCH kvmtool v2 01/22] ioport: Remove ioport__setup_arch() Andre Przywara 2021-02-25 0:58 ` Andre Przywara 2021-03-02 18:02 ` Alexandru Elisei [this message] 2021-03-02 18:02 ` Alexandru Elisei 2021-02-25 0:58 ` [PATCH kvmtool v2 02/22] hw/serial: Use device abstraction for FDT generator function Andre Przywara 2021-02-25 0:58 ` Andre Przywara 2021-02-25 0:58 ` [PATCH kvmtool v2 03/22] ioport: Retire .generate_fdt_node functionality Andre Przywara 2021-02-25 0:58 ` Andre Przywara 2021-02-25 0:58 ` [PATCH kvmtool v2 04/22] mmio: Extend handling to include ioport emulation Andre Przywara 2021-02-25 0:58 ` Andre Przywara 2021-03-03 17:58 ` Alexandru Elisei 2021-03-03 17:58 ` Alexandru Elisei 2021-03-15 12:24 ` Andre Przywara 2021-03-15 12:24 ` Andre Przywara 2021-02-25 0:58 ` [PATCH kvmtool v2 05/22] hw/i8042: Clean up data types Andre Przywara 2021-02-25 0:58 ` Andre Przywara 2021-02-25 0:58 ` [PATCH kvmtool v2 06/22] hw/i8042: Refactor trap handler Andre Przywara 2021-02-25 0:58 ` Andre Przywara 2021-03-08 17:16 ` Alexandru Elisei 2021-03-08 17:16 ` Alexandru Elisei 2021-02-25 0:59 ` [PATCH kvmtool v2 07/22] hw/i8042: Switch to new trap handlers Andre Przywara 2021-02-25 0:59 ` Andre Przywara 2021-03-08 17:22 ` Alexandru Elisei 2021-03-08 17:22 ` Alexandru Elisei 2021-02-25 0:59 ` [PATCH kvmtool v2 08/22] x86/ioport: Refactor " Andre Przywara 2021-02-25 0:59 ` Andre Przywara 2021-03-09 11:49 ` Alexandru Elisei 2021-03-09 11:49 ` Alexandru Elisei 2021-03-15 12:26 ` Andre Przywara 2021-03-15 12:26 ` Andre Przywara 2021-02-25 0:59 ` [PATCH kvmtool v2 09/22] x86/ioport: Switch to new " Andre Przywara 2021-02-25 0:59 ` Andre Przywara 2021-03-09 12:09 ` Alexandru Elisei 2021-03-09 12:09 ` Alexandru Elisei 2021-03-15 12:26 ` Andre Przywara 2021-03-15 12:26 ` Andre Przywara 2021-02-25 0:59 ` [PATCH kvmtool v2 10/22] hw/rtc: Refactor " Andre Przywara 2021-02-25 0:59 ` Andre Przywara 2021-02-25 0:59 ` [PATCH kvmtool v2 11/22] hw/rtc: Switch to new trap handler Andre Przywara 2021-02-25 0:59 ` Andre Przywara 2021-02-25 0:59 ` [PATCH kvmtool v2 12/22] hw/vesa: Switch trap handling to use MMIO handler Andre Przywara 2021-02-25 0:59 ` Andre Przywara 2021-02-25 0:59 ` [PATCH kvmtool v2 13/22] hw/serial: Refactor trap handler Andre Przywara 2021-02-25 0:59 ` Andre Przywara 2021-03-12 11:29 ` Alexandru Elisei 2021-03-12 11:29 ` Alexandru Elisei 2021-03-15 12:26 ` Andre Przywara 2021-03-15 12:26 ` Andre Przywara 2021-02-25 0:59 ` [PATCH kvmtool v2 14/22] hw/serial: Switch to new trap handlers Andre Przywara 2021-02-25 0:59 ` Andre Przywara 2021-02-25 0:59 ` [PATCH kvmtool v2 15/22] vfio: Refactor ioport trap handler Andre Przywara 2021-02-25 0:59 ` Andre Przywara 2021-03-09 14:21 ` Alexandru Elisei 2021-03-09 14:21 ` Alexandru Elisei 2021-02-25 0:59 ` [PATCH kvmtool v2 16/22] vfio: Switch to new ioport trap handlers Andre Przywara 2021-02-25 0:59 ` Andre Przywara 2021-02-25 0:59 ` [PATCH kvmtool v2 17/22] virtio: Switch trap handling to use MMIO handler Andre Przywara 2021-02-25 0:59 ` Andre Przywara 2021-03-09 14:43 ` Alexandru Elisei 2021-03-09 14:43 ` Alexandru Elisei 2021-02-25 0:59 ` [PATCH kvmtool v2 18/22] pci: " Andre Przywara 2021-02-25 0:59 ` Andre Przywara 2021-02-25 0:59 ` [PATCH kvmtool v2 19/22] Remove ioport specific routines Andre Przywara 2021-02-25 0:59 ` Andre Przywara 2021-03-09 15:13 ` Alexandru Elisei 2021-03-09 15:13 ` Alexandru Elisei 2021-02-25 0:59 ` [PATCH kvmtool v2 20/22] arm: Reorganise and document memory map Andre Przywara 2021-02-25 0:59 ` Andre Przywara 2021-03-09 15:46 ` Alexandru Elisei 2021-03-09 15:46 ` Alexandru Elisei 2021-03-15 12:27 ` Andre Przywara 2021-03-15 12:27 ` Andre Przywara 2021-02-25 0:59 ` [PATCH kvmtool v2 21/22] hw/serial: ARM/arm64: Use MMIO at higher addresses Andre Przywara 2021-02-25 0:59 ` Andre Przywara 2021-03-09 16:02 ` Alexandru Elisei 2021-03-09 16:02 ` Alexandru Elisei 2021-03-15 12:27 ` Andre Przywara 2021-03-15 12:27 ` Andre Przywara 2021-02-25 0:59 ` [PATCH kvmtool v2 22/22] hw/rtc: " Andre Przywara 2021-02-25 0:59 ` Andre Przywara 2021-03-09 16:06 ` Alexandru Elisei 2021-03-09 16:06 ` Alexandru Elisei 2021-03-02 17:57 ` [PATCH kvmtool v2 00/22] Unify I/O port and MMIO trap handling Alexandru Elisei 2021-03-02 17:57 ` Alexandru Elisei 2021-03-03 18:41 ` Andre Przywara 2021-03-03 18:41 ` Andre Przywara
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=e4dc0a95-f0ee-7f98-1e96-12f30dc447bd@arm.com \ --to=alexandru.elisei@arm.com \ --cc=andre.przywara@arm.com \ --cc=julien.thierry.kdev@gmail.com \ --cc=kvm@vger.kernel.org \ --cc=kvmarm@lists.cs.columbia.edu \ --cc=maz@kernel.org \ --cc=sami.mujawar@arm.com \ --cc=will@kernel.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: linkBe 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.