<feed xmlns='http://www.w3.org/2005/Atom'>
<title>src/tests/sys/kqueue, branch releng/12.0</title>
<subtitle>FreeBSD source tree</subtitle>
<id>https://cgit-dev.freebsd.org/src/atom?h=releng%2F12.0</id>
<link rel='self' href='https://cgit-dev.freebsd.org/src/atom?h=releng%2F12.0'/>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/'/>
<updated>2018-08-14T20:26:54Z</updated>
<entry>
<title>Fix a couple whitespace errors in r337814.</title>
<updated>2018-08-14T20:26:54Z</updated>
<author>
<name>David Bright</name>
<email>dab@FreeBSD.org</email>
</author>
<published>2018-08-14T20:26:54Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=4e258e2d9f7e2ea1f7a06f8583fa82cc59e0f07f'/>
<id>urn:sha1:4e258e2d9f7e2ea1f7a06f8583fa82cc59e0f07f</id>
<content type='text'>
Reported by:	Renato Botelho &lt;garga.bsd@gmail.com&gt;
MFC after:	3 days
X-MFC-with:	r337814
Sponsored by:	Dell EMC
</content>
</entry>
<entry>
<title>Fix several (more) memory leaks.</title>
<updated>2018-08-14T19:31:06Z</updated>
<author>
<name>David Bright</name>
<email>dab@FreeBSD.org</email>
</author>
<published>2018-08-14T19:31:06Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=45bed28c116daf076d33b00d1830c62f9dcd30b3'/>
<id>urn:sha1:45bed28c116daf076d33b00d1830c62f9dcd30b3</id>
<content type='text'>
A follow-up to r337812 to catch a couple more memory leaks that should
have been included in that change.

Reported by:	Coverity
CID:		1296064, 1296067 (for real this time)
MFC after:	3 days
X-MFC-with:	r337812
Sponsored by:	Dell EMC
</content>
</entry>
<entry>
<title>Fix several memory leaks.</title>
<updated>2018-08-14T19:12:45Z</updated>
<author>
<name>David Bright</name>
<email>dab@FreeBSD.org</email>
</author>
<published>2018-08-14T19:12:45Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=53e992cfb9bfbc5c77588165b3a250a1fa555a23'/>
<id>urn:sha1:53e992cfb9bfbc5c77588165b3a250a1fa555a23</id>
<content type='text'>
The libkqueue tests have several places that leak memory by using an
idiom like:

puts(kevent_to_str(kevp));

Rework to save the pointer returned from kevent_to_str() and then
free() it after it has been used.

Reported by:	asomers (pointer to Coverity), Coverity
CID:		1296063, 1296064, 1296065, 1296066, 1296067, 1350287, 1394960
Sponsored by:	Dell EMC
</content>
</entry>
<entry>
<title>Correct possible misleading error message in kqtest.</title>
<updated>2018-07-30T14:21:49Z</updated>
<author>
<name>David Bright</name>
<email>dab@FreeBSD.org</email>
</author>
<published>2018-07-30T14:21:49Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=d0a179019f599cbd27c2f80daec4b074c886a6d0'/>
<id>urn:sha1:d0a179019f599cbd27c2f80daec4b074c886a6d0</id>
<content type='text'>
ian@ pointed out that in the test_abstime() function time(NULL) is
used twice; once in an "if" test and again in the enclosed error
message. If the true branch was taken and the process got preempted
before the second time(NULL) call, by the time the error message was
generated enough time could have elapsed that the message could claim
that the event came "too early" but print an event time that was after
the expected timeout. Correct by making the time(NULL) call only once
and using that returned time in both the "if" test and the error
message.

Reported by:	ian@
MFC after:	4 days
X-MFC-with:	r336761, r336781, r336802
Sponsored by:	Dell EMC
</content>
</entry>
<entry>
<title>Fix compilation error on some arches after r336761 &amp; r336781.</title>
<updated>2018-07-28T02:53:36Z</updated>
<author>
<name>David Bright</name>
<email>dab@FreeBSD.org</email>
</author>
<published>2018-07-28T02:53:36Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=4faa0dc19364692d2f5fa0e0167203d9c87c7f39'/>
<id>urn:sha1:4faa0dc19364692d2f5fa0e0167203d9c87c7f39</id>
<content type='text'>
Another cast for printing an intmax_t was needed in a kqueue test for
some arches.

