On 2014-03-20, at 6:27 PM, David H. Gutteridge wrote: > On Tue, 18 Mar 2014 at 19:17:01, Taylor R Campbell wrote: >> I merged the riastradh-drm2 branch to HEAD today. This shouldn't >> cause any problems for anyone, because it touched very little outside >> sys/external/bsd/drm2 -- it's not hooked into any kernels other than >> the new amd64/DRMKMS one. But let me know if you observe any fallout. >> >> Update to userland X.org should be coming soon, so that userlands can >> take advantage of the new DRM/KMS drivers. > > Hello, > > I doubt I'm telling you anything you don't already know, but I tried > compiling a DRMKMS kernel for both amd64 and i386 to test, and > neither compiled.
> With i386, I hit this first: > > In file included from > /usr/builds/netbsd-current/src/sys/external/bsd/drm2/dist/include/drm/drmP.h:52:0, > from > /usr/builds/netbsd-current/src/sys/external/bsd/drm2/dist/drm/drm_agpsupport.c:34: > /usr/builds/netbsd-current/src/sys/external/bsd/drm2/include/linux/pci.h: In > function 'pci_bus_alloc_resource': > /usr/builds/netbsd-current/src/sys/external/bsd/drm2/include/linux/pci.h:255:6: > error: large integer implicitly truncated to unsigned type > *** [drm_agpsupport.o] Error code 1 > > I realize you only provided an amd64 kernel, the implication being > i386 might not yet be supported, but I tried it anyway, as the machine > I'd test with isn't capable of running 64-bit code. > > From looking at the code, it's clear you're already aware of the > issue, given your "XXX" notation. > > error = bus_space_alloc(bst, start, 0xffffffffffffffffULL /* XXX */, > size, align, 0, 0, &resource->start, &resource->r_bsh); > If this is of interest to anyone, I opened a PR detailing some issues that prevent this code from being used on i386. (The PR is 48676.) Dave
