Re: [systemd-devel] [PATCH v2] Add FDB support

2014-12-16 Thread Rauta, Alin
Regards, Alin -Original Message- From: Tom Gundersen [mailto:t...@jklm.no] Sent: Monday, December 15, 2014 5:49 PM To: Rauta, Alin Cc: Lennart Poettering; systemd Mailing List; Kinsella, Ray Subject: Re: [systemd-devel] [PATCH v2] Add FDB support Hi Alin, Thanks for the patch! Overall

Re: [systemd-devel] [PATCH v2] Add FDB support

2014-12-16 Thread Tom Gundersen
On Tue, Dec 16, 2014 at 4:28 PM, Rauta, Alin alin.ra...@intel.com wrote: Regarding the flushing of entries that's done because we don't want to have dangling configuration between system restarts, You mean these settings are remembered by the hardware between reboots? Cheers, Tom

Re: [systemd-devel] [PATCH v2] Add FDB support

2014-12-16 Thread Rauta, Alin
Sorry, between systemd restarts. /Alin -Original Message- From: Tom Gundersen [mailto:t...@jklm.no] Sent: Tuesday, December 16, 2014 3:56 PM To: Rauta, Alin Cc: Lennart Poettering; systemd Mailing List; Kinsella, Ray Subject: Re: [systemd-devel] [PATCH v2] Add FDB support On Tue, Dec

Re: [systemd-devel] [PATCH v2] Add FDB support

2014-12-16 Thread Tom Gundersen
to that. Cheers, Tom -Original Message- From: Tom Gundersen [mailto:t...@jklm.no] Sent: Tuesday, December 16, 2014 3:56 PM To: Rauta, Alin Cc: Lennart Poettering; systemd Mailing List; Kinsella, Ray Subject: Re: [systemd-devel] [PATCH v2] Add FDB support On Tue, Dec 16, 2014 at 4

Re: [systemd-devel] [PATCH v2] Add FDB support

2014-12-15 Thread Tom Gundersen
Hi Alin, Thanks for the patch! Overall it looks excellent, my only concern is with the clearing of FDB entries. Is there any reason to treat this differently than how we currently treat routes and addresses? The current logic is that we don't support run-time reconfiguration (we only apply our