[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: |
Emil Condrea |
Subject: |
Re: [Qemu-devel] [PATCH 15/15] xen: Rename xen_be_frontend_changed |
Date: |
Thu, 13 Oct 2016 09:08:54 +0300 |
As you suggested, I've dropped the all patches for xen_frontend.
Emil
On Wed, Oct 12, 2016 at 2:00 PM, Paolo Bonzini <address@hidden> wrote:
>
>
> On 09/10/2016 21:50, Emil Condrea wrote:
>> On Tue, Oct 4, 2016 at 11:06 AM, Paolo Bonzini <address@hidden> 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.
>>>
>>
>> You are right, thanks for the feedback! I will drop this patch
>> together with the hunk
>> from 04/15 patch which moves this function to xen_frontend.c
>
> Actually all of your new xen_frontend.c seems to be reading frontend
> information from XenStore, which is typically something that the backend
> does. So I suggest dropping the patch altogether.
>
> Thanks,
>
> Paolo
>
>>> This function processes *in the backed* a notification that the frontend
>>> state changed, hence the name should be xen_be_frontend_changed.
>>>
>>> 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 */
>>>>
>>
>>
- Re: [Qemu-devel] [PATCH 13/15] xen: Rename xen_be_find_xendev, (continued)
- [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, 2016/10/04
- 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 <=
- 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