On Mon, Sep 22, 2003 at 01:45:26PM -0500, Matt Mackall wrote: > +static void zap_completion_queue(void) > +{ > + unsigned long flags; > + struct softnet_data *sd = &get_cpu_var(softnet_data); > + > + if (sd->completion_queue) { > + struct sk_buff *clist; > + > + local_irq_save(flags); > + clist = sd->completion_queue; > + sd->completion_queue = NULL; > + local_irq_save(flags); ^^^^^^^^^^^^^^^^^^^^^ Two consecutive local_irq_save()'s?