[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/3] tests/acceptance: Handle ppc64le host arch
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH 2/3] tests/acceptance: Handle ppc64le host arch correctly |
Date: |
Fri, 14 Jun 2019 07:55:53 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.6.1 |
On 6/14/19 3:53 AM, David Gibson wrote:
> On Fri, Jun 14, 2019 at 11:40:56AM +1000, David Gibson wrote:
>> On Thu, Jun 13, 2019 at 11:01:19AM +0200, Philippe Mathieu-Daudé wrote:
>>> On 6/13/19 8:07 AM, David Gibson wrote:
>>>> ppc64 and ppc64le are different archs from the host kernel point of view
>>>> and are advertised as such in uname. But these cover the same set of CPUs,
>>>> just in different endianness modes. qemu-system-ppc64 handles both modes,
>>>> so make sure we select the correct binary when running on ppc64le host
>>>> architecture.
>>>>
>>>> Signed-off-by: David Gibson <address@hidden>
>>>> ---
>>>> tests/acceptance/avocado_qemu/__init__.py | 2 ++
>>>> 1 file changed, 2 insertions(+)
>>>>
>>>> diff --git a/tests/acceptance/avocado_qemu/__init__.py
>>>> b/tests/acceptance/avocado_qemu/__init__.py
>>>> index 2b236a1cf0..0ba9c536f4 100644
>>>> --- a/tests/acceptance/avocado_qemu/__init__.py
>>>> +++ b/tests/acceptance/avocado_qemu/__init__.py
>>>> @@ -39,6 +39,8 @@ def pick_default_qemu_bin(arch=None):
>>>> """
>>>> if arch is None:
>>>> arch = os.uname()[4]
>>>> + if arch == 'ppc64le':
>>>> + arch = 'ppc64'
>>>
>>> I prefer the generic patch from Cleber:
>>> https://lists.gnu.org/archive/html/qemu-devel/2018-10/msg03418.html
>>> (I guess remember another version with a json file)
>>
>> I hadn't seen that, that does indeed look better.
>
> Hrm. Well, it is better but it's been outstanding for like 8 months.
> I'd just like to get this working on a ppc host in the near future.
True :S I wanted you to know Cleber was working on this, but I don't
have any problem to get your fix merged and Cleber work added on top of
it, so:
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
signature.asc
Description: OpenPGP digital signature