[irqbalance] Re: [PATCH 4/5] irqbalance: free list of irqs banned by cmd argument

  • From: Neil Horman <nhorman@xxxxxxxxxxxxx>
  • To: Petr Holasek <pholasek@xxxxxxxxxx>
  • Date: Thu, 19 Mar 2015 08:47:52 -0400

On Thu, Mar 19, 2015 at 11:52:49AM +0100, Petr Holasek wrote:
> On Wed, 18 Mar 2015, Neil Horman <nhorman@xxxxxxxxxxxxx> wrote:
> > On Wed, Mar 18, 2015 at 03:57:57PM +0100, Petr Holasek wrote:
> > > Signed-off-by: Petr Holasek <pholasek@xxxxxxxxxx>
> > > ---
> > >  classify.c | 9 ++++++++-
> > >  1 file changed, 8 insertions(+), 1 deletion(-)
> > > 
> > > diff --git a/classify.c b/classify.c
> > > index cc8293f..5a16dc2 100644
> > > --- a/classify.c
> > > +++ b/classify.c
> > > @@ -658,7 +658,14 @@ void free_irq_db(void)
> > >   rebalance_irq_list = NULL;
> > >  }
> > >  
> > > -static void add_new_irq(int irq, struct irq_info *hint)
> > > +void free_cl_opts(void)
> > 
> > 
> > Where does this get called from?
> > 
> > Neil
> 
> Apologize, it is called from "[PATCH] irqbalance: introduced new --banmod 
> option".
> I've detached the patch from this cleanup series right before sending since 
> it is
> adding a new feature.
> 
Ok, If you can please fixup the dump_balance_obj thing and repost the entire
series please, I'll take it all

Neil
 
> > 
> > > +{
> > > + g_list_free_full(cl_banned_modules, free);
> > > + g_list_free_full(cl_banned_irqs, free);
> > > + g_list_free(banned_irqs);
> > > +}
> > > +
> > > +static void add_new_irq(int irq, struct irq_info *hint, GList 
> > > *proc_interrupts)
> > >  {
> > >   struct irq_info *new;
> > >   struct user_irq_policy pol;
> > > -- 
> > > 2.1.0
> > > 
> > > 
> 
> -- 
> Petr Holasek
> pholasek@xxxxxxxxxx
> 

Other related posts: