From patchwork Mon Jul 31 13:02:45 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Federico Serafini X-Patchwork-Id: 13334580 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 7C118C001DC for ; Mon, 31 Jul 2023 13:03:07 +0000 (UTC) Received: from list by lists.xenproject.org with outflank-mailman.573027.897318 (Exim 4.92) (envelope-from ) id 1qQSY6-0002AW-CQ; Mon, 31 Jul 2023 13:02:58 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version Received: by outflank-mailman (output) from mailman id 573027.897318; Mon, 31 Jul 2023 13:02:58 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1qQSY6-0002AP-9e; Mon, 31 Jul 2023 13:02:58 +0000 Received: by outflank-mailman (input) for mailman id 573027; Mon, 31 Jul 2023 13:02:57 +0000 Received: from se1-gles-flk1-in.inumbo.com ([94.247.172.50] helo=se1-gles-flk1.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1qQSY4-0002A2-Vr for xen-devel@lists.xenproject.org; Mon, 31 Jul 2023 13:02:56 +0000 Received: from support.bugseng.com (mail.bugseng.com [162.55.131.47]) by se1-gles-flk1.inumbo.com (Halon) with ESMTPS id 8ff88595-2fa2-11ee-8613-37d641c3527e; Mon, 31 Jul 2023 15:02:54 +0200 (CEST) Received: from Dell.homenet.telecomitalia.it (host-79-23-62-117.retail.telecomitalia.it [79.23.62.117]) by support.bugseng.com (Postfix) with ESMTPSA id 7460F4EE073F; Mon, 31 Jul 2023 15:02:53 +0200 (CEST) X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: 8ff88595-2fa2-11ee-8613-37d641c3527e From: Federico Serafini To: xen-devel@lists.xenproject.org Cc: consulting@bugseng.com, Federico Serafini , Stefano Stabellini , Julien Grall , Bertrand Marquis , Volodymyr Babchuk , Andrew Cooper , George Dunlap , Jan Beulich , Wei Liu Subject: [XEN PATCH v3] xen/IRQ: address violations of MISRA C: 2012 Rules 8.2 and 8.3 Date: Mon, 31 Jul 2023 15:02:45 +0200 Message-Id: X-Mailer: git-send-email 2.34.1 MIME-Version: 1.0 Give a name to unnamed parameters thus addressing violations of MISRA C:2012 Rule 8.2 ("Function types shall be in prototype form with named parameters"). Keep consistency between parameter names and types used in function declarations and the ones used in the corresponding function definitions, thus addressing violations of MISRA C:2012 Rule 8.3 ("All declarations of an object or function shall use the same names and type qualifiers"). No functional changes. Signed-off-by: Federico Serafini --- Changes in v3: - uniformed parameter names between arm and x86 for irq_set_affinity(). --- Changes in v2: - use 'pirq' instead of 'irq'; - removed changes involving function pointers. --- xen/arch/arm/include/asm/irq.h | 2 +- xen/arch/arm/irq.c | 6 ++--- xen/include/xen/irq.h | 41 +++++++++++++++++----------------- xen/include/xen/softirq.h | 2 +- 4 files changed, 26 insertions(+), 25 deletions(-) diff --git a/xen/arch/arm/include/asm/irq.h b/xen/arch/arm/include/asm/irq.h index 105b33b37d..c8044b0371 100644 --- a/xen/arch/arm/include/asm/irq.h +++ b/xen/arch/arm/include/asm/irq.h @@ -91,7 +91,7 @@ int platform_get_irq(const struct dt_device_node *device, int index); int platform_get_irq_byname(const struct dt_device_node *np, const char *name); -void irq_set_affinity(struct irq_desc *desc, const cpumask_t *cpu_mask); +void irq_set_affinity(struct irq_desc *desc, const cpumask_t *mask); /* * Use this helper in places that need to know whether the IRQ type is diff --git a/xen/arch/arm/irq.c b/xen/arch/arm/irq.c index 16e56f8945..09648db17a 100644 --- a/xen/arch/arm/irq.c +++ b/xen/arch/arm/irq.c @@ -58,7 +58,7 @@ hw_irq_controller no_irq_type = { static irq_desc_t irq_desc[NR_IRQS]; static DEFINE_PER_CPU(irq_desc_t[NR_LOCAL_IRQS], local_irq_desc); -irq_desc_t *__irq_to_desc(int irq) +struct irq_desc *__irq_to_desc(int irq) { if ( irq < NR_LOCAL_IRQS ) return &this_cpu(local_irq_desc)[irq]; @@ -175,10 +175,10 @@ static inline struct domain *irq_get_domain(struct irq_desc *desc) return irq_get_guest_info(desc)->d; } -void irq_set_affinity(struct irq_desc *desc, const cpumask_t *cpu_mask) +void irq_set_affinity(struct irq_desc *desc, const cpumask_t *mask) { if ( desc != NULL ) - desc->handler->set_affinity(desc, cpu_mask); + desc->handler->set_affinity(desc, mask); } int request_irq(unsigned int irq, unsigned int irqflags, diff --git a/xen/include/xen/irq.h b/xen/include/xen/irq.h index 300625e56d..9747e818f7 100644 --- a/xen/include/xen/irq.h +++ b/xen/include/xen/irq.h @@ -110,22 +110,22 @@ typedef struct irq_desc { #define irq_to_desc(irq) (&irq_desc[irq]) #endif -int init_one_irq_desc(struct irq_desc *); -int arch_init_one_irq_desc(struct irq_desc *); +int init_one_irq_desc(struct irq_desc *desc); +int arch_init_one_irq_desc(struct irq_desc *desc); #define irq_desc_initialized(desc) ((desc)->handler != NULL) extern int setup_irq(unsigned int irq, unsigned int irqflags, - struct irqaction *); + struct irqaction *new); extern void release_irq(unsigned int irq, const void *dev_id); extern int request_irq(unsigned int irq, unsigned int irqflags, void (*handler)(int, void *, struct cpu_user_regs *), - const char * devname, void *dev_id); + const char *devname, void *dev_id); extern hw_irq_controller no_irq_type; void cf_check no_action(int cpl, void *dev_id, struct cpu_user_regs *regs); -unsigned int cf_check irq_startup_none(struct irq_desc *); -void cf_check irq_actor_none(struct irq_desc *); +unsigned int cf_check irq_startup_none(struct irq_desc *desc); +void cf_check irq_actor_none(struct irq_desc *desc); #define irq_shutdown_none irq_actor_none #define irq_disable_none irq_actor_none #define irq_enable_none irq_actor_none @@ -146,7 +146,7 @@ struct pirq { #define pirq_info(d, p) ((struct pirq *)radix_tree_lookup(&(d)->pirq_tree, p)) /* Use this instead of pirq_info() if the structure may need allocating. */ -extern struct pirq *pirq_get_info(struct domain *, int pirq); +extern struct pirq *pirq_get_info(struct domain *d, int pirq); #define pirq_field(d, p, f, def) ({ \ const struct pirq *__pi = pirq_info(d, p); \ @@ -155,30 +155,31 @@ extern struct pirq *pirq_get_info(struct domain *, int pirq); #define pirq_to_evtchn(d, pirq) pirq_field(d, pirq, evtchn, 0) #define pirq_masked(d, pirq) pirq_field(d, pirq, masked, 0) -void pirq_cleanup_check(struct pirq *, struct domain *); +void pirq_cleanup_check(struct pirq *pirq, struct domain *d); #define pirq_cleanup_check(pirq, d) \ ((pirq)->evtchn ? pirq_cleanup_check(pirq, d) : (void)0) -extern void pirq_guest_eoi(struct pirq *); -extern void desc_guest_eoi(struct irq_desc *, struct pirq *); +extern void pirq_guest_eoi(struct pirq *pirq); +extern void desc_guest_eoi(struct irq_desc *desc, struct pirq *pirq); extern int pirq_guest_unmask(struct domain *d); -extern int pirq_guest_bind(struct vcpu *, struct pirq *, int will_share); -extern void pirq_guest_unbind(struct domain *d, struct pirq *); -extern void pirq_set_affinity(struct domain *d, int irq, const cpumask_t *); -extern irq_desc_t *domain_spin_lock_irq_desc( - struct domain *d, int irq, unsigned long *pflags); -extern irq_desc_t *pirq_spin_lock_irq_desc( - const struct pirq *, unsigned long *pflags); +extern int pirq_guest_bind(struct vcpu *v, struct pirq *pirq, int will_share); +extern void pirq_guest_unbind(struct domain *d, struct pirq *pirq); +extern void pirq_set_affinity(struct domain *d, int pirq, + const cpumask_t *mask); +extern struct irq_desc *domain_spin_lock_irq_desc( + struct domain *d, int pirq, unsigned long *pflags); +extern struct irq_desc *pirq_spin_lock_irq_desc( + const struct pirq *pirq, unsigned long *pflags); -unsigned int set_desc_affinity(struct irq_desc *, const cpumask_t *); +unsigned int set_desc_affinity(struct irq_desc *desc, const cpumask_t *mask); #ifndef arch_hwdom_irqs -unsigned int arch_hwdom_irqs(domid_t); +unsigned int arch_hwdom_irqs(domid_t domid); #endif #ifndef arch_evtchn_bind_pirq -void arch_evtchn_bind_pirq(struct domain *, int pirq); +void arch_evtchn_bind_pirq(struct domain *d, int pirq); #endif #endif /* __XEN_IRQ_H__ */ diff --git a/xen/include/xen/softirq.h b/xen/include/xen/softirq.h index 1f6c4783da..33d6f2ecd2 100644 --- a/xen/include/xen/softirq.h +++ b/xen/include/xen/softirq.h @@ -26,7 +26,7 @@ typedef void (*softirq_handler)(void); void do_softirq(void); void open_softirq(int nr, softirq_handler handler); -void cpumask_raise_softirq(const cpumask_t *, unsigned int nr); +void cpumask_raise_softirq(const cpumask_t *mask, unsigned int nr); void cpu_raise_softirq(unsigned int cpu, unsigned int nr); void raise_softirq(unsigned int nr);