>>>
>>> Which header are these coming from, and why aren't we including it? And
is it
>>> going to still build with CONFIG_KVM=n?
>>
>> From include/asm/hvcall.h in the h_set_mode patch set I sent before.
>>
>> And yes it compiles with CONFIG_KVM=n fine.
>
>
> Please split that patch into one that
On 30.05.14 11:44, Michael Neuling wrote:
>>>
>>> Which header are these coming from, and why aren't we including
it? And is it
>>> going to still build with CONFIG_KVM=n?
>>
>> From include/asm/hvcall.h in the h_set_mode patch set I sent before.
>>
>> And yes it compiles with CONFIG_KVM=n
2
+#define H_SET_MODE_RESOURCE_ADDR_TRANS_MODE3
+#define H_SET_MODE_RESOURCE_LE4
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 :)
powerpc/pseries: Use new defines when calling
e H_SET_MODE_RESOURCE_SET_DAWR2
> > > > +#define H_SET_MODE_RESOURCE_ADDR_TRANS_MODE3
> > > > +#define H_SET_MODE_RESOURCE_LE4
> > >
> > > Much better, but I think you want to make use of these in non-kvm code
> > > too,
> > > no?
_ADDR_TRANS_MODE3
> > > +#define H_SET_MODE_RESOURCE_LE4
> >
> > 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 :)
>
> powerpc/pseries: Use new defines when
On 29.05.14 23:52, Benjamin Herrenschmidt wrote:
On Thu, 2014-05-29 at 23:27 +0200, Alexander Graf wrote:
On 29.05.14 09:45, Michael Neuling wrote:
+/* Values for 2nd argument to H_SET_MODE */
+#define H_SET_MODE_RESOURCE_SET_CIABR1
+#define H_SET_MODE_RESOURCE_SET_DAWR2
+#defi
only and whoever hits
> Linus' tree first wins ;).
No biggie. Worst case it's a trivial conflict.
Cheers,
Ben.
>
> Alex
>
> >
> > Mikey
> >
> >
> > powerpc/pseries: Use new defines when calling h_set_mode
> >
> > Now that
queue. Alternatively we could both
carry a patch that adds the H_SET_MODE header bits only and whoever hits
Linus' tree first wins ;).
Alex
Mikey
powerpc/pseries: Use new defines when calling h_set_mode
Now that we define these in the KVM code, use these defines when we call
h_set
> 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.
Mikey
powerpc/pseries: Use new defines when calling h_set_mode
Now that we define these