From: Laurent Vivier <laurent@vivier.eu>
To: Filip Bozuta <Filip.Bozuta@rt-rk.com>, qemu-devel@nongnu.org
Cc: peter.maydell@linaro.org, berrange@redhat.com, arnd@arndb.de,
richard.henderson@linaro.org, jcmvbkbc@gmail.com,
amarkovic@wavecomp.com, philmd@redhat.com
Subject: Re: [PATCH 00/12] linux-user: Add support for real time clock and
Date: Wed, 15 Jan 2020 17:35:17 +0100 [thread overview]
Message-ID: <956c6158-803b-3196-ff92-8dfd2fc03295@vivier.eu> (raw)
In-Reply-To: <1579103618-20217-1-git-send-email-Filip.Bozuta@rt-rk.com>
Hi Filip,
for the next time, please add the version number in the subject of all
the patches of the series, for instance:
[PATCH v6 XX/12]
Thanks,
Laurent
Le 15/01/2020 à 16:53, Filip Bozuta a écrit :
> MIME-Version: 1.0
> Content-Type: text/plain; charset=UTF-8
> Content-Transfer-Encoding: 8bit
>
> This series covers following RTC and sound timer ioctls:
>
> RTC ioctls(22):
>
> * RTC_AIE_ON * RTC_ALM_SET * RTC_WKALM_SET
> * RTC_AIE_OFF * RTC_ALM_READ * RTC_WKALM_RD
> * RTC_UIE_ON * RTC_RD_TIME * RTC_PLL_GET
> * RTC_UIE_OFF * RTC_SET_TIME * RTC_PLL_SET
> * RTC_PIE_ON * RTC_IRQP_READ * RTC_VL_READ
> * RTC_PIE_OFF * RTC_IRQP_SET * RTC_VL_CLR
> * RTC_WIE_ON * RTC_EPOCH_READ
> * RTC_WIE_OFF * RTC_EPOCH_SET
>
> Sound timer ioctls(14):
>
> * SNDRV_TIMER_IOCTL_PVERSION * SNDRV_TIMER_IOCTL_INFO
> * SNDRV_TIMER_IOCTL_NEXT_DEVICE * SNDRV_TIMER_IOCTL_PARAMS
> * SNDRV_TIMER_IOCTL_TREAD * SNDRV_TIMER_IOCTL_STATUS
> * SNDRV_TIMER_IOCTL_GINFO * SNDRV_TIMER_IOCTL_START
> * SNDRV_TIMER_IOCTL_GPARAMS * SNDRV_TIMER_IOCTL_STOP
> * SNDRV_TIMER_IOCTL_GSTATUS * SNDRV_TIMER_IOCTL_CONTINUE
> * SNDRV_TIMER_IOCTL_SELECT * SNDRV_TIMER_IOCTL_PAUSE
>
> The functionalities of individual ioctls were described in this series
> patch commit messages.
>
> Testing method for RTC ioctls:
>
> Mini test programs were written for each ioctl. Those programs were
> compiled (sometimes using cross-compilers) for the following
> architectures:
>
> * Intel 64-bit (little endian)
> * Power pc 32-bit (big endian)
> * Power pc 64-bit (big endian)
>
> The corresponding native programs were executed without using
> QEMU on following hosts:
>
> * Intel Core i7-4790K (x86_64 host)
> * Power 7447A (ppc32 host)
>
> All applicable compiled programs were in turn executed through QEMU
> and the results obtained were the same ones gotten for native
> execution.
>
> Example of a test program:
>
> For ioctl RTC_RD_TIME the following test program was used:
>
> #include <stdio.h>
> #include <stdlib.h>
> #include <linux/rtc.h>
> #include <fcntl.h>
> #include <linux/input.h>
> #include <sys/types.h>
> #include <unistd.h>
>
> #define ERROR -1
>
> int main()
> {
>
> int fd = open("/dev/rtc", O_RDWR | O_NONBLOCK);
>
> if(fd == ERROR)
> {
> perror("open");
> return -1;
> }
>
> struct rtc_time cur_time;
>
> if(ioctl(fd, RTC_RD_TIME, &cur_time) < 0)
> {
> perror("ioctl");
> return -1;
> }
>
> printf("Second: %d, Minute: %d, Hour: %d, Day: %d, Month: %d, Year: %d,",
> cur_time.tm_sec, cur_time.tm_min, cur_time.tm_hour,
> cur_time.tm_mday, cur_time.tm_mon, cur_time.tm_year);
>
> return 0;
> }
>
> Limitations of testing:
>
> The test host pc that was used for testing (intel pc) has RTC
> that doesn't support all RTC features that are accessible
> through ioctls. This means that testing can't discover
> functionality errors related to the third argument of ioctls
> that are used for features which are not supported. For example,
> running the test program for ioctl RTC_EPOCH_READ gives
> the error output: inappropriate ioctl for device. As expected,
> the same output was obtained through QEMU which means that this
> ioctl is recognized in QEMU but doesn't really do anything
> because it is not supported in the host computer's RTC.
>
> Conclusion: Some RTC ioctls need to be tested on computers
> that support their functionalities so that it can be inferred
> that they are really supported in QEMU. In absence of such
> test hosts, the specifications of those ioctls need to be
> carefully checked manually and the implementations should be
> updated accordingly.
>
> Testing method for sound timer ioctls:
>
> The alsa ioctl test suite, that can be found on github
> ("https://github.com/alsa-project/alsa-utils"), was used the test
> the implemented ioctls. The file "timer.c", located in this test
> suite, contains test functions that are used to test alsa timer
> ioctls. This file was compiled (sometimes using cross-compilers)
> for the following architectures:
>
> * Intel 64-bit (little endian)
> * Power pc 32-bit (big endian)
> * Power pc 64-bit (big endian)
>
> The corresponding compiled test files were executed without using
> QEMU on following hosts:
>
> * Intel Core i7-4790K (x86_64 host)
> * Power 7447A (ppc32 host)
>
> The corresponding native compiled test files were executed without using
> QEMU on following hosts:
>
> * Intel Core i7-4790K (x86_64 host)
> * Power 7447A (ppc32 host)
>
> All compiled test files were in turn executed through QEMU
> and the results obtained were the same ones gotten for native
> execution.
>
> Also, mini test programs were written to test further functionalities
> of individual ioctls. Those programs were, like the file "timer.c",
> compiled for different architectures and were executed both natively
> and thgrough QEMU to compare the results.
>
> Example of a test program:
>
> For ioctl SNDRV_TIMER_IOCTL_GINFO the following test program was used:
>
> #include <stdio.h>
> #include <stdlib.h>
> #include <sys/types.h>
> #include <fcntl.h>
> #include <sys/ioctl.h>
> #include <sound/asound.h>
>
> #define ERROR -1
>
> int main()
> {
> int fd = open("/dev/snd/timer", O_RDWR);
>
> if(fd == ERROR)
> {
> perror("open");
> return -1;
> }
>
> struct snd_timer_id id = {SNDRV_TIMER_CLASS_GLOBAL,
> SNDRV_TIMER_SCLASS_NONE, -1,
> SNDRV_TIMER_GLOBAL_SYSTEM, 0};
>
> struct snd_timer_ginfo ginfo;
> ginfo.tid = id;
>
> if(ioctl(fd, SNDRV_TIMER_IOCTL_GINFO, &ginfo) == ERROR)
> {
> perror("ioctl");
> return -1;
> }
>
> printf("flags: %u\n", ginfo.flags);
> printf("card: %d\n", ginfo.card);
> printf("id: %s\n", ginfo.id);
> printf("name: %s\n", ginfo.name);
> printf("reserved0: %lu\n", ginfo.reserved0);
> printf("resolution: %lu\n", ginfo.resolution);
> printf("resolution_min: %lu\n", ginfo.resolution_min);
> printf("reolution_max: %lu\n", ginfo.resolution_max);
> printf("clients: %u\n", ginfo.clients);
> printf("reserved: %s\n", ginfo.reserved);
>
> return 0;
> }
>
> v6:
>
> * fixed one patch by adding a case statement for 'unsigned long'
> to recognize two ioctls that are implemented in that patch
>
> * changed patch descriptions a little bit so that they are more
> comprehensible
>
> v5:
>
> * added support for alsa sound timer ioctls
>
> v4:
>
> * changed patch descriptions so that they are better
> formatted and more comprehensible
>
> v3:
>
> * changed two instances of MK_PTR(TYPE_ULONG) to TYPE_ULONG
>
> v2:
>
> * added description of each ioctl in patches
> * wrote a more detailed cover letter with description of testing
> * changed one instance of TYPE_INT to MK_PTR(TYPE_INT)
>
> Filip Bozuta (12):
> linux-user: Add support for enabling/disabling RTC features using
> ioctls
> linux-user: Add support for getting/setting RTC time and alarm using
> ioctls
> linux-user: Add support for getting/setting RTC periodic interrupt and
> epoch using ioctls
> linux-user: Add support for getting/setting RTC wakeup alarm using
> ioctls
> linux-user: Add support for getting/setting RTC PLL correction using
> ioctls
> linux-user: Add support for read/clear RTC voltage low detector using
> ioctls
> linux-user: Add support for getting alsa timer version and id
> linux-user: Add support for setting alsa timer enhanced read using
> ioctl
> linux-user: Add support for getting/setting specified alsa timer
> parameters using ioctls
> linux-user: Add support for selecting alsa timer using ioctl
> linux-user: Add support for getting/setting selected alsa timer
> parameters using ioctls
> linux-user: Add support for selected alsa timer instructions using
> ioctls
>
> linux-user/ioctls.h | 45 +++++++++++++++++
> linux-user/syscall.c | 3 ++
> linux-user/syscall_defs.h | 121 +++++++++++++++++++++++++++++++++++++++++++++
> linux-user/syscall_types.h | 91 ++++++++++++++++++++++++++++++++++
> 4 files changed, 260 insertions(+)
>
next prev parent reply other threads:[~2020-01-15 16:51 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-01-15 15:53 [PATCH 00/12] linux-user: Add support for real time clock and Filip Bozuta
2020-01-15 15:53 ` [PATCH 01/12] linux-user: Add support for enabling/disabling RTC features using ioctls Filip Bozuta
2020-01-15 15:53 ` [PATCH 02/12] linux-user: Add support for getting/setting RTC time and alarm " Filip Bozuta
2020-01-15 15:53 ` [PATCH 03/12] linux-user: Add support for getting/setting RTC periodic interrupt and epoch " Filip Bozuta
2020-01-15 16:46 ` Laurent Vivier
2020-01-15 15:53 ` [PATCH 04/12] linux-user: Add support for getting/setting RTC wakeup alarm " Filip Bozuta
2020-01-15 15:53 ` [PATCH 05/12] linux-user: Add support for getting/setting RTC PLL correction " Filip Bozuta
2020-01-15 15:53 ` [PATCH 06/12] linux-user: Add support for read/clear RTC voltage low detector " Filip Bozuta
2020-01-15 15:53 ` [PATCH 07/12] linux-user: Add support for getting alsa timer version and id Filip Bozuta
2020-01-15 15:53 ` [PATCH 08/12] linux-user: Add support for setting alsa timer enhanced read using ioctl Filip Bozuta
2020-01-15 16:18 ` Arnd Bergmann
2020-01-15 16:32 ` Laurent Vivier
2020-01-15 16:37 ` Arnd Bergmann
2020-01-15 19:17 ` Filip Bozuta
2020-01-15 19:51 ` Arnd Bergmann
2020-01-15 21:30 ` Laurent Vivier
2020-01-15 21:52 ` Laurent Vivier
2020-01-16 2:49 ` Aleksandar Markovic
2020-01-16 11:27 ` Aleksandar Markovic
2020-01-16 12:00 ` Arnd Bergmann
2020-01-17 20:50 ` Aleksandar Markovic
2020-01-17 21:45 ` Arnd Bergmann
2020-01-17 21:54 ` Alexandre Belloni
2020-01-15 15:53 ` [PATCH 09/12] linux-user: Add support for getting/setting specified alsa timer parameters using ioctls Filip Bozuta
2020-01-15 15:53 ` [PATCH 10/12] linux-user: Add support for selecting alsa timer using ioctl Filip Bozuta
2020-01-15 15:53 ` [PATCH 11/12] linux-user: Add support for getting/setting selected alsa timer parameters using ioctls Filip Bozuta
2020-01-15 15:53 ` [PATCH 12/12] linux-user: Add support for selected alsa timer instructions " Filip Bozuta
2020-01-15 16:35 ` Laurent Vivier [this message]
-- strict thread matches above, loose matches on Subject: below --
2020-01-09 12:59 [PATCH 00/12] linux-user: Add support for real time clock and Filip Bozuta
2020-01-14 12:39 ` Philippe Mathieu-Daudé
2020-01-14 12:47 ` 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=956c6158-803b-3196-ff92-8dfd2fc03295@vivier.eu \
--to=laurent@vivier.eu \
--cc=Filip.Bozuta@rt-rk.com \
--cc=amarkovic@wavecomp.com \
--cc=arnd@arndb.de \
--cc=berrange@redhat.com \
--cc=jcmvbkbc@gmail.com \
--cc=peter.maydell@linaro.org \
--cc=philmd@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=richard.henderson@linaro.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).