file "/usr/src/sys/dev/pci/drm/drm_linux.c", line 1277
...
[1] https://thfr.info/tmp/DG2-error-20230203.jpg
>
> Index: sys/dev/pci/drm/i915/i915_pci.c
> ===
> RCS file: /cvs/src/sys/dev/pci/drm/i915/i915_pci.
On Fri, Feb 03, 2023 at 02:27:09PM -0500, Thomas Frohwein wrote:
> On Fri, Feb 03, 2023 at 12:54:52PM +1100, Jonathan Gray wrote:
> [...]
> > > +xehp_load_dss_mask: stub
> > > +xehp_load_dss_mask: stub
> > > +intel_slicemask_from_xehp_dssmask: stub
> > > +intel_slicemask_from_xehp_dssmask: stub
>
Hi,
kevent(2) system call is ulocked more than year ago. Since select(2)
and pselect(2) are kqueue(2)/kevent(2) wrappers, it makes sense to
unlock them too. select(2) does the temporary kernel event queue
initialization for this call only and does queue scan with events
conversion between
On Fri, Feb 03, 2023 at 12:54:52PM +1100, Jonathan Gray wrote:
[...]
> > +xehp_load_dss_mask: stub
> > +xehp_load_dss_mask: stub
> > +intel_slicemask_from_xehp_dssmask: stub
> > +intel_slicemask_from_xehp_dssmask: stub
> > +i915_gem_stolen_lmem_setup: stub
> >
> Here are a few more instances of KASSERT being placed in an #ifdef DIAGNOSTIC
> block. This is redundant, as KASSERT is already a nop if DIAGNOSTIC is not
> defined.
Yes and no. You are right that KASSERT is a nop on non-DIAGNOSTIC
kernels, but there are sometimes good reasons to keep a single