On Thu, Apr 10, 2008 at 8:01 AM, Pavel Emelyanov <xemul@xxxxxxxxxx> wrote: > +int register_pernet_gen_device(int *id, struct pernet_operations *ops) > +{ > + int error; > + mutex_lock(&net_mutex); > +again: > + error = ida_get_new_above(&net_generic_ids, 1, id); > + if (error) { > + if (error == -EAGAIN) { > + ida_pre_get(&net_generic_ids, GFP_KERNEL); > + goto again; > + } Shouldn't you handle non-EAGAIN errors here? Paul _______________________________________________ Containers mailing list Containers@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/containers