[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 05/15] qio: refactor net listener source oper
From: |
Peter Xu |
Subject: |
Re: [Qemu-devel] [PATCH v2 05/15] qio: refactor net listener source operations |
Date: |
Mon, 5 Mar 2018 13:34:42 +0800 |
User-agent: |
Mutt/1.9.1 (2017-09-22) |
On Fri, Mar 02, 2018 at 10:51:01AM +0000, Daniel P. Berrangé wrote:
> On Fri, Mar 02, 2018 at 11:58:52AM +0800, Peter Xu wrote:
> > On Thu, Mar 01, 2018 at 10:47:17AM +0000, Daniel P. Berrangé wrote:
> > > On Thu, Mar 01, 2018 at 04:44:28PM +0800, Peter Xu wrote:
> > > > Three functions are abstracted from the old code:
> > > >
> > > > - qio_net_listener_source_add(): create one source for listener
> > > > - qio_net_listener_sources_clear(): unset existing net lister sources
> > > > - qio_net_listener_sources_update(): setup all sources for listener
> > > >
> > > > Use them where possible.
> > > >
> > > > Signed-off-by: Peter Xu <address@hidden>
> > > > ---
> > > > io/net-listener.c | 82
> > > > +++++++++++++++++++++++++++----------------------------
> > > > 1 file changed, 41 insertions(+), 41 deletions(-)
> > >
> > > This patch can be dropped since nothing else in the series now
> > > depends on it.
> >
> > Do you think it's still acceptable even as a cleanup? Thanks,
>
> I don't think it really adds value over what's there already.
I'll drop it. Thanks.
--
Peter Xu
- Re: [Qemu-devel] [PATCH v2 03/15] qio: introduce qio_channel_add_watch_{full|source}, (continued)
[Qemu-devel] [PATCH v2 05/15] qio: refactor net listener source operations, Peter Xu, 2018/03/01
[Qemu-devel] [PATCH v2 06/15] qio: store gsources for net listeners, Peter Xu, 2018/03/01
[Qemu-devel] [PATCH v2 07/15] qio/chardev: update net listener gcontext, Peter Xu, 2018/03/01