Pointy-hat:	me (twice)
MFC after:	1 week
X-MFC-with:	r336761, r336781
Sponsored by:	Dell EMC
</content>
</entry>
<entry>
<title>Fix compilation error on some arches after r336761.</title>
<updated>2018-07-27T20:14:58Z</updated>
<author>
<name>David Bright</name>
<email>dab@FreeBSD.org</email>
</author>
<published>2018-07-27T20:14:58Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=b6429f4bff7076d6ee822a6ba4d41fe15a93d850'/>
<id>urn:sha1:b6429f4bff7076d6ee822a6ba4d41fe15a93d850</id>
<content type='text'>
A cast for printing an intmax_t was needed in a kqueue test for some
arches.

MFC after:	1 week
X-MFC-with:	r336761
Sponsored by:	Dell EMC
</content>
</entry>
<entry>
<title>Allow a EVFILT_TIMER kevent to be updated.</title>
<updated>2018-07-27T13:49:17Z</updated>
<author>
<name>David Bright</name>
<email>dab@FreeBSD.org</email>
</author>
<published>2018-07-27T13:49:17Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=95c05062ec15cf323488d4c5e1986f5866bf7464'/>
<id>urn:sha1:95c05062ec15cf323488d4c5e1986f5866bf7464</id>
<content type='text'>
If a timer is updated (re-added) with a different time period
(specified in the .data field of the kevent), the new time period has
no effect; the timer will not expire until the original time has
elapsed. This violates the documented behavior as the kqueue(2) man
page says (in part) "Re-adding an existing event will modify the
parameters of the original event, and not result in a duplicate
entry."

This modification, adapted from a patch submitted by cem@ to PR214987,
fixes the kqueue system to allow updating a timer entry. The
kevent timer behavior is changed to:

  * When a timer is re-added, update the timer parameters to and
    re-start the timer using the new parameters.
  * Allow updating both active and already expired timers.
  * When the timer has already expired, dequeue any undelivered events
    and clear the count of expirations.

All of these changes address the original PR and also bring the
FreeBSD and macOS kevent timer behaviors into agreement.

A few other changes were made along the way:

  * Update the kqueue(2) man page to reflect the new timer behavior.
  * Fix man page style issues in kqueue(2) diagnosed by igor.
  * Update the timer libkqueue system test to test for the updated
    timer behavior.
  * Fix the (test) libkqueue common.h file so that it includes
    config.h which defines various HAVE_* feature defines, before the
    #if tests for such variables in common.h. This enables the use of
    the actual err(3) family of functions.
  * Fix the usages of the err(3) functions in the tests for incorrect
    type of variables. Those were formerly undiagnosed due to the
    disablement of the err(3) functions (see previous bullet point).

PR:		214987
Reported by:	Brian Wellington &lt;bwelling@xbill.org&gt;
Reviewed by:	kib
MFC after:	1 week
Relnotes:	yes
Sponsored by:	Dell EMC
Differential Revision:	https://reviews.freebsd.org/D15778
</content>
</entry>
<entry>
<title>DIRDEPS_BUILD: Update dependencies.</title>
<updated>2017-10-31T00:07:04Z</updated>
<author>
<name>Bryan Drewery</name>
<email>bdrewery@FreeBSD.org</email>
</author>
<published>2017-10-31T00:07:04Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=ea825d02749f382c3f7e17f28247f20a48733eab'/>
<id>urn:sha1:ea825d02749f382c3f7e17f28247f20a48733eab</id>
<content type='text'>
Sponsored by:	Dell EMC Isilon
</content>
</entry>
<entry>
<title>DIRDEPS_BUILD: Connect new directories.</title>
<updated>2017-10-31T00:04:07Z</updated>
<author>
<name>Bryan Drewery</name>
<email>bdrewery@FreeBSD.org</email>
</author>
<published>2017-10-31T00:04:07Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=3806950135d2c8633ec0764e8807eacc87cf3e10'/>
<id>urn:sha1:3806950135d2c8633ec0764e8807eacc87cf3e10</id>
<content type='text'>
Sponsored by:	Dell EMC Isilon
</content>
</entry>
<entry>
<title>Make test scripts under tests/... non-executable</title>
<updated>2017-08-08T04:59:16Z</updated>
<author>
<name>Enji Cooper</name>
<email>ngie@FreeBSD.org</email>
</author>
<published>2017-08-08T04:59:16Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=d2ba5111c125104b09aa1acd1bfe8af2a24c79cc'/>
<id>urn:sha1:d2ba5111c125104b09aa1acd1bfe8af2a24c79cc</id>
<content type='text'>
Executable bits should be set at install time instead of in the repo.
Setting executable bits on files triggers false positives with Phabricator.

MFC after:	2 months
</content>
</entry>
</feed>
