mirror of
https://github.com/Motorhead1991/qemu.git
synced 2025-08-02 23:33:54 -06:00
semihosting: Split out semihost_sys_write
Split out the non-ARM specific portions of SYS_WRITE to a reusable function. This handles all GuestFD. This removes the last use of common_semi_syscall_len. Note that gdb_do_syscall %x reads target_ulong, not int. Reviewed-by: Luc Michel <lmichel@kalray.eu> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
This commit is contained in:
parent
af0484b502
commit
aa915bd0a6
3 changed files with 61 additions and 51 deletions
|
@ -107,6 +107,13 @@ static void gdb_read(CPUState *cs, gdb_syscall_complete_cb complete,
|
|||
(target_ulong)gf->hostfd, buf, len);
|
||||
}
|
||||
|
||||
static void gdb_write(CPUState *cs, gdb_syscall_complete_cb complete,
|
||||
GuestFD *gf, target_ulong buf, target_ulong len)
|
||||
{
|
||||
gdb_do_syscall(complete, "write,%x,%x,%x",
|
||||
(target_ulong)gf->hostfd, buf, len);
|
||||
}
|
||||
|
||||
/*
|
||||
* Host semihosting syscall implementations.
|
||||
*/
|
||||
|
@ -193,6 +200,22 @@ static void host_read(CPUState *cs, gdb_syscall_complete_cb complete,
|
|||
}
|
||||
}
|
||||
|
||||
static void host_write(CPUState *cs, gdb_syscall_complete_cb complete,
|
||||
GuestFD *gf, target_ulong buf, target_ulong len)
|
||||
{
|
||||
CPUArchState *env G_GNUC_UNUSED = cs->env_ptr;
|
||||
void *ptr = lock_user(VERIFY_READ, buf, len, 1);
|
||||
ssize_t ret;
|
||||
|
||||
if (!ptr) {
|
||||
complete(cs, -1, EFAULT);
|
||||
return;
|
||||
}
|
||||
ret = write(gf->hostfd, ptr, len);
|
||||
complete(cs, ret, ret == -1 ? errno : 0);
|
||||
unlock_user(ptr, buf, 0);
|
||||
}
|
||||
|
||||
/*
|
||||
* Static file semihosting syscall implementations.
|
||||
*/
|
||||
|
@ -286,3 +309,34 @@ void semihost_sys_read(CPUState *cs, gdb_syscall_complete_cb complete,
|
|||
complete(cs, -1, EBADF);
|
||||
}
|
||||
}
|
||||
|
||||
void semihost_sys_write_gf(CPUState *cs, gdb_syscall_complete_cb complete,
|
||||
GuestFD *gf, target_ulong buf, target_ulong len)
|
||||
{
|
||||
switch (gf->type) {
|
||||
case GuestFDGDB:
|
||||
gdb_write(cs, complete, gf, buf, len);
|
||||
break;
|
||||
case GuestFDHost:
|
||||
host_write(cs, complete, gf, buf, len);
|
||||
break;
|
||||
case GuestFDStatic:
|
||||
/* Static files are never open for writing: EBADF. */
|
||||
complete(cs, -1, EBADF);
|
||||
break;
|
||||
default:
|
||||
g_assert_not_reached();
|
||||
}
|
||||
}
|
||||
|
||||
void semihost_sys_write(CPUState *cs, gdb_syscall_complete_cb complete,
|
||||
int fd, target_ulong buf, target_ulong len)
|
||||
{
|
||||
GuestFD *gf = get_guestfd(fd);
|
||||
|
||||
if (gf) {
|
||||
semihost_sys_write_gf(cs, complete, gf, buf, len);
|
||||
} else {
|
||||
complete(cs, -1, EBADF);
|
||||
}
|
||||
}
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue