From: Claudio Imbrenda <imbrenda@linux.ibm.com>
To: Cornelia Huck <cohuck@redhat.com>
Cc: Thomas Huth <thuth@redhat.com>,
David Hildenbrand <david@redhat.com>,
Janosch Frank <frankja@linux.ibm.com>,
kvm@vger.kernel.org, linux-s390@vger.kernel.org
Subject: Re: [kvm-unit-tests PATCH] s390x: unify header guards
Date: Tue, 1 Jun 2021 18:25:45 +0200 [thread overview]
Message-ID: <20210601182545.36acdc98@ibm-vm> (raw)
In-Reply-To: <20210601161525.462315-1-cohuck@redhat.com>
On Tue, 1 Jun 2021 18:15:25 +0200
Cornelia Huck <cohuck@redhat.com> wrote:
> Let's unify the header guards to _ASM_S390X_FILE_H_ respectively
> _S390X_FILE_H_. This makes it more obvious what the file is
> about, and avoids possible name space collisions.
>
> Signed-off-by: Cornelia Huck <cohuck@redhat.com>
LGTM, but... what about the other architectures? I think we should
try to standardize more generally
> ---
>
> Only did s390x for now; the other archs seem to be inconsistent in
> places as well, and I can also try to tackle them if it makes sense.
>
> ---
> lib/s390x/asm/bitops.h | 4 ++--
> lib/s390x/asm/cpacf.h | 6 +++---
> lib/s390x/asm/interrupt.h | 4 ++--
> lib/s390x/asm/io.h | 4 ++--
> lib/s390x/asm/mem.h | 4 ++--
> lib/s390x/asm/memory_areas.h | 4 ++--
> lib/s390x/asm/page.h | 4 ++--
> lib/s390x/asm/pgtable.h | 6 +++---
> lib/s390x/asm/sigp.h | 6 +++---
> lib/s390x/asm/spinlock.h | 4 ++--
> lib/s390x/asm/stack.h | 4 ++--
> lib/s390x/asm/time.h | 4 ++--
> lib/s390x/asm/uv.h | 4 ++--
> lib/s390x/css.h | 4 ++--
> lib/s390x/interrupt.h | 6 +++---
> lib/s390x/mmu.h | 6 +++---
> lib/s390x/sclp.h | 6 +++---
> lib/s390x/sie.h | 6 +++---
> lib/s390x/smp.h | 4 ++--
> lib/s390x/uv.h | 6 +++---
> lib/s390x/vm.h | 6 +++---
> s390x/sthyi.h | 4 ++--
> 22 files changed, 53 insertions(+), 53 deletions(-)
>
> diff --git a/lib/s390x/asm/bitops.h b/lib/s390x/asm/bitops.h
> index 792881ec3249..61cd38fd36b7 100644
> --- a/lib/s390x/asm/bitops.h
> +++ b/lib/s390x/asm/bitops.h
> @@ -8,8 +8,8 @@
> * Author(s): Martin Schwidefsky <schwidefsky@de.ibm.com>,
> *
> */
> -#ifndef _ASMS390X_BITOPS_H_
> -#define _ASMS390X_BITOPS_H_
> +#ifndef _ASM_S390X_BITOPS_H_
> +#define _ASM_S390X_BITOPS_H_
>
> #ifndef _BITOPS_H_
> #error only <bitops.h> can be included directly
> diff --git a/lib/s390x/asm/cpacf.h b/lib/s390x/asm/cpacf.h
> index 805fcf1a2d71..8e9b8d754c92 100644
> --- a/lib/s390x/asm/cpacf.h
> +++ b/lib/s390x/asm/cpacf.h
> @@ -8,8 +8,8 @@
> * Harald Freudenberger (freude@de.ibm.com)
> * Martin Schwidefsky <schwidefsky@de.ibm.com>
> */
> -#ifndef _ASM_S390_CPACF_H
> -#define _ASM_S390_CPACF_H
> +#ifndef _ASM_S390X_CPACF_H_
> +#define _ASM_S390X_CPACF_H_
>
> #include <asm/facility.h>
> #include <linux/compiler.h>
> @@ -471,4 +471,4 @@ static inline void cpacf_pckmo(long func, void
> *param) : "cc", "memory");
> }
>
> -#endif /* _ASM_S390_CPACF_H */
> +#endif /* _ASM_S390X_CPACF_H_ */
> diff --git a/lib/s390x/asm/interrupt.h b/lib/s390x/asm/interrupt.h
> index 31e4766d23d5..c095a76f1e7d 100644
> --- a/lib/s390x/asm/interrupt.h
> +++ b/lib/s390x/asm/interrupt.h
> @@ -5,8 +5,8 @@
> * Authors:
> * David Hildenbrand <david@redhat.com>
> */
> -#ifndef _ASMS390X_IRQ_H_
> -#define _ASMS390X_IRQ_H_
> +#ifndef _ASM_S390X_IRQ_H_
> +#define _ASM_S390X_IRQ_H_
> #include <asm/arch_def.h>
>
> #define EXT_IRQ_EMERGENCY_SIG 0x1201
> diff --git a/lib/s390x/asm/io.h b/lib/s390x/asm/io.h
> index 1dc6283b641f..8b602ca70075 100644
> --- a/lib/s390x/asm/io.h
> +++ b/lib/s390x/asm/io.h
> @@ -6,8 +6,8 @@
> * Thomas Huth <thuth@redhat.com>
> * David Hildenbrand <david@redhat.com>
> */
> -#ifndef _ASMS390X_IO_H_
> -#define _ASMS390X_IO_H_
> +#ifndef _ASM_S390X_IO_H_
> +#define _ASM_S390X_IO_H_
>
> #define __iomem
>
> diff --git a/lib/s390x/asm/mem.h b/lib/s390x/asm/mem.h
> index 281390ebd816..082655759d54 100644
> --- a/lib/s390x/asm/mem.h
> +++ b/lib/s390x/asm/mem.h
> @@ -5,8 +5,8 @@
> * Copyright IBM Corp. 2018
> * Author(s): Janosch Frank <frankja@de.ibm.com>
> */
> -#ifndef _ASM_S390_MEM_H
> -#define _ASM_S390_MEM_H
> +#ifndef _ASM_S390X_MEM_H_
> +#define _ASM_S390X_MEM_H_
>
> #define SKEY_ACC 0xf0
> #define SKEY_FP 0x08
> diff --git a/lib/s390x/asm/memory_areas.h
> b/lib/s390x/asm/memory_areas.h index 827bfb356007..c368bbb5dc5e 100644
> --- a/lib/s390x/asm/memory_areas.h
> +++ b/lib/s390x/asm/memory_areas.h
> @@ -1,5 +1,5 @@
> -#ifndef _ASMS390X_MEMORY_AREAS_H_
> -#define _ASMS390X_MEMORY_AREAS_H_
> +#ifndef _ASM_S390X_MEMORY_AREAS_H_
> +#define _ASM_S390X_MEMORY_AREAS_H_
>
> #define AREA_NORMAL_PFN (1 << 19)
> #define AREA_NORMAL_NUMBER 0
> diff --git a/lib/s390x/asm/page.h b/lib/s390x/asm/page.h
> index f130f936c5da..c22c089c13e4 100644
> --- a/lib/s390x/asm/page.h
> +++ b/lib/s390x/asm/page.h
> @@ -6,8 +6,8 @@
> * Thomas Huth <thuth@redhat.com>
> * David Hildenbrand <david@redhat.com>
> */
> -#ifndef _ASMS390X_PAGE_H_
> -#define _ASMS390X_PAGE_H_
> +#ifndef _ASM_S390X_PAGE_H_
> +#define _ASM_S390X_PAGE_H_
>
> #include <asm-generic/page.h>
>
> diff --git a/lib/s390x/asm/pgtable.h b/lib/s390x/asm/pgtable.h
> index 277f34801460..210d6ab398e2 100644
> --- a/lib/s390x/asm/pgtable.h
> +++ b/lib/s390x/asm/pgtable.h
> @@ -7,8 +7,8 @@
> * Authors:
> * David Hildenbrand <david@redhat.com>
> */
> -#ifndef _ASMS390X_PGTABLE_H_
> -#define _ASMS390X_PGTABLE_H_
> +#ifndef _ASM_S390X_PGTABLE_H_
> +#define _ASM_S390X_PGTABLE_H_
>
> #include <asm/page.h>
> #include <alloc_page.h>
> @@ -219,4 +219,4 @@ static inline void ipte(unsigned long vaddr,
> pteval_t *p_pte)
> void configure_dat(int enable);
>
> -#endif /* _ASMS390X_PGTABLE_H_ */
> +#endif /* _ASM_S390X_PGTABLE_H_ */
> diff --git a/lib/s390x/asm/sigp.h b/lib/s390x/asm/sigp.h
> index 00844d26d15a..b3bf7e9cb50c 100644
> --- a/lib/s390x/asm/sigp.h
> +++ b/lib/s390x/asm/sigp.h
> @@ -5,8 +5,8 @@
> * Copied from the Linux kernel file arch/s390/include/asm/sigp.h
> */
>
> -#ifndef ASM_S390X_SIGP_H
> -#define ASM_S390X_SIGP_H
> +#ifndef _ASM_S390X_SIGP_H_
> +#define _ASM_S390X_SIGP_H_
>
> /* SIGP order codes */
> #define SIGP_SENSE 1
> @@ -73,4 +73,4 @@ static inline int sigp_retry(uint16_t addr, uint8_t
> order, unsigned long parm, }
>
> #endif /* __ASSEMBLER__ */
> -#endif /* ASM_S390X_SIGP_H */
> +#endif /* _ASM_S390X_SIGP_H_ */
> diff --git a/lib/s390x/asm/spinlock.h b/lib/s390x/asm/spinlock.h
> index 677d2cd6e287..ca9bf8616749 100644
> --- a/lib/s390x/asm/spinlock.h
> +++ b/lib/s390x/asm/spinlock.h
> @@ -6,8 +6,8 @@
> * Thomas Huth <thuth@redhat.com>
> * David Hildenbrand <david@redhat.com>
> */
> -#ifndef __ASMS390X_SPINLOCK_H
> -#define __ASMS390X_SPINLOCK_H
> +#ifndef _ASM_S390X_SPINLOCK_H_
> +#define _ASM_S390X_SPINLOCK_H_
>
> #include <asm-generic/spinlock.h>
>
> diff --git a/lib/s390x/asm/stack.h b/lib/s390x/asm/stack.h
> index 909da36dce47..b49978051d88 100644
> --- a/lib/s390x/asm/stack.h
> +++ b/lib/s390x/asm/stack.h
> @@ -6,8 +6,8 @@
> * Thomas Huth <thuth@redhat.com>
> * David Hildenbrand <david@redhat.com>
> */
> -#ifndef _ASMS390X_STACK_H_
> -#define _ASMS390X_STACK_H_
> +#ifndef _ASM_S390X_STACK_H_
> +#define _ASM_S390X_STACK_H_
>
> #ifndef _STACK_H_
> #error Do not directly include <asm/stack.h>. Just use <stack.h>.
> diff --git a/lib/s390x/asm/time.h b/lib/s390x/asm/time.h
> index 0d67f7231992..72a43228d73d 100644
> --- a/lib/s390x/asm/time.h
> +++ b/lib/s390x/asm/time.h
> @@ -8,8 +8,8 @@
> * Copied from the s390/intercept test by:
> * Pierre Morel <pmorel@linux.ibm.com>
> */
> -#ifndef ASM_S390X_TIME_H
> -#define ASM_S390X_TIME_H
> +#ifndef _ASM_S390X_TIME_H_
> +#define _ASM_S390X_TIME_H_
>
> #define STCK_SHIFT_US (63 - 51)
> #define STCK_MAX ((1UL << 52) - 1)
> diff --git a/lib/s390x/asm/uv.h b/lib/s390x/asm/uv.h
> index b22cbaa87109..12c4fdcce669 100644
> --- a/lib/s390x/asm/uv.h
> +++ b/lib/s390x/asm/uv.h
> @@ -9,8 +9,8 @@
> * This code is free software; you can redistribute it and/or modify
> it
> * under the terms of the GNU General Public License version 2.
> */
> -#ifndef ASM_S390X_UV_H
> -#define ASM_S390X_UV_H
> +#ifndef _ASM_S390X_UV_H_
> +#define _ASM_S390X_UV_H_
>
> #define UVC_RC_EXECUTED 0x0001
> #define UVC_RC_INV_CMD 0x0002
> diff --git a/lib/s390x/css.h b/lib/s390x/css.h
> index 7e3d2613402e..d644971fb2b7 100644
> --- a/lib/s390x/css.h
> +++ b/lib/s390x/css.h
> @@ -6,8 +6,8 @@
> * Author: Pierre Morel <pmorel@linux.ibm.com>
> */
>
> -#ifndef CSS_H
> -#define CSS_H
> +#ifndef _S390X_CSS_H_
> +#define _S390X_CSS_H_
>
> #define lowcore_ptr ((struct lowcore *)0x0)
>
> diff --git a/lib/s390x/interrupt.h b/lib/s390x/interrupt.h
> index 1973d267c2f1..48d90cec1f23 100644
> --- a/lib/s390x/interrupt.h
> +++ b/lib/s390x/interrupt.h
> @@ -1,8 +1,8 @@
> -#ifndef INTERRUPT_H
> -#define INTERRUPT_H
> +#ifndef _S390X_INTERRUPT_H_
> +#define _S390X_INTERRUPT_H_
> #include <asm/interrupt.h>
>
> int register_io_int_func(void (*f)(void));
> int unregister_io_int_func(void (*f)(void));
>
> -#endif /* INTERRUPT_H */
> +#endif /* _S390X_INTERRUPT_H_ */
> diff --git a/lib/s390x/mmu.h b/lib/s390x/mmu.h
> index 603f289e8e00..328a25513aae 100644
> --- a/lib/s390x/mmu.h
> +++ b/lib/s390x/mmu.h
> @@ -7,12 +7,12 @@
> * Authors:
> * Janosch Frank <frankja@de.ibm.com>
> */
> -#ifndef _ASMS390X_MMU_H_
> -#define _ASMS390X_MMU_H_
> +#ifndef _S390X_MMU_H_
> +#define _S390X_MMU_H_
>
> void protect_page(void *vaddr, unsigned long prot);
> void protect_range(void *start, unsigned long len, unsigned long
> prot); void unprotect_page(void *vaddr, unsigned long prot);
> void unprotect_range(void *start, unsigned long len, unsigned long
> prot);
> -#endif /* _ASMS390X_MMU_H_ */
> +#endif /* _S390X_MMU_H_ */
> diff --git a/lib/s390x/sclp.h b/lib/s390x/sclp.h
> index 7abf1038f5ee..28e526e2c915 100644
> --- a/lib/s390x/sclp.h
> +++ b/lib/s390x/sclp.h
> @@ -10,8 +10,8 @@
> * Author: Christian Borntraeger <borntraeger@de.ibm.com>
> */
>
> -#ifndef SCLP_H
> -#define SCLP_H
> +#ifndef _S390X_SCLP_H_
> +#define _S390X_SCLP_H_
>
> #define SCLP_CMD_CODE_MASK 0xffff00ff
>
> @@ -329,4 +329,4 @@ void sclp_memory_setup(void);
> uint64_t get_ram_size(void);
> uint64_t get_max_ram_size(void);
>
> -#endif /* SCLP_H */
> +#endif /* _S390X_SCLP_H_ */
> diff --git a/lib/s390x/sie.h b/lib/s390x/sie.h
> index 518613baf1fa..db30d6164ab6 100644
> --- a/lib/s390x/sie.h
> +++ b/lib/s390x/sie.h
> @@ -1,6 +1,6 @@
> /* SPDX-License-Identifier: GPL-2.0-or-later */
> -#ifndef SIE_H
> -#define SIE_H
> +#ifndef _S390X_SIE_H_
> +#define _S390X_SIE_H_
>
> #define CPUSTAT_STOPPED 0x80000000
> #define CPUSTAT_WAIT 0x10000000
> @@ -195,4 +195,4 @@ extern void sie_entry(void);
> extern void sie_exit(void);
> extern void sie64a(struct kvm_s390_sie_block *sblk, struct
> vm_save_area *save_area);
> -#endif /* SIE_H */
> +#endif /* _S390X_SIE_H_ */
> diff --git a/lib/s390x/smp.h b/lib/s390x/smp.h
> index 67ff16ca3c52..d99f9fc6cced 100644
> --- a/lib/s390x/smp.h
> +++ b/lib/s390x/smp.h
> @@ -7,8 +7,8 @@
> * Authors:
> * Janosch Frank <frankja@linux.ibm.com>
> */
> -#ifndef SMP_H
> -#define SMP_H
> +#ifndef _S390_SMP_H_
> +#define _S390_SMP_H_
>
> #include <asm/arch_def.h>
>
> diff --git a/lib/s390x/uv.h b/lib/s390x/uv.h
> index 42608a967a03..96257851b1d4 100644
> --- a/lib/s390x/uv.h
> +++ b/lib/s390x/uv.h
> @@ -1,10 +1,10 @@
> /* SPDX-License-Identifier: GPL-2.0-or-later */
> -#ifndef UV_H
> -#define UV_H
> +#ifndef _S390X_UV_H_
> +#define _S390X_UV_H_
>
> bool uv_os_is_guest(void);
> bool uv_os_is_host(void);
> bool uv_query_test_call(unsigned int nr);
> int uv_setup(void);
>
> -#endif /* UV_H */
> +#endif /* _S390X_UV_H_ */
> diff --git a/lib/s390x/vm.h b/lib/s390x/vm.h
> index 16722760cb46..7abba0ccae3d 100644
> --- a/lib/s390x/vm.h
> +++ b/lib/s390x/vm.h
> @@ -5,9 +5,9 @@
> * Copyright (c) 2020 Red Hat Inc
> */
>
> -#ifndef S390X_VM_H
> -#define S390X_VM_H
> +#ifndef _S390X_VM_H_
> +#define _S390X_VM_H_
>
> bool vm_is_tcg(void);
>
> -#endif /* S390X_VM_H */
> +#endif /* _S390X_VM_H_ */
> diff --git a/s390x/sthyi.h b/s390x/sthyi.h
> index bbd74c6197c3..eb92fdd2f2b2 100644
> --- a/s390x/sthyi.h
> +++ b/s390x/sthyi.h
> @@ -7,8 +7,8 @@
> * Authors:
> * Janosch Frank <frankja@linux.vnet.ibm.com>
> */
> -#ifndef _STHYI_H_
> -#define _STHYI_H_
> +#ifndef _S390X_STHYI_H_
> +#define _S390X_STHYI_H_
>
> #include <stdint.h>
>
next prev parent reply other threads:[~2021-06-01 16:25 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-06-01 16:15 [kvm-unit-tests PATCH] s390x: unify header guards Cornelia Huck
2021-06-01 16:25 ` Claudio Imbrenda [this message]
2021-06-01 16:55 ` Cornelia Huck
2021-06-02 3:56 ` Thomas Huth
2021-06-02 6:49 ` Janosch Frank
2021-06-02 6:59 ` Cornelia Huck
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=20210601182545.36acdc98@ibm-vm \
--to=imbrenda@linux.ibm.com \
--cc=cohuck@redhat.com \
--cc=david@redhat.com \
--cc=frankja@linux.ibm.com \
--cc=kvm@vger.kernel.org \
--cc=linux-s390@vger.kernel.org \
--cc=thuth@redhat.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).