On Mon, Jun 29, 2015 at 05:50:36PM +0900, Seiichi Ikarashi wrote:
> irqbalance forgets command line options like banned irqs during rescan.
>
> In main(),
>
> if (need_rescan) {
> :
> free_object_tree(); // forget banned irqs
> build_object_tree(); // all irqs are added into DB ignoring --banirq= options
> :
> }
>
> So let's preserve command line options across rescan.
>
> Signed-off-by: Seiichi Ikarashi <s.ikarashi at jp.fujitsu.com>
>
> ---
> irqbalance.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/irqbalance.c b/irqbalance.c
> index 2674887..c63850b 100644
> --- a/irqbalance.c
> +++ b/irqbalance.c
> @@ -214,7 +214,6 @@ static void free_object_tree(void)
> free_numa_node_list();
> clear_cpu_tree();
> free_irq_db();
> - free_cl_opts();
> }
>
> static void dump_object_tree(void)
> @@ -418,6 +417,7 @@ int main(int argc, char** argv)
>
> }
> free_object_tree();
> + free_cl_opts();
>
> /* Remove pidfile */
> if (!foreground_mode && pidfile)
>
> _______________________________________________
> irqbalance mailing list
> irqbalance at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/irqbalance
>
Applied, thanks!
Neil
