From: "Hans-Bernhard Bröker" <HBBroeker@t-online.de>
To: newlib@sourceware.org
Subject: Re: Help porting newlib to a new CPU architecture (sorta)
Date: Wed, 7 Jul 2021 20:43:36 +0200 [thread overview]
Message-ID: <661253a7-00b8-02ac-4956-051f7be70231@t-online.de> (raw)
In-Reply-To: <2bb2d181-36e4-1cc6-bdff-9eb0aea895ec@gmail.com>
Am 06.07.2021 um 22:46 schrieb Orlando Arias:
> Consider the AVR architecture, where program and data spaces have
> distinct address spaces. We have a pointer to a string literal that
> resides in program memory.
You're already mixing stuff up again. The memory C string literals are
in is, by definition, _not_ program memory. It's read-only data memory.
That distinction is crucial.
Small-ish embedded CPUs do not usually implement the strict Harvard
architecture principle, precisely because that does not support constant
data. A strict Harvard would have all data, including the
const-qualified parts, in RAM, and initialize it all by running a very
boring piece of program that just writes it all using immediate
operands. const data would thus consume normal RAM, without any
write-protection by the hardware at all.
Micro controller designers have pulled different kinds of tricks to get
around the need to have constants directly in ROM, ranging from the
simple loop-hole instruction that does read from program memory anyway
(like the 8051's MOVC), to various kinds of mirroring schemes that just
map ROM into data space, essentially breaking the Harvard architecture
rather fundamentally.
But that's ultimately a problem for the implementer of the C compiler
and run-time library to address, if they decide to try doing that on
such small architectures.
> The problem with this code is that we are treating a as a pointer in
> data memory. Declaring a to be PROGMEM does not help. We actually need
> to rewrite the code to force the compiler to use the proper instruction:
That's what you get for throwing Standard C compatibility out the window
by declaring that string constant using a compiler extension like PROGMEM.
Generally the compiler would be required by the Standard to implement
"generic pointers" that can reach _all_ kinds of data defined without
use of non-standard means. If it doesn't do that, it is by definition
not a C compiler. Which can be fine, e.g. if the architecture just
cannot have a correct C implementation otherwise, or only a horribly
inefficient one.
But porting a generic standard C library like newlib or glibc onto a
platform that needs non-standard compiler extensions just to emulate
strcmp() may quickly turn into a lost cause.
> Now, I believe that doing something like (char*)fn_ptr
> in C is either undefined behavior
It quite explicitly is.
next prev parent reply other threads:[~2021-07-07 18:45 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-07-06 0:49 ElijaxApps
2021-07-06 4:35 ` Mike Frysinger
2021-07-06 13:05 ` Paul Koning
2021-07-07 20:32 ` ElijaxApps
2021-07-07 20:56 ` Orlando Arias
2021-07-06 14:02 ` Brian Inglis
2021-07-06 14:35 ` Orlando Arias
2021-07-06 18:08 ` Brian Inglis
2021-07-06 19:04 ` Orlando Arias
2021-07-06 20:01 ` Hans-Bernhard Bröker
2021-07-06 20:46 ` Orlando Arias
2021-07-07 5:45 ` Brian Inglis
2021-07-07 13:58 ` Orlando Arias
2021-07-07 15:18 ` Dave Nadler
2021-07-07 18:43 ` Hans-Bernhard Bröker [this message]
2021-07-07 20:23 ` Orlando Arias
2021-07-06 21:08 ElijaxApps
2021-07-06 22:00 ` Joel Sherrill
2021-07-06 23:50 ` Paul Koning
2021-07-07 0:29 ` ElijaxApps
2021-07-07 15:09 ` Grant Edwards
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=661253a7-00b8-02ac-4956-051f7be70231@t-online.de \
--to=hbbroeker@t-online.de \
--cc=newlib@sourceware.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: 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 read-only IMAP folder(s) and NNTP newsgroup(s).