Searched hist:"0 c115681a5194a53a4b373c4ff6d8b42ddd23b55" (Results 1 – 3 of 3) sorted by relevance
/qemu/target/ppc/ |
H A D | kvm_ppc.h | 0c115681a5194a53a4b373c4ff6d8b42ddd23b55 Thu Dec 05 17:46:21 UTC 2019 Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com> ppc: make Error **errp const where it is appropriate
Mostly, Error ** is for returning error from the function, so the callee sets it. However kvmppc_hint_smt_possible gets already filled errp parameter. It doesn't change the pointer itself, only change the internal state of referenced Error object. So we can make it Error *const * errp, to stress the behavior. It will also help coccinelle script (in future) to distinguish such cases from common errp usage.
While there, rename the function to kvmppc_error_append_smt_possible_hint().
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com> Reviewed-by: Greg Kurz <groug@kaod.org> Message-Id: <20191205174635.18758-8-vsementsov@virtuozzo.com> Acked-by: David Gibson <david@gibson.dropbear.id.au> Reviewed-by: Markus Armbruster <armbru@redhat.com> [Commit message replaced] Signed-off-by: Markus Armbruster <armbru@redhat.com>
|
H A D | kvm.c | 0c115681a5194a53a4b373c4ff6d8b42ddd23b55 Thu Dec 05 17:46:21 UTC 2019 Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com> ppc: make Error **errp const where it is appropriate
Mostly, Error ** is for returning error from the function, so the callee sets it. However kvmppc_hint_smt_possible gets already filled errp parameter. It doesn't change the pointer itself, only change the internal state of referenced Error object. So we can make it Error *const * errp, to stress the behavior. It will also help coccinelle script (in future) to distinguish such cases from common errp usage.
While there, rename the function to kvmppc_error_append_smt_possible_hint().
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com> Reviewed-by: Greg Kurz <groug@kaod.org> Message-Id: <20191205174635.18758-8-vsementsov@virtuozzo.com> Acked-by: David Gibson <david@gibson.dropbear.id.au> Reviewed-by: Markus Armbruster <armbru@redhat.com> [Commit message replaced] Signed-off-by: Markus Armbruster <armbru@redhat.com>
|
/qemu/hw/ppc/ |
H A D | spapr.c | 0c115681a5194a53a4b373c4ff6d8b42ddd23b55 Thu Dec 05 17:46:21 UTC 2019 Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com> ppc: make Error **errp const where it is appropriate
Mostly, Error ** is for returning error from the function, so the callee sets it. However kvmppc_hint_smt_possible gets already filled errp parameter. It doesn't change the pointer itself, only change the internal state of referenced Error object. So we can make it Error *const * errp, to stress the behavior. It will also help coccinelle script (in future) to distinguish such cases from common errp usage.
While there, rename the function to kvmppc_error_append_smt_possible_hint().
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com> Reviewed-by: Greg Kurz <groug@kaod.org> Message-Id: <20191205174635.18758-8-vsementsov@virtuozzo.com> Acked-by: David Gibson <david@gibson.dropbear.id.au> Reviewed-by: Markus Armbruster <armbru@redhat.com> [Commit message replaced] Signed-off-by: Markus Armbruster <armbru@redhat.com>
|