All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] nx: Rename GRUB_DL_ALIGN to DL_ALIGN
@ 2024-10-16 13:04 Daniel Kiper via Grub-devel
  2024-10-16 16:05 ` Sudeep Holla
  2024-10-16 16:40 ` Ross Philipson via Grub-devel
  0 siblings, 2 replies; 3+ messages in thread
From: Daniel Kiper via Grub-devel @ 2024-10-16 13:04 UTC (permalink / raw)
  To: grub-devel
  Cc: Daniel Kiper, alec.r.brown, jan.setjeeilers, phcoder, pjones,
	mate.kukri, ross.philipson

Rename has been skipped by mistake in the original commit.

Fixes: 94649c026 (nx: Set page permissions for loaded modules)

Signed-off-by: Daniel Kiper <daniel.kiper@oracle.com>
---
 grub-core/kern/dl.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/grub-core/kern/dl.c b/grub-core/kern/dl.c
index 5a66e0886..8ad015b07 100644
--- a/grub-core/kern/dl.c
+++ b/grub-core/kern/dl.c
@@ -646,7 +646,7 @@ grub_dl_set_mem_attrs (grub_dl_t mod, void *ehdr)
   grub_err_t err;
 #if !defined (__i386__) && !defined (__x86_64__) && !defined(__riscv) && \
   !defined (__loongarch__)
-  grub_size_t arch_addralign = GRUB_DL_ALIGN;
+  grub_size_t arch_addralign = DL_ALIGN;
   grub_addr_t tgaddr;
   grub_size_t tgsz;
 #endif
-- 
2.11.0


_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] nx: Rename GRUB_DL_ALIGN to DL_ALIGN
  2024-10-16 13:04 [PATCH] nx: Rename GRUB_DL_ALIGN to DL_ALIGN Daniel Kiper via Grub-devel
@ 2024-10-16 16:05 ` Sudeep Holla
  2024-10-16 16:40 ` Ross Philipson via Grub-devel
  1 sibling, 0 replies; 3+ messages in thread
From: Sudeep Holla @ 2024-10-16 16:05 UTC (permalink / raw)
  To: Daniel Kiper
  Cc: grub-devel, alec.r.brown, jan.setjeeilers, phcoder, pjones,
	mate.kukri, ross.philipson

On Wed, Oct 16, 2024 at 03:04:17PM +0200, Daniel Kiper wrote:
> Rename has been skipped by mistake in the original commit.
> 
> Fixes: 94649c026 (nx: Set page permissions for loaded modules)

Fixes the build on aarch64/arm64 for me.

Tested-by: Sudeep Holla <sudeep.holla@arm.com>

-- 
Regards,
Sudeep

_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] nx: Rename GRUB_DL_ALIGN to DL_ALIGN
  2024-10-16 13:04 [PATCH] nx: Rename GRUB_DL_ALIGN to DL_ALIGN Daniel Kiper via Grub-devel
  2024-10-16 16:05 ` Sudeep Holla
@ 2024-10-16 16:40 ` Ross Philipson via Grub-devel
  1 sibling, 0 replies; 3+ messages in thread
From: Ross Philipson via Grub-devel @ 2024-10-16 16:40 UTC (permalink / raw)
  To: Daniel Kiper, grub-devel
  Cc: ross.philipson, alec.r.brown, jan.setjeeilers, phcoder, pjones,
	mate.kukri

On 10/16/24 6:04 AM, Daniel Kiper wrote:
> Rename has been skipped by mistake in the original commit.
> 
> Fixes: 94649c026 (nx: Set page permissions for loaded modules)
> 
> Signed-off-by: Daniel Kiper <daniel.kiper@oracle.com>

Reviewed-by: Ross Philipson <ross.philipson@oracle.com>

> ---
>   grub-core/kern/dl.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/grub-core/kern/dl.c b/grub-core/kern/dl.c
> index 5a66e0886..8ad015b07 100644
> --- a/grub-core/kern/dl.c
> +++ b/grub-core/kern/dl.c
> @@ -646,7 +646,7 @@ grub_dl_set_mem_attrs (grub_dl_t mod, void *ehdr)
>     grub_err_t err;
>   #if !defined (__i386__) && !defined (__x86_64__) && !defined(__riscv) && \
>     !defined (__loongarch__)
> -  grub_size_t arch_addralign = GRUB_DL_ALIGN;
> +  grub_size_t arch_addralign = DL_ALIGN;
>     grub_addr_t tgaddr;
>     grub_size_t tgsz;
>   #endif


_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2024-10-16 16:50 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-10-16 13:04 [PATCH] nx: Rename GRUB_DL_ALIGN to DL_ALIGN Daniel Kiper via Grub-devel
2024-10-16 16:05 ` Sudeep Holla
2024-10-16 16:40 ` Ross Philipson via Grub-devel

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.