Thread (11 messages) 11 messages, 6 authors, 2021-09-08

Re: [PATCH] drm/ttm: provide default page protection for UML

From: Johannes Berg <johannes@sipsolutions.net>
Date: 2021-09-02 08:12:23
Also in: dri-devel, lkml

On Thu, 2021-09-02 at 09:10 +0100, Anton Ivanov wrote:
On 02/09/2021 08:43, Johannes Berg wrote:
quoted
On Thu, 2021-09-02 at 07:19 +0100, Anton Ivanov wrote:
quoted
quoted
quoted
I have a question though - why all of DRM is not !UML in config. Not
like we can use them.
I have no idea about that.
Hopefully one of the (other) UML maintainers can answer you.
Touche.

We will discuss that and possibly push a patch to !UML that part of the
tree. IMHO it is not applicable.
As I just said on the other patch, all of this is fallout from my commit
68f5d3f3b654 ("um: add PCI over virtio emulation driver") which is the
first time that you could have PCI on UML.

Without having checked, in this particular case it's probably something
like

	depends on PCI && X86_64

as we've seen in other drivers (idxd, ioat).

The biggest problem is probably that UML internally uses X86_64
(arch/x86/um/Kconfig), which is ... unexpected ... since CONFIG_X86_64
is typically considered the ARCH, and now the ARCH is actually um.

I think we can just fix that and get rid of this entire class of
problems? Something like

https://p.sipsolutions.net/fbac19d86637e286.txt
Good idea. I suspect that DRM will not be the only thing to pop up out 
of this so we might as well fix it "at the top".
Yeah I sent like a handful of similar patches ...

I'll send that patch out in a minute, just writing a reasonable commit
log. Seems to build & work fine for me.

johannes


_______________________________________________
linux-um mailing list
linux-um@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-um
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help