public inbox for glibc-cvs@sourceware.org
help / color / mirror / Atom feed
* [glibc/arm/morello/main] TODO(api): cheri: fix dl_iterate_phdr dlpi_addr
@ 2022-10-26 15:20 Szabolcs Nagy
  0 siblings, 0 replies; 4+ messages in thread
From: Szabolcs Nagy @ 2022-10-26 15:20 UTC (permalink / raw)
  To: glibc-cvs

https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=40ede395252b8dfedabd2cf3bcff1ee87d94e2ee

commit 40ede395252b8dfedabd2cf3bcff1ee87d94e2ee
Author: Szabolcs Nagy <szabolcs.nagy@arm.com>
Date:   Wed Sep 7 17:37:38 2022 +0100

    TODO(api): cheri: fix dl_iterate_phdr dlpi_addr
    
    The dlpi_addr field is a capability that has value l_addr, but we can
    only do this for libraries (ET_DYN) where l_addr == l_map_start,
    otherwise we return l_addr which is normally 0 then (ET_EXEC) so the
    caller can detect and special case it.
    
    For now l_addr != 0 and l_addr != l_map_start case is not supported.
    Note: this api may be used by the unwinder to find and read .eh_frame
    data.
    
    TODO: dlpi_addr could be address only, but requires unwinder update
    and agreement about the abi.

Diff:
---
 elf/dl-iteratephdr.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/elf/dl-iteratephdr.c b/elf/dl-iteratephdr.c
