[Qemu-devel] [PATCH] qemugdb/coroutine: fix arch_prctl has unknown return type

[Qemu-devel] [PATCH] qemugdb/coroutine: fix arch_prctl has unknown return type

[Vladimir Sementsov-Ogievskiy]

qemu coroutine command results in following error output:

Python Exception <class 'gdb.error'> 'arch_prctl' has unknown return
type; cast the call to its declared return type: Error occurred in
Python command: 'arch_prctl' has unknown return type; cast the call to
its declared return type

Fix it by giving it what it wants: arch_prctl return type.

Information on the topic:
   https://sourceware.org/gdb/onlinedocs/gdb/Calling.html

Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
---
 scripts/qemugdb/coroutine.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/qemugdb/coroutine.py b/scripts/qemugdb/coroutine.py
index ab699794ab..81f811ac00 100644
--- a/scripts/qemugdb/coroutine.py
+++ b/scripts/qemugdb/coroutine.py
@@ -22,7 +22,7 @@ def get_fs_base():
        pthread_self().'''
     # %rsp - 120 is scratch space according to the SystemV ABI
     old = gdb.parse_and_eval('*(uint64_t*)($rsp - 120)')
-    gdb.execute('call arch_prctl(0x1003, $rsp - 120)', False, True)
+    gdb.execute('call (int)arch_prctl(0x1003, $rsp - 120)', False, True)
     fs_base = gdb.parse_and_eval('*(uint64_t*)($rsp - 120)')
     gdb.execute('set *(uint64_t*)($rsp - 120) = %s' % old, False, True)
     return fs_base
-- 
2.18.0

Re: [Qemu-devel] [PATCH] qemugdb/coroutine: fix arch_prctl has unknown return type

[Stefan Hajnoczi]

[-- Attachment #1: Type: text/plain, Size: 818 bytes --]

On Wed, Feb 06, 2019 at 06:14:25PM +0300, Vladimir Sementsov-Ogievskiy wrote:
> qemu coroutine command results in following error output:
> 
> Python Exception <class 'gdb.error'> 'arch_prctl' has unknown return
> type; cast the call to its declared return type: Error occurred in
> Python command: 'arch_prctl' has unknown return type; cast the call to
> its declared return type
> 
> Fix it by giving it what it wants: arch_prctl return type.
> 
> Information on the topic:
>    https://sourceware.org/gdb/onlinedocs/gdb/Calling.html
> 
> Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
> ---
>  scripts/qemugdb/coroutine.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Thanks, applied to my block tree:
https://github.com/stefanha/qemu/commits/block

Stefan

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 455 bytes --]