<feed xmlns='http://www.w3.org/2005/Atom'>
<title>src-test2/sys/dev/drm2, branch release/11.0.0</title>
<subtitle>FreeBSD source tree</subtitle>
<id>https://cgit-dev.freebsd.org/src-test2/atom?h=release%2F11.0.0</id>
<link rel='self' href='https://cgit-dev.freebsd.org/src-test2/atom?h=release%2F11.0.0'/>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test2/'/>
<updated>2016-08-03T00:03:03Z</updated>
<entry>
<title>MFC r302571,r302572,r302577,r302841:</title>
<updated>2016-08-03T00:03:03Z</updated>
<author>
<name>Enji Cooper</name>
<email>ngie@FreeBSD.org</email>
</author>
<published>2016-08-03T00:03:03Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test2/commit/?id=e6b932cf11bf4ab80f6c8de5d167987b467c7d04'/>
<id>urn:sha1:e6b932cf11bf4ab80f6c8de5d167987b467c7d04</id>
<content type='text'>
Approved by: re (gjb)

r302571:

Remove redundant declaration for radeon_pm_acpi_event_handler(..) to fix
-Wredundant-decls warning

PR:		209924
Tested with:	devel/amd64-gcc (5.3.0)

r302572:

Remove redundant declarations for intel_fbc_enabled(..) and
i915_gem_dump_object(..) to fix -Wredundant-decls warning

PR:		209924
Tested with:	devel/amd64-gcc (5.3.0)

r302577:

Add missing default case to capable(..) function definition

By definition (enum __drm_capabilities), cases other than CAP_SYS_ADMIN
aren't possible. Add in a KASSERT safety belt and return false in
!INVARIANTS case if an invalid value is passed in, as it would be a
programmer error.

This fixes a -Wreturn-type error with gcc 5.3.0.

r302841:

Always panic if an invalid capability is passed to `capable(..)` instead of
just with INVARIANTS

rwatson's point was valid in the sense that if the data passed at runtime is
invalid, it should always trip the invariant, not just in the debug case.
This is a deterrent against malicious input, or input caused by hardware
errors.

Requested by: rwatson
</content>
</entry>
<entry>
<title>[drm] fix up hex_dump_to_buffer to not overflow linebuf.</title>
<updated>2016-06-03T05:01:35Z</updated>
<author>
<name>Adrian Chadd</name>
<email>adrian@FreeBSD.org</email>
</author>
<published>2016-06-03T05:01:35Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test2/commit/?id=57dbe10e6d6253c6355bfdd6240055c1da65f81b'/>
<id>urn:sha1:57dbe10e6d6253c6355bfdd6240055c1da65f81b</id>
<content type='text'>
That check wasn't enough to handle appending a two byte character
following it.

This prevented my T400 (Intel Core 2 Duo P8400) from attaching;
it would panic from a stack overflow detection.
</content>
</entry>
<entry>
<title>Don't repeat the the word 'the'</title>
<updated>2016-05-17T12:52:31Z</updated>
<author>
<name>Eitan Adler</name>
<email>eadler@FreeBSD.org</email>
</author>
<published>2016-05-17T12:52:31Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test2/commit/?id=cef367e6a15b6e3e2aca88d453c74962cf1f94e7'/>
<id>urn:sha1:cef367e6a15b6e3e2aca88d453c74962cf1f94e7</id>
<content type='text'>
(one manual change to fix grammar)

Confirmed With: db
Approved by: secteam (not really, but this is a comment typo fix)
</content>
</entry>
<entry>
<title>Ensure waiting loops terminate during cold boot. This fixes boot with</title>
<updated>2016-05-10T18:18:29Z</updated>
<author>
<name>Hans Petter Selasky</name>
<email>hselasky@FreeBSD.org</email>
</author>
<published>2016-05-10T18:18:29Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test2/commit/?id=6a55ff4866f30ea55a5324e81f11a5f1b66d5478'/>
<id>urn:sha1:6a55ff4866f30ea55a5324e81f11a5f1b66d5478</id>
<content type='text'>
MacBookPro and i915kms_load="YES" in /boot/loader.conf.

A lowlevel timeout in one of the display ports caused an infinite wait
because a ticks/jiffies comparison was constant. The clock subsystem
which makes ticks/jiffies increment is started after the initial
driver probing is done. Refer to sys/kernel.h and SI_SUB_DRIVERS vs
SI_SUB_CLOCKS .

