Re: Adding options INSECURE to GENERIC

On Sun, Apr 19, 2020 at 08:18:05PM -0400, Michael wrote:
> That's for accessing IO space. I don't think any of the common PCI
> graphics cards on sparc64 really need it, but the drivers may request
> it because x86.
> So there might be fallout. Trivial to fix fallout but still fallout.

Sounds correct, here is an example:

radeonfb0 at pci4 dev 2 function 0: ATI Technologies Radeon 7000/VE QY (rev. 0x00)
trying to read disabled BIOS...
radeonfb0: No video BIOS, using default clocks
radeonfb0: refclk = 27.000 MHz, refdiv = 12 minpll = 125000, maxpll = 400000
no data for est. mode 640x480x67
radeonfb0: 64 MB aperture at 0x08000000, 64 KB registers at 0x00100000

and IIRC mach64 is similar (with a mem type register window used instead
of the PCish IO space).

So we could either #ifndef __sparc__ the IO access in xsrc, or leave it
there, make it fail in the kernel, and have xsrc deal with failure. The
latter would probably be easier to upstream.

Martin