[PATCH v6] tests: Add test for decoding of PTP_* ioctl commands
Harsha Sharma
harshasharmaiitr at gmail.com
Tue Mar 27 16:46:47 UTC 2018
On Tue, Mar 27, 2018 at 10:10 PM, Dmitry V. Levin <ldv at altlinux.org> wrote:
> Hi Harsha,
Hello,
> Is there any progress with this?
Sorry, I'm unable to find any such system and hence unable to check
that part of test.
Thanks.
> On Mon, Mar 12, 2018 at 02:43:23AM +0300, Dmitry V. Levin wrote:
>> On Sun, Mar 11, 2018 at 07:00:50PM +0530, Harsha Sharma wrote:
>> > * tests/ioctl_ptp.c: New file.
>> > * tests/.gitignore: Add ioctl_ptp.
>> > * tests/gen_tests.in (ioctl_ptp): New entry.
>> > * tests/pure_executables.list: Likewise.
>> > ---
>> > Changes in v6:
>> > * Remove unnecessary checks
>> >
>> > Chenges in v5:
>> > * change sysoff->n_samples = PTP_MAX_SAMPLES
>> >
>> > Changes in v4:
>> > * Change 2017 to 2018 in copyright header
>> > * change position of ifdef
>> > * explicitly set value of sysoff->n_samples
>> >
>> > Changes in v3:
>> > * check for macro HAVE_STRUCT_PTP_SYS_OFFSET
>> > * print field names as per fixed decoder
>> > * cover all cases like when call succeeds
>> >
>> > Changes in v2:
>> > * Add decoding of non-zero arguments in test_no_device
>> > * Use TAIL_ALLOC_OBJECT_CONST_PTR
>> > * Change variable names and error message
>> > * Remove hunk from Makefile.am
>> > * Other minor changes
>> >
>> > tests/.gitignore | 1 +
>> > tests/gen_tests.in | 1 +
>> > tests/ioctl_ptp.c | 157 ++++++++++++++++++++++++++++++++++++++++++++
>> > tests/pure_executables.list | 1 +
>> > 4 files changed, 160 insertions(+)
>> > create mode 100644 tests/ioctl_ptp.c
>> >
>> > diff --git a/tests/.gitignore b/tests/.gitignore
>> > index dd0c8e12..fb240e32 100644
>> > --- a/tests/.gitignore
>> > +++ b/tests/.gitignore
>> > @@ -137,6 +137,7 @@ ioctl_loop-nv
>> > ioctl_loop-v
>> > ioctl_mtd
>> > ioctl_nsfs
>> > +ioctl_ptp
>> > ioctl_rtc
>> > ioctl_rtc-v
>> > ioctl_scsi
>> > diff --git a/tests/gen_tests.in b/tests/gen_tests.in
>> > index 55cc98de..b37adcf7 100644
>> > --- a/tests/gen_tests.in
>> > +++ b/tests/gen_tests.in
>> > @@ -136,6 +136,7 @@ ioctl_loop-nv +ioctl.test -a22 -e verbose=none
>> > ioctl_loop-v +ioctl.test -v
>> > ioctl_mtd +ioctl.test
>> > ioctl_nsfs +ioctl.test -esignal=none
>> > +ioctl_ptp +ioctl.test
>> > ioctl_rtc +ioctl.test
>> > ioctl_rtc-v +ioctl.test -v
>> > ioctl_scsi +ioctl.test
>> > diff --git a/tests/ioctl_ptp.c b/tests/ioctl_ptp.c
>> > new file mode 100644
>> > index 00000000..9fbc320f
>> > --- /dev/null
>> > +++ b/tests/ioctl_ptp.c
>> > @@ -0,0 +1,157 @@
>> > +/*
>> > + * Check decoding of PTP_* commands of ioctl syscall.
>> > + *
>> > + * Copyright (c) 2018 Harsha Sharma <harshasharmaiitr at gmail.com>
>> > + * All rights reserved.
>> > + *
>> > + * Redistribution and use in source and binary forms, with or without
>> > + * modification, are permitted provided that the following conditions
>> > + * are met:
>> > + * 1. Redistributions of source code must retain the above copyright
>> > + * notice, this list of conditions and the following disclaimer.
>> > + * 2. Redistributions in binary form must reproduce the above copyright
>> > + * notice, this list of conditions and the following disclaimer in the
>> > + * documentation and/or other materials provided with the distribution.
>> > + * 3. The name of the author may not be used to endorse or promote products
>> > + * derived from this software without specific prior written permission.
>> > + *
>> > + * THIS SOFTWARE IS PROVIDED BY THE AUTHOR ``AS IS'' AND ANY EXPRESS OR
>> > + * IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
>> > + * OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED.
>> > + * IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY DIRECT, INDIRECT,
>> > + * INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT
>> > + * NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
>> > + * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
>> > + * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
>> > + * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
>> > + * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
>> > + */
>> > +
>> > +#include "tests.h"
>> > +
>> > +#ifdef HAVE_STRUCT_PTP_SYS_OFFSET
>> > +
>> > +#include "xlat.h"
>> > +#include <fcntl.h>
>> > +#include <stdio.h>
>> > +#include <stdlib.h>
>> > +#include <sys/ioctl.h>
>> > +#include <inttypes.h>
>> > +#include <unistd.h>
>> > +#include <linux/ptp_clock.h>
>> > +
>> > +#include "xlat/ptp_flags_options.h"
>> > +
>> > +static void
>> > +test_no_device(void)
>> > +{
>> > + TAIL_ALLOC_OBJECT_CONST_PTR(struct ptp_clock_caps, caps);
>> > + fill_memory(caps, sizeof(*caps));
>> > + TAIL_ALLOC_OBJECT_CONST_PTR(struct ptp_sys_offset, sysoff);
>> > + fill_memory(sysoff, sizeof(*sysoff));
>> > + TAIL_ALLOC_OBJECT_CONST_PTR(struct ptp_extts_request, extts);
>> > + fill_memory(extts, sizeof(*extts));
>> > + TAIL_ALLOC_OBJECT_CONST_PTR(struct ptp_perout_request, perout);
>> > +
>> > + /* PTP_CLOCK_GETCAPS */
>> > + ioctl(-1, PTP_CLOCK_GETCAPS, NULL);
>> > + printf("ioctl(-1, PTP_CLOCK_GETCAPS, NULL) = -1 EBADF (%m)\n");
>> > + ioctl(-1, PTP_CLOCK_GETCAPS, caps);
>> > + printf("ioctl(-1, PTP_CLOCK_GETCAPS, %p) = -1 EBADF (%m)\n", caps);
>> > +
>> > + /* PTP_SYS_OFFSET */
>> > + ioctl(-1, PTP_SYS_OFFSET, NULL);
>> > + printf("ioctl(-1, PTP_SYS_OFFSET, NULL) = -1 EBADF (%m)\n");
>> > + ioctl(-1, PTP_SYS_OFFSET, sysoff);
>> > + printf("ioctl(-1, PTP_SYS_OFFSET, {n_samples=%u}) = -1 EBADF (%m)\n",
>> > + sysoff->n_samples);
>> > +
>> > + /* PTP_ENABLE_PPS */
>> > + ioctl(-1, PTP_ENABLE_PPS, 0);
>> > + printf("ioctl(-1, PTP_ENABLE_PPS, 0) = -1 EBADF (%m)\n");
>> > + ioctl(-1, PTP_ENABLE_PPS, 1);
>> > + printf("ioctl(-1, PTP_ENABLE_PPS, 1) = -1 EBADF (%m)\n");
>> > +
>> > + /* PTP_EXTTS_REQUEST */
>> > + ioctl(-1, PTP_EXTTS_REQUEST, NULL);
>> > + printf("ioctl(-1, PTP_EXTTS_REQUEST, NULL) = -1 EBADF (%m)\n");
>> > + ioctl(-1, PTP_EXTTS_REQUEST, extts);
>> > + printf("ioctl(-1, PTP_EXTTS_REQUEST, {index=%d, flags=", extts->index);
>> > + printflags(ptp_flags_options, extts->flags, "PTP_???");
>> > + printf("}) = -1 EBADF (%m)\n");
>> > +
>> > + /* PTP_PEROUT_REQUEST */
>> > + ioctl(-1, PTP_PEROUT_REQUEST, NULL);
>> > + printf("ioctl(-1, PTP_PEROUT_REQUEST, NULL) = -1 EBADF (%m)\n");
>> > + ioctl(-1, PTP_PEROUT_REQUEST, perout);
>> > + printf("ioctl(-1, PTP_PEROUT_REQUEST, {start={sec=%" PRId64 ""
>> > + ", nsec=%" PRIu32 "}, period={sec=%" PRId64 ", nsec=%" PRIu32 "}"
>> > + ", index=%d, flags=", (int64_t)perout->start.sec,
>> > + perout->start.nsec, (int64_t)perout->period.sec,
>> > + perout->period.nsec, perout->index);
>> > + printflags(ptp_flags_options, perout->flags, "PTP_???");
>> > + printf("}) = -1 EBADF (%m)\n");
>> > +}
>> > +static void
>> > +test_ptp0_device(void)
>> > +{
>> > +
>> > + TAIL_ALLOC_OBJECT_CONST_PTR(struct ptp_clock_caps, caps);
>> > + fill_memory(caps, sizeof(*caps));
>> > + TAIL_ALLOC_OBJECT_CONST_PTR(struct ptp_sys_offset, sysoff);
>> > + fill_memory(sysoff, sizeof(*sysoff));
>> > +
>> > + static const char device[] = "/dev/ptp0";
>> > + int fd = open(device, O_RDWR);
>> > + if (fd < 0) {
>> > + perror(device);
>> > + return;
>> > + }
>> > +
>> > + /* PTP_CLOCK_GETCAPS */
>> > + int getcaps_ret = ioctl(fd, PTP_CLOCK_GETCAPS, caps);
>> > + if (getcaps_ret < 0) {
>> > + printf("ioctl(%d, PTP_CLOCK_GETCAPS, %p) = %s\n",
>> > + fd, caps, sprintrc(getcaps_ret));
>> > + } else {
>> > + printf("ioctl(%d, PTP_CLOCK_GETCAPS, {max_adj=%d, n_alarm=%d"
>> > + ", n_ext_ts=%d, n_per_out=%d, pps=%d}) = %s\n",
>> > + fd, caps->max_adj, caps->n_alarm, caps->n_ext_ts,
>> > + caps->n_per_out, caps->pps, sprintrc(getcaps_ret));
>> > + }
>> > +
>> > + /* PTP_SYS_OFFSET */
>> > + sysoff->n_samples = PTP_MAX_SAMPLES;
>> > + int sysoff_ret = ioctl(fd, PTP_SYS_OFFSET, sysoff);
>> > + if (sysoff_ret < 0) {
>> > + printf("ioctl(%d, PTP_SYS_OFFSET, {n_samples=%u}) = %s\n",
>> > + fd, sysoff->n_samples, sprintrc(sysoff_ret));
>> > + } else {
>> > + unsigned int i;
>> > +
>> > + printf("ioctl(%d, PTP_SYS_OFFSET, ts=[", fd);
>>
>> I'm sure the success path of decoder prints the same structure that starts
>> with {n_samples=%u
>>
>> Please find a system where PTP_SYS_OFFSET works and make sure the test
>> passes on that system.
>>
>> > + for (i = 0; i < 2 * sysoff->n_samples + 1; ++i) {
>> > + if (i > 0)
>> > + printf(", ");
>> > + printf("{sec=%" PRId64 ", nsec=%" PRIu32 "}",
>> > + (int64_t)sysoff->ts[i].sec, sysoff->ts[i].nsec);
>> > + }
>> > + printf("]) = %s\n", sprintrc(sysoff_ret));
>>
>> At this point errno may be clobbered by earlier printf calls, with
>> sprintrc returning garbage.
>
>
> --
> ldv
More information about the Strace-devel
mailing list