Discussed with:	kmacy @
</content>
</entry>
<entry>
<title>sys/dev: minor spelling fixes.</title>
<updated>2016-05-03T03:41:25Z</updated>
<author>
<name>Pedro F. Giffuni</name>
<email>pfg@FreeBSD.org</email>
</author>
<published>2016-05-03T03:41:25Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test2/commit/?id=453130d9bfc1c6d68b366dfcb041689d69f81295'/>
<id>urn:sha1:453130d9bfc1c6d68b366dfcb041689d69f81295</id>
<content type='text'>
Most affect comments, very few have user-visible effects.
</content>
</entry>
<entry>
<title>Revert bus_get_cpus() for now.</title>
<updated>2016-05-03T01:17:40Z</updated>
<author>
<name>John Baldwin</name>
<email>jhb@FreeBSD.org</email>
</author>
<published>2016-05-03T01:17:40Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test2/commit/?id=8a08b7d36b7946ba88a40afe0165de8671d4f568'/>
<id>urn:sha1:8a08b7d36b7946ba88a40afe0165de8671d4f568</id>
<content type='text'>
I really thought I had run this through the tinderbox before committing,
but many places need &lt;sys/types.h&gt; -&gt; &lt;sys/param.h&gt; for &lt;sys/bus.h&gt; now.
</content>
</entry>
<entry>
<title>Add a new bus method to fetch device-specific CPU sets.</title>
<updated>2016-05-02T18:00:38Z</updated>
<author>
<name>John Baldwin</name>
<email>jhb@FreeBSD.org</email>
</author>
<published>2016-05-02T18:00:38Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test2/commit/?id=bc153c692ff962bda50532211fdc0086a58e6c52'/>
<id>urn:sha1:bc153c692ff962bda50532211fdc0086a58e6c52</id>
<content type='text'>
bus_get_cpus() returns a specified set of CPUs for a device.  It accepts
an enum for the second parameter that indicates the type of cpuset to
request.  Currently two valus are supported:

 - LOCAL_CPUS (on x86 this returns all the CPUs in the package closest to
   the device when DEVICE_NUMA is enabled)
 - INTR_CPUS (like LOCAL_CPUS but only returns 1 SMT thread for each core)

For systems that do not support NUMA (or if it is not enabled in the kernel
config), LOCAL_CPUS fails with EINVAL.  INTR_CPUS is mapped to 'all_cpus'
by default.  The idea is that INTR_CPUS should always return a valid set.

Device drivers which want to use per-CPU interrupts should start using
INTR_CPUS instead of simply assigning interrupts to all available CPUs.
In the future we may wish to add tunables to control the policy of
INTR_CPUS (e.g. should it be local-only or global, should it ignore
SMT threads or not).

The x86 nexus driver exposes the internal set of interrupt CPUs from the
the x86 interrupt code via INTR_CPUS.

The ACPI bus driver and PCI bridge drivers use _PXM to return a suitable
LOCAL_CPUS set when _PXM exists and DEVICE_NUMA is enabled.  They also and
the global INTR_CPUS set from the nexus driver with the per-domain set from
_PXM to generate a local INTR_CPUS set for child devices.

Reviewed by:	wblock (manpage)
Differential Revision:	https://reviews.freebsd.org/D5519
</content>
</entry>
<entry>
<title>etc: minor spelling fixes.</title>
<updated>2016-05-02T16:47:28Z</updated>
<author>
<name>Pedro F. Giffuni</name>
<email>pfg@FreeBSD.org</email>
</author>
<published>2016-05-02T16:47:28Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test2/commit/?id=b790c1938dd94d20de89e148169aab4ae3edd771'/>
<id>urn:sha1:b790c1938dd94d20de89e148169aab4ae3edd771</id>
<content type='text'>
Mostly comments but also some user-visible strings.

MFC after:	2 weeks
</content>
</entry>
<entry>
<title>i915kms: Fix memory leak if a CRT is detected</title>
<updated>2016-04-20T03:48:57Z</updated>
<author>
<name>Conrad Meyer</name>
<email>cem@FreeBSD.org</email>
</author>
<published>2016-04-20T03:48:57Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test2/commit/?id=1a7dfcc5a365f4439eb5cd44f3d0e38d44be15ae'/>
<id>urn:sha1:1a7dfcc5a365f4439eb5cd44f3d0e38d44be15ae</id>
<content type='text'>
Reported by:	Coverity
CID:		1090729
Sponsored by:	EMC / Isilon Storage Division
</content>
</entry>
<entry>
<title>drm2(4): Fix double-free in low-memory error path</title>
<updated>2016-04-20T03:45:45Z</updated>
<author>
<name>Conrad Meyer</name>
<email>cem@FreeBSD.org</email>
</author>
<published>2016-04-20T03:45:45Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test2/commit/?id=1940692911968315d1e12d2c3cddfe95a094de58'/>
<id>urn:sha1:1940692911968315d1e12d2c3cddfe95a094de58</id>
<content type='text'>
Reallocf frees 'block'; don't attempt to free it again.

Reported by:	Coverity
CID:		1091165
Sponsored by:	EMC / Isilon Storage Division
</content>
</entry>
</feed>
