[OE-core] [PATCH 01/10] oeqa/targetcontrol.py: simplify checking for qemu_use_kvm
Robert Yang
liezhi.yang at windriver.com
Mon Aug 21 08:02:54 UTC 2017
Ping.
// Robert
On 08/09/2017 05:46 PM, Robert Yang wrote:
>
>
> On 08/08/2017 11:30 PM, Richard Purdie wrote:
>> On Mon, 2017-07-31 at 02:50 -0700, Robert Yang wrote:
>>> The "if qemu_use_kvm" is not needed.
>>>
>>> Signed-off-by: Robert Yang <liezhi.yang at windriver.com>
>>> ---
>>> meta/lib/oeqa/targetcontrol.py | 5 ++---
>>> 1 file changed, 2 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/meta/lib/oeqa/targetcontrol.py
>>> b/meta/lib/oeqa/targetcontrol.py
>>> index 3255e3a5c63..11e6c820e85 100644
>>> --- a/meta/lib/oeqa/targetcontrol.py
>>> +++ b/meta/lib/oeqa/targetcontrol.py
>>> @@ -132,9 +132,8 @@ class QemuTarget(BaseTarget):
>>> dump_host_cmds = d.getVar("testimage_dump_host")
>>> dump_dir = d.getVar("TESTIMAGE_DUMP_DIR")
>>> qemu_use_kvm = d.getVar("QEMU_USE_KVM")
>>> - if qemu_use_kvm and \
>>> - (qemu_use_kvm == "True" and "x86" in d.getVar("MACHINE") or \
>>> - d.getVar("MACHINE") in qemu_use_kvm.split()):
>>> + if qemu_use_kvm == "True" and "x86" in d.getVar("MACHINE") or \
>>> + d.getVar("MACHINE") in qemu_use_kvm.split():
>>> use_kvm = True
>>> else:
>>> use_kvm = False
>>>
>>
>> It is needed since qemu_use_kvm could be None and None.split() will
>> cause an error.
>
> Thanks, I removed from the repo:
>
> git://git.openembedded.org/openembedded-core-contrib rbt/ptest
> http://cgit.openembedded.org/openembedded-core-contrib/log/?h=rbt/ptest
>
> // Robert
>
>>
>> Cheers,
>>
>> Richard
>>
More information about the Openembedded-core
mailing list