[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 15/15] xen: Rename xen_be_frontend_changed
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 15/15] xen: Rename xen_be_frontend_changed |
Date: |
Tue, 4 Oct 2016 10:08:58 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.3.0 |
On 04/10/2016 10:06, Paolo Bonzini wrote:
>
>
> On 04/10/2016 08:43, Emil Condrea wrote:
>> xen_be_frontend_changed -> xen_fe_frontend_changed
>
> This is not correct. The front-end is implemented in the guest domain,
> while the back-end is implemented in the dom0 or stubdom.
>
> This function processes *in the backed* a notification that the frontend
> state changed, hence the name should be xen_be_frontend_changed.
Sorry, this comment was in the wrong place. It should have referred
only to the hunk in hw/xen/xen_backend.c.
Paolo
> Paolo
>
>> Signed-off-by: Emil Condrea <address@hidden>
>> ---
>> hw/xen/xen_backend.c | 2 +-
>> hw/xen/xen_frontend.c | 4 ++--
>> include/hw/xen/xen_frontend.h | 2 +-
>> 3 files changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/hw/xen/xen_backend.c b/hw/xen/xen_backend.c
>> index 30d3aaa..b79e83e 100644
>> --- a/hw/xen/xen_backend.c
>> +++ b/hw/xen/xen_backend.c
>> @@ -213,7 +213,7 @@ static int xen_be_try_setup(struct XenDevice *xendev)
>> xen_be_set_state(xendev, XenbusStateInitialising);
>>
>> xen_be_backend_changed(xendev, NULL);
>> - xen_be_frontend_changed(xendev, NULL);
>> + xen_fe_frontend_changed(xendev, NULL);
>> return 0;
>> }
>>
>> diff --git a/hw/xen/xen_frontend.c b/hw/xen/xen_frontend.c
>> index 1407f5f..761688b 100644
>> --- a/hw/xen/xen_frontend.c
>> +++ b/hw/xen/xen_frontend.c
>> @@ -39,7 +39,7 @@ int xenstore_read_fe_uint64(struct XenDevice *xendev,
>> const char *node,
>> return xenstore_read_uint64(xendev->fe, node, uval);
>> }
>>
>> -void xen_be_frontend_changed(struct XenDevice *xendev, const char *node)
>> +void xen_fe_frontend_changed(struct XenDevice *xendev, const char *node)
>> {
>> int fe_state;
>>
>> @@ -85,6 +85,6 @@ void xenstore_update_fe(char *watch, struct XenDevice
>> *xendev)
>> }
>> node = watch + len + 1;
>>
>> - xen_be_frontend_changed(xendev, node);
>> + xen_fe_frontend_changed(xendev, node);
>> xen_be_check_state(xendev);
>> }
>> diff --git a/include/hw/xen/xen_frontend.h b/include/hw/xen/xen_frontend.h
>> index bb0bc23..2a5f03f 100644
>> --- a/include/hw/xen/xen_frontend.h
>> +++ b/include/hw/xen/xen_frontend.h
>> @@ -9,6 +9,6 @@ int xenstore_read_fe_uint64(struct XenDevice *xendev, const
>> char *node,
>> uint64_t *uval);
>> void xenstore_update_fe(char *watch, struct XenDevice *xendev);
>>
>> -void xen_be_frontend_changed(struct XenDevice *xendev, const char *node);
>> +void xen_fe_frontend_changed(struct XenDevice *xendev, const char *node);
>>
>> #endif /* QEMU_HW_XEN_FRONTEND_H */
>>
>
> _______________________________________________
> Xen-devel mailing list
> address@hidden
> https://lists.xen.org/xen-devel
>
- Re: [Qemu-devel] [PATCH 11/15] xen: Rename xen_be_send_notify, (continued)
- [Qemu-devel] [PATCH 13/15] xen: Rename xen_be_find_xendev, Emil Condrea, 2016/10/04
- [Qemu-devel] [PATCH 12/15] xen: Rename xen_be_evtchn_event, Emil Condrea, 2016/10/04
- [Qemu-devel] [PATCH 15/15] xen: Rename xen_be_frontend_changed, Emil Condrea, 2016/10/04
- Re: [Qemu-devel] [PATCH 15/15] xen: Rename xen_be_frontend_changed, Paolo Bonzini, 2016/10/04
- Re: [Qemu-devel] [PATCH 15/15] xen: Rename xen_be_frontend_changed,
Paolo Bonzini <=
- Re: [Qemu-devel] [PATCH 15/15] xen: Rename xen_be_frontend_changed, Emil Condrea, 2016/10/09
- Re: [Qemu-devel] [PATCH 15/15] xen: Rename xen_be_frontend_changed, Paolo Bonzini, 2016/10/12
- Re: [Qemu-devel] [PATCH 15/15] xen: Rename xen_be_frontend_changed, Emil Condrea, 2016/10/13
- Re: [Qemu-devel] [PATCH 15/15] xen: Rename xen_be_frontend_changed, Xuquan (Quan Xu), 2016/10/13
- Re: [Qemu-devel] [PATCH 15/15] xen: Rename xen_be_frontend_changed, Paolo Bonzini, 2016/10/13
[Qemu-devel] [PATCH 14/15] xen: Rename xen_be_del_xendev, Emil Condrea, 2016/10/04
[Qemu-devel] [PATCH 09/15] xen: Rename xen_be_printf to xen_pv_printf, Emil Condrea, 2016/10/04