index 4d12a8ed50..146850b88a 100644
--- a/elf/dl-iteratephdr.c
+++ b/elf/dl-iteratephdr.c
@@ -61,7 +61,13 @@ __dl_iterate_phdr (int (*callback) (struct dl_phdr_info *info,
 
   for (l = GL(dl_ns)[ns]._ns_loaded; l != NULL; l = l->l_next)
     {
+#ifdef __CHERI_PURE_CAPABILITY__
+      info.dlpi_addr = l->l_real->l_addr == l->l_real->l_map_start
+		       ? l->l_real->l_map_start
+		       : l->l_real->l_addr;
+#else
       info.dlpi_addr = l->l_real->l_addr;
+#endif
       info.dlpi_name = l->l_real->l_name;
       info.dlpi_phdr = l->l_real->l_phdr;
       info.dlpi_phnum = l->l_real->l_phnum;

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

* [glibc/arm/morello/main] TODO(api): cheri: fix dl_iterate_phdr dlpi_addr
@ 2022-11-23 14:49 Szabolcs Nagy
  0 siblings, 0 replies; 4+ messages in thread
From: Szabolcs Nagy @ 2022-11-23 14:49 UTC (permalink / raw)
  To: glibc-cvs

https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=9d6a71514dee3ec6bdc6977339de5d31cbef8c4c

commit 9d6a71514dee3ec6bdc6977339de5d31cbef8c4c
Author: Szabolcs Nagy <szabolcs.nagy@arm.com>
Date:   Wed Sep 7 17:37:38 2022 +0100

    TODO(api): cheri: fix dl_iterate_phdr dlpi_addr
    
    The dlpi_addr field is a capability that has value l_addr, but we can
    only do this for libraries (ET_DYN) where l_addr == l_map_start,
    otherwise we return l_addr which is normally 0 then (ET_EXEC) so the
    caller can detect and special case it.
    
    For now l_addr != 0 and l_addr != l_map_start case is not supported.
    Note: this api may be used by the unwinder to find and read .eh_frame
    data.
    
    TODO: dlpi_addr could be address only, but requires unwinder update
    and agreement about the abi.

Diff:
---
 elf/dl-iteratephdr.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/elf/dl-iteratephdr.c b/elf/dl-iteratephdr.c
index 4d12a8ed50..146850b88a 100644
--- a/elf/dl-iteratephdr.c
+++ b/elf/dl-iteratephdr.c
@@ -61,7 +61,13 @@ __dl_iterate_phdr (int (*callback) (struct dl_phdr_info *info,
 
   for (l = GL(dl_ns)[ns]._ns_loaded; l != NULL; l = l->l_next)
     {
+#ifdef __CHERI_PURE_CAPABILITY__
+      info.dlpi_addr = l->l_real->l_addr == l->l_real->l_map_start
+		       ? l->l_real->l_map_start
+		       : l->l_real->l_addr;
+#else
       info.dlpi_addr = l->l_real->l_addr;
+#endif
       info.dlpi_name = l->l_real->l_name;
       info.dlpi_phdr = l->l_real->l_phdr;
       info.dlpi_phnum = l->l_real->l_phnum;

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

* [glibc/arm/morello/main] TODO(api): cheri: fix dl_iterate_phdr dlpi_addr
@ 2022-10-27 13:59 Szabolcs Nagy
  0 siblings, 0 replies; 4+ messages in thread
From: Szabolcs Nagy @ 2022-10-27 13:59 UTC (permalink / raw)
  To: glibc-cvs

https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=5ea99c81afe66b84460685114e79b95b97c7ba83

commit 5ea99c81afe66b84460685114e79b95b97c7ba83
Author: Szabolcs Nagy <szabolcs.nagy@arm.com>
Date:   Wed Sep 7 17:37:38 2022 +0100

    TODO(api): cheri: fix dl_iterate_phdr dlpi_addr
    
    The dlpi_addr field is a capability that has value l_addr, but we can
    only do this for libraries (ET_DYN) where l_addr == l_map_start,
    otherwise we return l_addr which is normally 0 then (ET_EXEC) so the
    caller can detect and special case it.
    
    For now l_addr != 0 and l_addr != l_map_start case is not supported.
    Note: this api may be used by the unwinder to find and read .eh_frame
    data.
    
    TODO: dlpi_addr could be address only, but requires unwinder update
    and agreement about the abi.

Diff:
---
 elf/dl-iteratephdr.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/elf/dl-iteratephdr.c b/elf/dl-iteratephdr.c
index 4d12a8ed50..146850b88a 100644
--- a/elf/dl-iteratephdr.c
+++ b/elf/dl-iteratephdr.c
@@ -61,7 +61,13 @@ __dl_iterate_phdr (int (*callback) (struct dl_phdr_info *info,
 
   for (l = GL(dl_ns)[ns]._ns_loaded; l != NULL; l = l->l_next)
     {
+#ifdef __CHERI_PURE_CAPABILITY__
+      info.dlpi_addr = l->l_real->l_addr == l->l_real->l_map_start
+		       ? l->l_real->l_map_start
+		       : l->l_real->l_addr;
+#else
       info.dlpi_addr = l->l_real->l_addr;
+#endif
       info.dlpi_name = l->l_real->l_name;
       info.dlpi_phdr = l->l_real->l_phdr;
       info.dlpi_phnum = l->l_real->l_phnum;

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

* [glibc/arm/morello/main] TODO(api): cheri: fix dl_iterate_phdr dlpi_addr
@ 2022-10-12 14:18 Szabolcs Nagy
  0 siblings, 0 replies; 4+ messages in thread
From: Szabolcs Nagy @ 2022-10-12 14:18 UTC (permalink / raw)
  To: glibc-cvs

https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=bbfb104b5087108bc2068d0c2b60391b636b9b48

commit bbfb104b5087108bc2068d0c2b60391b636b9b48
Author: Szabolcs Nagy <szabolcs.nagy@arm.com>
Date:   Wed Sep 7 17:37:38 2022 +0100

    TODO(api): cheri: fix dl_iterate_phdr dlpi_addr
    
    The dlpi_addr field is a capability that has value l_addr, but we can
    only do this for libraries (ET_DYN) where l_addr == l_map_start,
    otherwise we return l_addr which is normally 0 then (ET_EXEC) so the
    caller can detect and special case it.
    
    For now l_addr != 0 and l_addr != l_map_start case is not supported.
    Note: this api may be used by the unwinder to find and read .eh_frame
    data.
    
    TODO: dlpi_addr could be address only, but requires unwinder update
    and agreement about the abi.

Diff:
---
 elf/dl-iteratephdr.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/elf/dl-iteratephdr.c b/elf/dl-iteratephdr.c
index 4d12a8ed50..146850b88a 100644
--- a/elf/dl-iteratephdr.c
+++ b/elf/dl-iteratephdr.c
@@ -61,7 +61,13 @@ __dl_iterate_phdr (int (*callback) (struct dl_phdr_info *info,
 
   for (l = GL(dl_ns)[ns]._ns_loaded; l != NULL; l = l->l_next)
     {
+#ifdef __CHERI_PURE_CAPABILITY__
+      info.dlpi_addr = l->l_real->l_addr == l->l_real->l_map_start
+		       ? l->l_real->l_map_start
+		       : l->l_real->l_addr;
+#else
       info.dlpi_addr = l->l_real->l_addr;
+#endif
       info.dlpi_name = l->l_real->l_name;
       info.dlpi_phdr = l->l_real->l_phdr;
       info.dlpi_phnum = l->l_real->l_phnum;

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

end of thread, other threads:[~2022-11-23 14:49 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-10-26 15:20 [glibc/arm/morello/main] TODO(api): cheri: fix dl_iterate_phdr dlpi_addr Szabolcs Nagy
  -- strict thread matches above, loose matches on Subject: below --
2022-11-23 14:49 Szabolcs Nagy
2022-10-27 13:59 Szabolcs Nagy
2022-10-12 14:18 Szabolcs Nagy

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 read-only IMAP folder(s) and NNTP newsgroup(s).