[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Xen-devel] [PATCH trivial] xen-hvm: Remove redandant v
From: |
Chen Gang |
Subject: |
Re: [Qemu-devel] [Xen-devel] [PATCH trivial] xen-hvm: Remove redandant varialbe 'xstate' |
Date: |
Tue, 11 Nov 2014 22:32:06 +0800 |
User-agent: |
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 |
On 11/11/14 22:24, Konrad Rzeszutek Wilk wrote:
> On Tue, Nov 11, 2014 at 10:53:01AM +0000, Stefano Stabellini wrote:
>> On Tue, 11 Nov 2014, Chen Gang wrote:
>>> In xen_hvm_change_state_handler(), can pass 'opaque' with type cast to
>>> xen_main_loop_prepare() directly, need not use additional variable for
>>> it.
>
> The title of your patch should say 'variable'.
Oh, yes, thanks. If necessary to send patch v2 for it, please let me
know.
Thanks.
>>>
>>> Signed-off-by: Chen Gang <address@hidden>
>>
>> Acked-by: Stefano Stabellini <address@hidden>
>>
>>
>>> xen-hvm.c | 3 +--
>>> 1 file changed, 1 insertion(+), 2 deletions(-)
>>>
>>> diff --git a/xen-hvm.c b/xen-hvm.c
>>> index 21f1cbb..7548794 100644
>>> --- a/xen-hvm.c
>>> +++ b/xen-hvm.c
>>> @@ -993,9 +993,8 @@ static void xen_main_loop_prepare(XenIOState *state)
>>> static void xen_hvm_change_state_handler(void *opaque, int running,
>>> RunState rstate)
>>> {
>>> - XenIOState *xstate = opaque;
>>> if (running) {
>>> - xen_main_loop_prepare(xstate);
>>> + xen_main_loop_prepare((XenIOState *)opaque);
>>> }
>>> }
>>>
>>> --
>>> 1.8.5.2 (Apple Git-48)
>>>
>>
>> _______________________________________________
>> Xen-devel mailing list
>> address@hidden
>> http://lists.xen.org/xen-devel
--
Chen Gang
Open, share, and attitude like air, water, and life which God blessed