From: Aleksander Morgado <aleksander@aleksander.es>
To: barebox@lists.infradead.org
Cc: Aleksander Morgado <aleksander@aleksander.es>
Subject: [PATCH] sandbox: new --stdinout option to enable a bidirectional console
Date: Fri, 2 Jun 2017 14:01:05 +0200 [thread overview]
Message-ID: <20170602120105.20663-1-aleksander@aleksander.es> (raw)
In addition to allowing read-only and write-only consoles with --stdin
and --stdout, we now allow bidirectional read/write consoles with FIFO
files. This is e.g. to allow doing RATP over the FIFO based consoles.
Signed-off-by: Aleksander Morgado <aleksander@aleksander.es>
---
arch/sandbox/os/common.c | 53 ++++++++++++++++++++++++++++++++++++++----------
1 file changed, 42 insertions(+), 11 deletions(-)
diff --git a/arch/sandbox/os/common.c b/arch/sandbox/os/common.c
index 8cf087313..665e8194e 100644
--- a/arch/sandbox/os/common.c
+++ b/arch/sandbox/os/common.c
@@ -312,26 +312,28 @@ static int add_dtb(const char *file)
static void print_usage(const char*);
static struct option long_options[] = {
- {"help", 0, 0, 'h'},
- {"malloc", 1, 0, 'm'},
- {"image", 1, 0, 'i'},
- {"env", 1, 0, 'e'},
- {"dtb", 1, 0, 'd'},
- {"stdout", 1, 0, 'O'},
- {"stdin", 1, 0, 'I'},
- {"xres", 1, 0, 'x'},
- {"yres", 1, 0, 'y'},
+ {"help", 0, 0, 'h'},
+ {"malloc", 1, 0, 'm'},
+ {"image", 1, 0, 'i'},
+ {"env", 1, 0, 'e'},
+ {"dtb", 1, 0, 'd'},
+ {"stdout", 1, 0, 'O'},
+ {"stdin", 1, 0, 'I'},
+ {"stdinout", 1, 0, 'B'},
+ {"xres", 1, 0, 'x'},
+ {"yres", 1, 0, 'y'},
{0, 0, 0, 0},
};
-static const char optstring[] = "hm:i:e:d:O:I:x:y:";
+static const char optstring[] = "hm:i:e:d:O:I:B:x:y:";
int main(int argc, char *argv[])
{
void *ram;
- int opt, ret, fd;
+ int opt, ret, fd, fd2;
int malloc_size = CONFIG_MALLOC_SIZE;
int fdno = 0, envno = 0, option_index = 0;
+ char *aux;
while (1) {
option_index = 0;
@@ -421,6 +423,31 @@ int main(int argc, char *argv[])
barebox_register_console(fd, -1);
break;
+ case 'B':
+ aux = strchr(optarg, ',');
+ if (!aux) {
+ printf("-B, --stdinout requires two file paths given\n");
+ exit(1);
+ }
+
+ /* open stdout file */
+ fd = open(aux + 1, O_WRONLY);
+ if (fd < 0) {
+ perror("open stdout");
+ exit(1);
+ }
+
+ /* open stdin file */
+ aux = strndup(optarg, aux - optarg);
+ fd2 = open(aux, O_RDWR);
+ if (fd2 < 0) {
+ perror("open stdin");
+ exit(1);
+ }
+ free(aux);
+
+ barebox_register_console(fd2, fd);
+ break;
default:
break;
}
@@ -463,6 +490,10 @@ static void print_usage(const char *prgname)
" <file> can be a regular file or a FIFO.\n"
" -I, --stdin=<file> Register a file as a console capable of doing stdin.\n"
" <file> can be a regular file or a FIFO.\n"
+" -B, --stdinout=<filein>,<fileout>\n"
+" Register a bidirectional console capable of doing both\n"
+" stdin and stdout. <filein> and <fileout> can be regular\n"
+" files or FIFOs.\n"
" -x, --xres=<res> SDL width.\n"
" -y, --yres=<res> SDL height.\n",
prgname
--
2.13.0
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
next reply other threads:[~2017-06-02 12:01 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-06-02 12:01 Aleksander Morgado [this message]
2017-06-06 6:53 ` Sascha Hauer
2017-06-06 7:00 ` Aleksander Morgado
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=20170602120105.20663-1-aleksander@aleksander.es \
--to=aleksander@aleksander.es \
--cc=barebox@lists.infradead.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