> > + > > > > It seems same goes for > > > > + mc = kzalloc(sizeof(*mc), GFP_KERNEL); > > + if (!mc) > > + return NULL; > > We would need to set events_reported. IMO, probably worth it to init just this one field rather than use up initialized memory - and I think it's clearer. -- MST