[Spice-devel] [PATCH] server: input_channel: fix input on BE machines

[Denis Kirjanov]

On 5/12/15, Christophe Fergeau <cfergeau at redhat.com> wrote:
> On Mon, May 11, 2015 at 07:08:40PM +0300, Denis Kirjanov wrote:
>> On 5/11/15, Christophe Fergeau <cfergeau at redhat.com> wrote:
>> > Hi,
>> >
>> > On Tue, Apr 14, 2015 at 10:26:20AM +0300, Denis Kirjanov wrote:
>> >> > As this is a similar issue as your patch about the number of
>> >> > channels,
>> >> > can you double check that on your big endian builds, WORDS_BIGENDIAN
>> >> > is
>> >> > defined when generated_server_demarshallers.c is built?
>> >>
>> >> Oh,
>> >>
>> >> I'll double check that.
>> >
>> > Did you manage to get to the bottom of this in the end?
>> >
>> > Christophe
>> >
>>
>> Oh, yes. It looks like that I had a partially worked version of the
>> spice (that was not the upstream stuff, so I had missed something)
>
> But is this patch you sent needed or not?

Nope, the patch is not needed.

 I'd be interested in
> integrating the patches you have for better big-endian support, but
> this particular one raised some questions about a potential deeper
> problem, and I don't have easy access to test machines to dig further
> into this.
> So feel free to send any patches you have that are still needed for you.
>
> Thanks,
>
> Christophe
>