From: Mike Frysinger <vapier@gentoo.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC PATCH 04/20] sandbox: Add cpu files
Date: Mon, 26 Sep 2011 00:48:40 -0400 [thread overview]
Message-ID: <201109260048.41976.vapier@gentoo.org> (raw)
In-Reply-To: <CAPnjgZ356x=3WTHM6P9eMPmogHz+YAi6JctPwFJV7QU5oY4M9w@mail.gmail.com>
On Sunday, September 25, 2011 16:18:32 Simon Glass wrote:
> On Sun, Sep 25, 2011 at 12:25 PM, Wolfgang Denk wrote:
> > Simon Glass wrote:
> >> > do_reset() is not supposed to return
> >>
> >> I have adjusted the function meaning (which luckily for me was not
> >> defined) so that it can return -1 on failure. This makes my code
> >> correct :-)
> >>
> >> I think it is reasonable to provide a reset function which might not
> >> be able to do its job. That is the current state of sandbox.
> >
> > No, I don't want to change the current definition of reset().
>
> OK.
>
> > And "not able to do the job" is something different than
> > "unimplemented".
> >
> > Why cannot we do a real reset here? Re-exec'in the running binary or
> > performing a longjmp() to the start might be ideas how to implement
> > this.
>
> While this could be done I believe that it might be possible /
> desirable to exit out of the main loop, rather than longjmp or
> re-exec. I have not implemented it because I have not got to that bit
> yet and don't want to put time into a solution I will throw away.
>
> I'm happy to just put:
>
> while (1) ;
>
> in the reset code if you like?
i would expect "reset" in the sandbox to "exit(1)". how else would you exit ?
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20110926/018b4d6e/attachment.pgp
next prev parent reply other threads:[~2011-09-26 4:48 UTC|newest]
Thread overview: 57+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-09-17 16:48 [U-Boot] [RFC PATCH 01/20] sandbox: Add architecture header files Simon Glass
2011-09-17 16:48 ` [U-Boot] [RFC PATCH 02/20] sandbox: Add architecture image support Simon Glass
2011-09-17 19:29 ` Marek Vasut
2011-09-18 3:57 ` Simon Glass
2011-09-17 16:48 ` [U-Boot] [RFC PATCH 03/20] sandbox: Add compiler defines to support a 64-bit x86_64 platform Simon Glass
2011-09-17 23:56 ` Mike Frysinger
2011-09-17 16:48 ` [U-Boot] [RFC PATCH 04/20] sandbox: Add cpu files Simon Glass
2011-09-17 23:58 ` Mike Frysinger
2011-09-23 15:54 ` Simon Glass
2011-09-25 19:25 ` Wolfgang Denk
2011-09-25 20:18 ` Simon Glass
2011-09-26 4:48 ` Mike Frysinger [this message]
2011-09-26 16:49 ` Simon Glass
2011-09-26 17:10 ` Anton Staaf
2011-09-26 19:11 ` Simon Glass
2011-09-26 18:12 ` Wolfgang Denk
2011-09-26 18:16 ` Mike Frysinger
2011-09-17 16:48 ` [U-Boot] [RFC PATCH 05/20] sandbox: Add architecture lib files Simon Glass
2011-09-18 0:02 ` Mike Frysinger
2011-09-17 16:48 ` [U-Boot] [RFC PATCH 06/20] sandbox: Allow board_init_f() and board_init_r() to return Simon Glass
2011-09-18 0:05 ` Mike Frysinger
2011-09-23 15:55 ` Simon Glass
2011-09-25 19:55 ` Wolfgang Denk
2011-09-25 20:20 ` Simon Glass
2011-09-26 4:47 ` Mike Frysinger
2011-09-26 16:48 ` Simon Glass
2011-09-26 17:49 ` Anton Staaf
2011-09-26 19:22 ` Simon Glass
2011-09-26 19:25 ` Anton Staaf
2011-09-17 16:48 ` [U-Boot] [RFC PATCH 07/20] sandbox: Add sandbox board Simon Glass
2011-09-18 0:17 ` Mike Frysinger
2011-09-17 16:48 ` [U-Boot] [RFC PATCH 08/20] sandbox: Add board info for architecture Simon Glass
2011-09-17 16:48 ` [U-Boot] [RFC PATCH 09/20] sandbox: Add bootm support Simon Glass
2011-09-18 0:16 ` Mike Frysinger
2011-09-23 15:55 ` Simon Glass
2011-09-17 16:48 ` [U-Boot] [RFC PATCH 10/20] sandbox: Disable built-in malloc Simon Glass
2011-09-17 16:48 ` [U-Boot] [RFC PATCH 11/20] sandbox: Disable standalone/API support Simon Glass
2011-09-17 16:48 ` [U-Boot] [RFC PATCH 12/20] sandbox: Force command sections to be 4-byte aligned Simon Glass
2011-09-17 16:48 ` [U-Boot] [RFC PATCH 13/20] sandbox: Add OS dependent layer Simon Glass
2011-09-18 0:20 ` Mike Frysinger
2011-09-23 15:59 ` Simon Glass
2011-09-17 16:48 ` [U-Boot] [RFC PATCH 14/20] sandbox: Add board_init() Simon Glass
2011-09-17 16:48 ` [U-Boot] [RFC PATCH 15/20] sandbox: Add main program Simon Glass
2011-09-17 16:48 ` [U-Boot] [RFC PATCH 16/20] sandbox: Add serial uart Simon Glass
2011-09-17 16:48 ` [U-Boot] [RFC PATCH 17/20] sandbox: Add basic config file Simon Glass
2011-09-18 0:22 ` Mike Frysinger
2011-09-23 16:00 ` Simon Glass
2011-09-26 4:52 ` Mike Frysinger
2011-09-26 18:18 ` Simon Glass
2011-09-17 16:48 ` [U-Boot] [RFC PATCH 18/20] sandbox: Remove unused variable warnings Simon Glass
2011-09-18 0:25 ` Mike Frysinger
2011-09-23 16:01 ` Simon Glass
2011-09-26 4:49 ` Mike Frysinger
2011-09-26 18:07 ` Simon Glass
2011-09-17 16:48 ` [U-Boot] [RFC PATCH 19/20] sandbox: Use uintptr_t for 32/64-bit compatibility Simon Glass
2011-09-17 16:48 ` [U-Boot] [RFC PATCH 20/20] sandbox: Makefile changes to build sandbox architecture Simon Glass
2011-09-17 23:54 ` [U-Boot] [RFC PATCH 01/20] sandbox: Add architecture header files Mike Frysinger
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=201109260048.41976.vapier@gentoo.org \
--to=vapier@gentoo.org \
--cc=u-boot@lists.denx.de \
/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