From: Laurent Vivier <laurent@vivier.eu>
To: Filip Bozuta <Filip.Bozuta@syrmia.com>, qemu-devel@nongnu.org
Subject: Re: [PATCH v2 1/2] linux-user: Modify 'target_to_host/host_to_target_itimerspec()'
Date: Fri, 7 Aug 2020 11:56:28 +0200 [thread overview]
Message-ID: <f093b4b6-1899-f101-934d-d91f1fe9d5cf@vivier.eu> (raw)
In-Reply-To: <20200722153421.295411-2-Filip.Bozuta@syrmia.com>
Le 22/07/2020 à 17:34, Filip Bozuta a écrit :
> Functions 'target_to_host_itimerspec()' and 'host_to_target_itimerspec()'
> are used to convert values of type 'struct itimerspec' between target and
> host. This type has 'struct timespec' as its fields. That is the reason
> why this patch introduces a little modification to the converting functions
> to be implemented using already existing functions that convert 'struct timespec':
> 'target_to_host_timespec()' and 'host_to_target_timespec()'. This makes the
> code of 'target_to_host_itimerspec()' and 'host_to_target_itimerspec()' more
> clean and readable.
>
> Signed-off-by: Filip Bozuta <Filip.Bozuta@syrmia.com>
> ---
> linux-user/syscall.c | 46 ++++++++++++++++++--------------------------
> 1 file changed, 19 insertions(+), 27 deletions(-)
>
> diff --git a/linux-user/syscall.c b/linux-user/syscall.c
> index 1211e759c2..b1baed346c 100644
> --- a/linux-user/syscall.c
> +++ b/linux-user/syscall.c
> @@ -1236,7 +1236,9 @@ static inline abi_long copy_to_user_timeval64(abi_ulong target_tv_addr,
> defined(TARGET_NR_nanosleep) || defined(TARGET_NR_clock_settime) || \
> defined(TARGET_NR_utimensat) || defined(TARGET_NR_mq_timedsend) || \
> defined(TARGET_NR_mq_timedreceive) || defined(TARGET_NR_ipc) || \
> - defined(TARGET_NR_semop) || defined(TARGET_NR_semtimedop)
> + defined(TARGET_NR_semop) || defined(TARGET_NR_semtimedop) || \
> + defined(TARGET_NR_timer_settime) || \
> + (defined(TARGET_NR_timerfd_settime) && defined(CONFIG_TIMERFD))
> static inline abi_long target_to_host_timespec(struct timespec *host_ts,
> abi_ulong target_addr)
> {
> @@ -6790,46 +6792,36 @@ static inline abi_long target_ftruncate64(void *cpu_env, abi_long arg1,
>
> #if defined(TARGET_NR_timer_settime) || \
> (defined(TARGET_NR_timerfd_settime) && defined(CONFIG_TIMERFD))
> -static inline abi_long target_to_host_itimerspec(struct itimerspec *host_itspec,
> +static inline abi_long target_to_host_itimerspec(struct itimerspec *host_its,
> abi_ulong target_addr)
> {
> - struct target_itimerspec *target_itspec;
> -
> - if (!lock_user_struct(VERIFY_READ, target_itspec, target_addr, 1)) {
> + if (target_to_host_timespec(&host_its->it_interval, target_addr +
> + offsetof(struct target_itimerspec,
> + it_interval)) ||
> + target_to_host_timespec(&host_its->it_value, target_addr +
> + offsetof(struct target_itimerspec,
> + it_value))) {
> return -TARGET_EFAULT;
> }
>
> - host_itspec->it_interval.tv_sec =
> - tswapal(target_itspec->it_interval.tv_sec);
> - host_itspec->it_interval.tv_nsec =
> - tswapal(target_itspec->it_interval.tv_nsec);
> - host_itspec->it_value.tv_sec = tswapal(target_itspec->it_value.tv_sec);
> - host_itspec->it_value.tv_nsec = tswapal(target_itspec->it_value.tv_nsec);
> -
> - unlock_user_struct(target_itspec, target_addr, 1);
> return 0;
> }
> #endif
>
> #if ((defined(TARGET_NR_timerfd_gettime) || \
> defined(TARGET_NR_timerfd_settime)) && defined(CONFIG_TIMERFD)) || \
> - defined(TARGET_NR_timer_gettime) || defined(TARGET_NR_timer_settime)
> + defined(TARGET_NR_timer_gettime) || defined(TARGET_NR_timer_settime)
> static inline abi_long host_to_target_itimerspec(abi_ulong target_addr,
> - struct itimerspec *host_its)
> -{
> - struct target_itimerspec *target_itspec;
> -
> - if (!lock_user_struct(VERIFY_WRITE, target_itspec, target_addr, 0)) {
> + struct itimerspec *host_its)
> +{
> + if (host_to_target_timespec(target_addr + offsetof(struct target_itimerspec,
> + it_interval),
> + &host_its->it_interval) ||
> + host_to_target_timespec(target_addr + offsetof(struct target_itimerspec,
> + it_value),
> + &host_its->it_value)) {
> return -TARGET_EFAULT;
> }
> -
> - target_itspec->it_interval.tv_sec = tswapal(host_its->it_interval.tv_sec);
> - target_itspec->it_interval.tv_nsec = tswapal(host_its->it_interval.tv_nsec);
> -
> - target_itspec->it_value.tv_sec = tswapal(host_its->it_value.tv_sec);
> - target_itspec->it_value.tv_nsec = tswapal(host_its->it_value.tv_nsec);
> -
> - unlock_user_struct(target_itspec, target_addr, 0);
> return 0;
> }
> #endif
>
Reviewed-by: Laurent Vivier <laurent@vivier.eu>
next prev parent reply other threads:[~2020-08-07 9:57 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-07-22 15:34 [PATCH v2 0/2] This two patch series introduces functionality for a group Filip Bozuta
2020-07-22 15:34 ` [PATCH v2 1/2] linux-user: Modify 'target_to_host/host_to_target_itimerspec()' Filip Bozuta
2020-08-07 9:56 ` Laurent Vivier [this message]
2020-08-07 11:16 ` Laurent Vivier
2020-07-22 15:34 ` [PATCH v2 2/2] linux-user: Add support for a group of 2038 safe syscalls Filip Bozuta
2020-08-07 11:13 ` Laurent Vivier
2020-08-07 11:17 ` Laurent Vivier
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=f093b4b6-1899-f101-934d-d91f1fe9d5cf@vivier.eu \
--to=laurent@vivier.eu \
--cc=Filip.Bozuta@syrmia.com \
--cc=qemu-devel@nongnu.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 NNTP newsgroup(s).