Re: [libvirt] [PATCH 1/2] networkUpdateState: Create virMacMap module more frequently

2017-03-31 Thread Michal Privoznik
On 03/31/2017 03:41 PM, Laine Stump wrote: On 03/31/2017 07:20 AM, Michal Privoznik wrote: On 03/30/2017 06:00 PM, Laine Stump wrote: On 03/28/2017 05:08 AM, Michal Privoznik wrote: On 03/28/2017 02:22 AM, Laine Stump wrote: On 03/22/2017 10:43 AM, Michal Privoznik wrote:

Re: [libvirt] [PATCH 1/2] networkUpdateState: Create virMacMap module more frequently

2017-03-31 Thread Laine Stump
On 03/31/2017 07:20 AM, Michal Privoznik wrote: > On 03/30/2017 06:00 PM, Laine Stump wrote: >> On 03/28/2017 05:08 AM, Michal Privoznik wrote: >>> On 03/28/2017 02:22 AM, Laine Stump wrote: On 03/22/2017 10:43 AM, Michal Privoznik wrote: >>> > > Signed-off-by: Michal Privoznik

Re: [libvirt] [PATCH 1/2] networkUpdateState: Create virMacMap module more frequently

2017-03-31 Thread Michal Privoznik
On 03/30/2017 06:00 PM, Laine Stump wrote: On 03/28/2017 05:08 AM, Michal Privoznik wrote: On 03/28/2017 02:22 AM, Laine Stump wrote: On 03/22/2017 10:43 AM, Michal Privoznik wrote: Signed-off-by: Michal Privoznik --- src/network/bridge_driver.c | 21

Re: [libvirt] [PATCH 1/2] networkUpdateState: Create virMacMap module more frequently

2017-03-30 Thread Laine Stump
On 03/28/2017 05:08 AM, Michal Privoznik wrote: > On 03/28/2017 02:22 AM, Laine Stump wrote: >> On 03/22/2017 10:43 AM, Michal Privoznik wrote: > >> >>> >>> Signed-off-by: Michal Privoznik >>> --- >>> src/network/bridge_driver.c | 21 + >>> 1 file

Re: [libvirt] [PATCH 1/2] networkUpdateState: Create virMacMap module more frequently

2017-03-29 Thread Michal Privoznik
On 03/28/2017 10:42 PM, Martin Kletzander wrote: > Makes sense to me, however you might want to look at an idea in the second patch. Maybe it would help a bit. Maybe it wouldn't :-/ I feel like the consensus is either to have a pointer to an object (in which case it's enabled) or not

Re: [libvirt] [PATCH 1/2] networkUpdateState: Create virMacMap module more frequently

2017-03-28 Thread Martin Kletzander
On Tue, Mar 28, 2017 at 11:08:01AM +0200, Michal Privoznik wrote: On 03/28/2017 02:22 AM, Laine Stump wrote: On 03/22/2017 10:43 AM, Michal Privoznik wrote: The virMacMap module is there for dumping [domain, ] pairs into a file so that libvirt_guest NSS module can use it. Whenever a interface

Re: [libvirt] [PATCH 1/2] networkUpdateState: Create virMacMap module more frequently

2017-03-28 Thread Michal Privoznik
On 03/28/2017 02:22 AM, Laine Stump wrote: On 03/22/2017 10:43 AM, Michal Privoznik wrote: Signed-off-by: Michal Privoznik --- src/network/bridge_driver.c | 21 + 1 file changed, 9 insertions(+), 12 deletions(-) diff --git

Re: [libvirt] [PATCH 1/2] networkUpdateState: Create virMacMap module more frequently

2017-03-27 Thread Laine Stump
On 03/22/2017 10:43 AM, Michal Privoznik wrote: > The virMacMap module is there for dumping [domain, MACs>] pairs into a file so that libvirt_guest NSS module can use > it. Whenever a interface is allocated from network (e.g. on > domani startup or NIC hotplug), network is notified and so is

[libvirt] [PATCH 1/2] networkUpdateState: Create virMacMap module more frequently

2017-03-22 Thread Michal Privoznik
The virMacMap module is there for dumping [domain, ] pairs into a file so that libvirt_guest NSS module can use it. Whenever a interface is allocated from network (e.g. on domani startup or NIC hotplug), network is notified and so is virMacMap module subsequently. The module update functions