On 29.05.14 23:52, Benjamin Herrenschmidt wrote:

On Thu, 2014-05-29 at 23:27 +0200, Alexander Graf wrote:

quoted

On 29.05.14 09:45, Michael Neuling wrote:

quoted

quoted

quoted

+/* Values for 2nd argument to H_SET_MODE */
+#define H_SET_MODE_RESOURCE_SET_CIABR        1
+#define H_SET_MODE_RESOURCE_SET_DAWR        2
+#define H_SET_MODE_RESOURCE_ADDR_TRANS_MODE    3
+#define H_SET_MODE_RESOURCE_LE            4

Much better, but I think you want to make use of these in non-kvm code too,
no? At least the LE one is definitely already implemented as call :)

Sure but that's a different patch.... below.

Ben, how would you like to handle these 2 patches? If you give me an ack
I can just put this patch into my kvm queue. Alternatively we could both
carry a patch that adds the H_SET_MODE header bits only and whoever hits
Linus' tree first wins ;).

No biggie. Worst case it's a trivial conflict.

Well, the way the patches are split right now it won't be a conflict, 
but a build failure on either side.


Alex