From owner-svn-src-head@freebsd.org Tue Nov 27 18:35:06 2018 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 150EA1157A04; Tue, 27 Nov 2018 18:35:06 +0000 (UTC) (envelope-from cse.cem@gmail.com) Received: from mail-io1-f51.google.com (mail-io1-f51.google.com [209.85.166.51]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 688FD8C9AA; Tue, 27 Nov 2018 18:35:05 +0000 (UTC) (envelope-from cse.cem@gmail.com) Received: by mail-io1-f51.google.com with SMTP id u19so17801995ioc.2; Tue, 27 Nov 2018 10:35:05 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:reply-to :from:date:message-id:subject:to:cc; bh=mw8FawHzWYmqsnJ+2gcDfifCZJLvu2Nf0Xh/zLPTdek=; b=WCcshwAMHi8GkfDxRuITyXBvZq2ccmnfoJjwLpNS7fJ+Q5e+6R3/tczszqPLrQpatK eI218W0fsxpbas0F4bThRgMeC1N8CU9+ovp82I1Vc9Q5tbj2GNFpqxJ58pVDPCcPwsu7 I+LmBZwOatDcI33J25TCpaNlIGSiPQG+EfhKbhrHpSC6kaQ/F+NGNSi9SLAVzhRDEmZL w3PfXVsTmxrOKNULBknkvflYR2UJFCR0oaFcDDwfEQT87ohKZQFCfSWDDw4Dyi+IXLrD lxq9KYNeubzNgvmKo3EY5OQmUJO14Z+GFyVG8+16EPllcjXwDzVXOUxpdkWMaxV4mP7Z FDBA== X-Gm-Message-State: AA+aEWY4va1eGZ+IXaSaQ7uaX+DSP7YpTHRo+UBDPWs0sArdOE2E4Gs/ DGGHUem+d8m7xzZx797vS+s33sO7 X-Google-Smtp-Source: AFSGD/VMts5we0V1uDyBbublL8YvJh/OJ6kqrQZZv8x2ABYm45Uge10wjjJYo+2yI61S9yWIyqLjsw== X-Received: by 2002:a5e:c107:: with SMTP id v7mr6526613iol.155.1543343698201; Tue, 27 Nov 2018 10:34:58 -0800 (PST) Received: from mail-io1-f51.google.com (mail-io1-f51.google.com. [209.85.166.51]) by smtp.gmail.com with ESMTPSA id p74sm1625285iod.14.2018.11.27.10.34.57 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 27 Nov 2018 10:34:58 -0800 (PST) Received: by mail-io1-f51.google.com with SMTP id r200so17781250iod.11; Tue, 27 Nov 2018 10:34:57 -0800 (PST) X-Received: by 2002:a6b:6119:: with SMTP id v25mr27077137iob.107.1543343697354; Tue, 27 Nov 2018 10:34:57 -0800 (PST) MIME-Version: 1.0 References: <201811271512.wARFCYmP092105@repo.freebsd.org> In-Reply-To: <201811271512.wARFCYmP092105@repo.freebsd.org> Reply-To: cem@freebsd.org From: Conrad Meyer Date: Tue, 27 Nov 2018 10:34:46 -0800 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: svn commit: r341068 - head/tests/sys/kqueue/libkqueue To: dab@freebsd.org Cc: src-committers , svn-src-all@freebsd.org, svn-src-head@freebsd.org Content-Type: text/plain; charset="UTF-8" X-Rspamd-Queue-Id: 688FD8C9AA X-Spamd-Result: default: False [-5.92 / 15.00]; ARC_NA(0.00)[]; RCVD_VIA_SMTP_AUTH(0.00)[]; HAS_REPLYTO(0.00)[cem@freebsd.org]; FROM_HAS_DN(0.00)[]; RCPT_COUNT_THREE(0.00)[4]; R_SPF_ALLOW(-0.20)[+ip4:209.85.128.0/17]; TO_MATCH_ENVRCPT_ALL(0.00)[]; MIME_GOOD(-0.10)[text/plain]; RCVD_TLS_LAST(0.00)[]; DMARC_NA(0.00)[freebsd.org]; TO_DN_SOME(0.00)[]; REPLYTO_ADDR_EQ_FROM(0.00)[]; NEURAL_HAM_LONG(-1.00)[-1.000,0]; RCVD_COUNT_THREE(0.00)[4]; IP_SCORE(-2.92)[ip: (-9.63), ipnet: 209.85.128.0/17(-3.49), asn: 15169(-1.38), country: US(-0.09)]; MX_GOOD(-0.01)[cached: alt3.gmail-smtp-in.l.google.com]; NEURAL_HAM_SHORT(-0.99)[-0.994,0]; RCVD_IN_DNSWL_NONE(0.00)[51.166.85.209.list.dnswl.org : 127.0.5.0]; NEURAL_HAM_MEDIUM(-1.00)[-1.000,0]; FORGED_SENDER(0.30)[cem@freebsd.org,csecem@gmail.com]; R_DKIM_NA(0.00)[]; FREEMAIL_ENVFROM(0.00)[gmail.com]; ASN(0.00)[asn:15169, ipnet:209.85.128.0/17, country:US]; TAGGED_FROM(0.00)[]; FROM_NEQ_ENVFROM(0.00)[cem@freebsd.org,csecem@gmail.com] X-Rspamd-Server: mx1.freebsd.org X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 27 Nov 2018 18:35:06 -0000 As long as we were correcting indentation whitespace, it seems like it would have been good to go to tabs, per style(9), instead of spaces. Or are these considered contrib code from NetBSD? Best, Conrad On Tue, Nov 27, 2018 at 7:12 AM David Bright wrote: > > Author: dab > Date: Tue Nov 27 15:12:34 2018 > New Revision: 341068 > URL: https://svnweb.freebsd.org/changeset/base/341068 > > Log: > Make whitespace more consistent in libkqueue tests. > > After r337820, which "corrected" some spaces-instead-of-tab whitespace > issues in the libkqueue tests, jmg@ pointed out that these files were > originally space-based, not tab-spaced, and so the correction should > have been to get rid of the tabs that had been introduced in previous > changes, not the spaces. This change does that. This is a whitespace > only change; no functional change is intended. > > Reported by: jmg@ > MFC after: 3 days > Sponsored by: Dell EMC Isilon > > Modified: > head/tests/sys/kqueue/libkqueue/main.c > head/tests/sys/kqueue/libkqueue/proc.c > head/tests/sys/kqueue/libkqueue/signal.c > head/tests/sys/kqueue/libkqueue/timer.c > head/tests/sys/kqueue/libkqueue/user.c > head/tests/sys/kqueue/libkqueue/vnode.c > > Modified: head/tests/sys/kqueue/libkqueue/main.c > ============================================================================== > --- head/tests/sys/kqueue/libkqueue/main.c Tue Nov 27 14:58:19 2018 (r341067) > +++ head/tests/sys/kqueue/libkqueue/main.c Tue Nov 27 15:12:34 2018 (r341068) > @@ -85,7 +85,7 @@ kevent_get(int kqfd) > struct kevent *kev; > > if ((kev = calloc(1, sizeof(*kev))) == NULL) > - err(1, "out of memory"); > + err(1, "out of memory"); > > nfds = kevent(kqfd, NULL, 0, kev, 1, NULL); > if (nfds < 1) > @@ -103,7 +103,7 @@ kevent_get_timeout(int kqfd, int seconds) > struct timespec timeout = {seconds, 0}; > > if ((kev = calloc(1, sizeof(*kev))) == NULL) > - err(1, "out of memory"); > + err(1, "out of memory"); > > nfds = kevent(kqfd, NULL, 0, kev, 1, &timeout); > if (nfds < 0) { > @@ -123,10 +123,10 @@ kevent_fflags_dump(struct kevent *kev) > > #define KEVFFL_DUMP(attrib) \ > if (kev->fflags & attrib) \ > - strncat(buf, #attrib" ", 64); > + strncat(buf, #attrib" ", 64); > > if ((buf = calloc(1, 1024)) == NULL) > - abort(); > + abort(); > > /* Not every filter has meaningful fflags */ > if (kev->filter == EVFILT_PROC) { > @@ -160,7 +160,7 @@ kevent_fflags_dump(struct kevent *kev) > #endif > buf[strlen(buf) - 1] = ')'; > } else { > - snprintf(buf, 1024, "fflags = %x", kev->fflags); > + snprintf(buf, 1024, "fflags = %x", kev->fflags); > } > > return (buf); > @@ -173,10 +173,10 @@ kevent_flags_dump(struct kevent *kev) > > #define KEVFL_DUMP(attrib) \ > if (kev->flags & attrib) \ > - strncat(buf, #attrib" ", 64); > + strncat(buf, #attrib" ", 64); > > if ((buf = calloc(1, 1024)) == NULL) > - abort(); > + abort(); > > snprintf(buf, 1024, "flags = %d (", kev->flags); > KEVFL_DUMP(EV_ADD); > @@ -208,17 +208,17 @@ kevent_to_str(struct kevent *kev) > > snprintf(&buf[0], sizeof(buf), > "[ident=%ju, filter=%d, %s, %s, data=%jd, udata=%p, " > - "ext=[%jx %jx %jx %jx]", > + "ext=[%jx %jx %jx %jx]", > (uintmax_t) kev->ident, > kev->filter, > flags_str, > fflags_str, > (uintmax_t)kev->data, > kev->udata, > - (uintmax_t)kev->ext[0], > - (uintmax_t)kev->ext[1], > - (uintmax_t)kev->ext[2], > - (uintmax_t)kev->ext[3]); > + (uintmax_t)kev->ext[0], > + (uintmax_t)kev->ext[1], > + (uintmax_t)kev->ext[2], > + (uintmax_t)kev->ext[3]); > > free(flags_str); > free(fflags_str); > @@ -239,10 +239,10 @@ kevent_add(int kqfd, struct kevent *kev, > > EV_SET(kev, ident, filter, flags, fflags, data, NULL); > if (kevent(kqfd, kev, 1, NULL, 0, NULL) < 0) { > - kev_str = kevent_to_str(kev); > - printf("Unable to add the following kevent:\n%s\n", > - kev_str); > - free(kev_str); > + kev_str = kevent_to_str(kev); > + printf("Unable to add the following kevent:\n%s\n", > + kev_str); > + free(kev_str); > err(1, "kevent(): %s", strerror(errno)); > } > } > @@ -265,12 +265,12 @@ kevent_cmp(struct kevent *k1, struct kevent *k2) > k1->data != k2->data || k1->udata != k2->udata || > k1->ext[0] != k2->ext[0] || k1->ext[1] != k2->ext[1] || > k1->ext[0] != k2->ext[2] || k1->ext[0] != k2->ext[3]) { > - kev1_str = kevent_to_str(k1); > - kev2_str = kevent_to_str(k2); > - printf("kevent_cmp: mismatch:\n %s !=\n %s\n", > - kev1_str, kev2_str); > - free(kev1_str); > - free(kev2_str); > + kev1_str = kevent_to_str(k1); > + kev2_str = kevent_to_str(k2); > + printf("kevent_cmp: mismatch:\n %s !=\n %s\n", > + kev1_str, kev2_str); > + free(kev1_str); > + free(kev2_str); > abort(); > } > } > > Modified: head/tests/sys/kqueue/libkqueue/proc.c > ============================================================================== > --- head/tests/sys/kqueue/libkqueue/proc.c Tue Nov 27 14:58:19 2018 (r341067) > +++ head/tests/sys/kqueue/libkqueue/proc.c Tue Nov 27 15:12:34 2018 (r341068) > @@ -45,7 +45,7 @@ add_and_delete(void) > struct stat s; > if (fstat(kqfd, &s) != -1) > errx(1, "kqueue inherited across fork! (%s() at %s:%d)", > - __func__, __FILE__, __LINE__); > + __func__, __FILE__, __LINE__); > > pause(); > exit(2); > > Modified: head/tests/sys/kqueue/libkqueue/signal.c > ============================================================================== > --- head/tests/sys/kqueue/libkqueue/signal.c Tue Nov 27 14:58:19 2018 (r341067) > +++ head/tests/sys/kqueue/libkqueue/signal.c Tue Nov 27 15:12:34 2018 (r341068) > @@ -188,12 +188,12 @@ test_kevent_signal_oneshot(void) > void > test_evfilt_signal() > { > - kqfd = kqueue(); > - test_kevent_signal_add(); > - test_kevent_signal_del(); > - test_kevent_signal_get(); > - test_kevent_signal_disable(); > - test_kevent_signal_enable(); > - test_kevent_signal_oneshot(); > - close(kqfd); > + kqfd = kqueue(); > + test_kevent_signal_add(); > + test_kevent_signal_del(); > + test_kevent_signal_get(); > + test_kevent_signal_disable(); > + test_kevent_signal_enable(); > + test_kevent_signal_oneshot(); > + close(kqfd); > } > > Modified: head/tests/sys/kqueue/libkqueue/timer.c > ============================================================================== > --- head/tests/sys/kqueue/libkqueue/timer.c Tue Nov 27 14:58:19 2018 (r341067) > +++ head/tests/sys/kqueue/libkqueue/timer.c Tue Nov 27 15:12:34 2018 (r341068) > @@ -34,11 +34,10 @@ int kqfd; > static long > now(void) > { > - > - struct timeval tv; > + struct timeval tv; > > - gettimeofday(&tv, NULL); > - return SEC_TO_US(tv.tv_sec) + tv.tv_usec; > + gettimeofday(&tv, NULL); > + return SEC_TO_US(tv.tv_sec) + tv.tv_usec; > } > > /* Sleep for a given number of milliseconds. The timeout is assumed to > @@ -47,13 +46,12 @@ now(void) > void > mssleep(int t) > { > + struct timespec stime = { > + .tv_sec = 0, > + .tv_nsec = US_TO_NS(MS_TO_US(t)), > + }; > > - struct timespec stime = { > - .tv_sec = 0, > - .tv_nsec = US_TO_NS(MS_TO_US(t)), > - }; > - > - nanosleep(&stime, NULL); > + nanosleep(&stime, NULL); > } > > /* Sleep for a given number of microseconds. The timeout is assumed to > @@ -62,13 +60,12 @@ mssleep(int t) > void > ussleep(int t) > { > + struct timespec stime = { > + .tv_sec = 0, > + .tv_nsec = US_TO_NS(t), > + }; > > - struct timespec stime = { > - .tv_sec = 0, > - .tv_nsec = US_TO_NS(t), > - }; > - > - nanosleep(&stime, NULL); > + nanosleep(&stime, NULL); > } > > void > @@ -241,7 +238,7 @@ test_abstime(void) > kevent_cmp(&kev, kevent_get(kqfd)); > stop = time(NULL); > if (stop < start + timeout) > - err(1, "too early %jd %jd", (intmax_t)stop, (intmax_t)(start + timeout)); > + err(1, "too early %jd %jd", (intmax_t)stop, (intmax_t)(start + timeout)); > > /* Check if the event occurs again */ > sleep(3); > @@ -264,16 +261,16 @@ test_update(void) > > /* First set the timer to 1 second */ > EV_SET(&kev, vnode_fd, EVFILT_TIMER, EV_ADD | EV_ONESHOT, > - NOTE_USECONDS, SEC_TO_US(1), (void *)1); > + NOTE_USECONDS, SEC_TO_US(1), (void *)1); > start = now(); > if (kevent(kqfd, &kev, 1, NULL, 0, NULL) < 0) > - err(1, "%s", test_id); > + err(1, "%s", test_id); > > /* Now reduce the timer to 1 ms */ > EV_SET(&kev, vnode_fd, EVFILT_TIMER, EV_ADD | EV_ONESHOT, > - NOTE_USECONDS, MS_TO_US(1), (void *)2); > + NOTE_USECONDS, MS_TO_US(1), (void *)2); > if (kevent(kqfd, &kev, 1, NULL, 0, NULL) < 0) > - err(1, "%s", test_id); > + err(1, "%s", test_id); > > /* Wait for the event */ > kev.flags |= EV_CLEAR; > @@ -288,9 +285,9 @@ test_update(void) > */ > printf("timer expired after %ld us\n", elapsed); > if (elapsed < MS_TO_US(1)) > - errx(1, "early timer expiration: %ld us", elapsed); > + errx(1, "early timer expiration: %ld us", elapsed); > if (elapsed > SEC_TO_US(1)) > - errx(1, "late timer expiration: %ld us", elapsed); > + errx(1, "late timer expiration: %ld us", elapsed); > > success(); > } > @@ -309,9 +306,9 @@ test_update_equal(void) > > /* First set the timer to 1 ms */ > EV_SET(&kev, vnode_fd, EVFILT_TIMER, EV_ADD | EV_ONESHOT, > - NOTE_USECONDS, MS_TO_US(1), NULL); > + NOTE_USECONDS, MS_TO_US(1), NULL); > if (kevent(kqfd, &kev, 1, NULL, 0, NULL) < 0) > - err(1, "%s", test_id); > + err(1, "%s", test_id); > > /* Sleep for a significant fraction of the timeout. */ > ussleep(600); > @@ -319,7 +316,7 @@ test_update_equal(void) > /* Now re-add the timer with the same parameters */ > start = now(); > if (kevent(kqfd, &kev, 1, NULL, 0, NULL) < 0) > - err(1, "%s", test_id); > + err(1, "%s", test_id); > > /* Wait for the event */ > kev.flags |= EV_CLEAR; > @@ -334,7 +331,7 @@ test_update_equal(void) > */ > printf("timer expired after %ld us\n", elapsed); > if (elapsed < MS_TO_US(1)) > - errx(1, "early timer expiration: %ld us", elapsed); > + errx(1, "early timer expiration: %ld us", elapsed); > > success(); > } > @@ -353,9 +350,9 @@ test_update_expired(void) > > /* Set the timer to 1ms */ > EV_SET(&kev, vnode_fd, EVFILT_TIMER, EV_ADD | EV_ONESHOT, > - NOTE_USECONDS, MS_TO_US(1), NULL); > + NOTE_USECONDS, MS_TO_US(1), NULL); > if (kevent(kqfd, &kev, 1, NULL, 0, NULL) < 0) > - err(1, "%s", test_id); > + err(1, "%s", test_id); > > /* Wait for 2 ms to give the timer plenty of time to expire. */ > mssleep(2); > @@ -363,7 +360,7 @@ test_update_expired(void) > /* Now re-add the timer */ > start = now(); > if (kevent(kqfd, &kev, 1, NULL, 0, NULL) < 0) > - err(1, "%s", test_id); > + err(1, "%s", test_id); > > /* Wait for the event */ > kev.flags |= EV_CLEAR; > @@ -378,7 +375,7 @@ test_update_expired(void) > */ > printf("timer expired after %ld us\n", elapsed); > if (elapsed < MS_TO_US(1)) > - errx(1, "early timer expiration: %ld us", elapsed); > + errx(1, "early timer expiration: %ld us", elapsed); > > /* Make sure the re-added timer does not fire. In other words, > * test that the event received above was the only event from the > @@ -405,7 +402,7 @@ test_update_periodic(void) > > EV_SET(&kev, vnode_fd, EVFILT_TIMER, EV_ADD, 0, SEC_TO_MS(1), NULL); > if (kevent(kqfd, &kev, 1, NULL, 0, NULL) < 0) > - err(1, "%s", test_id); > + err(1, "%s", test_id); > > /* Retrieve the event */ > kev.flags = EV_ADD | EV_CLEAR; > @@ -420,7 +417,7 @@ test_update_periodic(void) > EV_SET(&kev, vnode_fd, EVFILT_TIMER, EV_ADD, 0, SEC_TO_MS(2), NULL); > start = now(); > if (kevent(kqfd, &kev, 1, NULL, 0, NULL) < 0) > - err(1, "%s", test_id); > + err(1, "%s", test_id); > > /* Retrieve the event */ > kev.flags = EV_ADD | EV_CLEAR; > @@ -434,12 +431,12 @@ test_update_periodic(void) > */ > printf("timer expired after %ld us\n", elapsed); > if (elapsed < MS_TO_US(2)) > - errx(1, "early timer expiration: %ld us", elapsed); > + errx(1, "early timer expiration: %ld us", elapsed); > > /* Delete the event */ > kev.flags = EV_DELETE; > if (kevent(kqfd, &kev, 1, NULL, 0, NULL) < 0) > - err(1, "%s", test_id); > + err(1, "%s", test_id); > > success(); > } > @@ -467,46 +464,46 @@ test_update_timing(void) > * received is from the update and not the original timer add. > */ > for (sleeptime = MIN_SLEEP, iteration = 1; > - sleeptime < MAX_SLEEP; > - ++sleeptime, ++iteration) { > + sleeptime < MAX_SLEEP; > + ++sleeptime, ++iteration) { > > - /* First set the timer to 1 ms */ > - EV_SET(&kev, vnode_fd, EVFILT_TIMER, EV_ADD | EV_ONESHOT, > - NOTE_USECONDS, MS_TO_US(1), NULL); > - if (kevent(kqfd, &kev, 1, NULL, 0, NULL) < 0) > - err(1, "%s", test_id); > + /* First set the timer to 1 ms */ > + EV_SET(&kev, vnode_fd, EVFILT_TIMER, EV_ADD | EV_ONESHOT, > + NOTE_USECONDS, MS_TO_US(1), NULL); > + if (kevent(kqfd, &kev, 1, NULL, 0, NULL) < 0) > + err(1, "%s", test_id); > > - /* Delay; the delay ranges from less than to greater than the > - * timer period. > - */ > - ussleep(sleeptime); > + /* Delay; the delay ranges from less than to greater than the > + * timer period. > + */ > + ussleep(sleeptime); > > - /* Now re-add the timer with the same parameters */ > - start = now(); > - if (kevent(kqfd, &kev, 1, NULL, 0, NULL) < 0) > - err(1, "%s", test_id); > + /* Now re-add the timer with the same parameters */ > + start = now(); > + if (kevent(kqfd, &kev, 1, NULL, 0, NULL) < 0) > + err(1, "%s", test_id); > > - /* Wait for the event */ > - kev.flags |= EV_CLEAR; > - kev.fflags &= ~NOTE_USECONDS; > - kev.data = 1; > - kevent_cmp(&kev, kevent_get(kqfd)); > - stop = now(); > - elapsed = stop - start; > + /* Wait for the event */ > + kev.flags |= EV_CLEAR; > + kev.fflags &= ~NOTE_USECONDS; > + kev.data = 1; > + kevent_cmp(&kev, kevent_get(kqfd)); > + stop = now(); > + elapsed = stop - start; > > - /* Check that the timer expired after at least 1 ms. This > - * check is to make sure that the timer re-started and that > - * the event is not from the original add of the timer. > - */ > - if (elapsed < MS_TO_US(1)) > - errx(1, "early timer expiration: %ld us", elapsed); > + /* Check that the timer expired after at least 1 ms. This > + * check is to make sure that the timer re-started and that > + * the event is not from the original add of the timer. > + */ > + if (elapsed < MS_TO_US(1)) > + errx(1, "early timer expiration: %ld us", elapsed); > > - /* Make sure the re-added timer does not fire. In other words, > - * test that the event received above was the only event from > - * the add and re-add of the timer. > - */ > - mssleep(2); > - test_no_kevents_quietly(); > + /* Make sure the re-added timer does not fire. In other words, > + * test that the event received above was the only event from > + * the add and re-add of the timer. > + */ > + mssleep(2); > + test_no_kevents_quietly(); > } > > success(); > @@ -515,18 +512,18 @@ test_update_timing(void) > void > test_evfilt_timer() > { > - kqfd = kqueue(); > - test_kevent_timer_add(); > - test_kevent_timer_del(); > - test_kevent_timer_get(); > - test_oneshot(); > - test_periodic(); > - test_abstime(); > - test_update(); > - test_update_equal(); > - test_update_expired(); > - test_update_timing(); > - test_update_periodic(); > - disable_and_enable(); > - close(kqfd); > + kqfd = kqueue(); > + test_kevent_timer_add(); > + test_kevent_timer_del(); > + test_kevent_timer_get(); > + test_oneshot(); > + test_periodic(); > + test_abstime(); > + test_update(); > + test_update_equal(); > + test_update_expired(); > + test_update_timing(); > + test_update_periodic(); > + disable_and_enable(); > + close(kqfd); > } > > Modified: head/tests/sys/kqueue/libkqueue/user.c > ============================================================================== > --- head/tests/sys/kqueue/libkqueue/user.c Tue Nov 27 14:58:19 2018 (r341067) > +++ head/tests/sys/kqueue/libkqueue/user.c Tue Nov 27 15:12:34 2018 (r341068) > @@ -117,7 +117,7 @@ oneshot(void) > void > test_evfilt_user() > { > - kqfd = kqueue(); > + kqfd = kqueue(); > > add_and_delete(); > event_wait(); > @@ -125,5 +125,5 @@ test_evfilt_user() > oneshot(); > /* TODO: try different fflags operations */ > > - close(kqfd); > + close(kqfd); > } > > Modified: head/tests/sys/kqueue/libkqueue/vnode.c > ============================================================================== > --- head/tests/sys/kqueue/libkqueue/vnode.c Tue Nov 27 14:58:19 2018 (r341067) > +++ head/tests/sys/kqueue/libkqueue/vnode.c Tue Nov 27 15:12:34 2018 (r341068) > @@ -251,16 +251,16 @@ test_kevent_vnode_dispatch(void) > void > test_evfilt_vnode() > { > - kqfd = kqueue(); > - test_kevent_vnode_add(); > - test_kevent_vnode_del(); > - test_kevent_vnode_disable_and_enable(); > + kqfd = kqueue(); > + test_kevent_vnode_add(); > + test_kevent_vnode_del(); > + test_kevent_vnode_disable_and_enable(); > #if HAVE_EV_DISPATCH > - test_kevent_vnode_dispatch(); > + test_kevent_vnode_dispatch(); > #endif > - test_kevent_vnode_note_write(); > - test_kevent_vnode_note_attrib(); > - test_kevent_vnode_note_rename(); > - test_kevent_vnode_note_delete(); > - close(kqfd); > + test_kevent_vnode_note_write(); > + test_kevent_vnode_note_attrib(); > + test_kevent_vnode_note_rename(); > + test_kevent_vnode_note_delete(); > + close(kqfd); > } >