Hey Mike, > > Also, add a comment before page_ext_operations that describes the > > field, and remove check if need() is set, as that is now a required > > Nit: fields OK > > struct pglist_data; > > + > > +/* > > Please make it kerneldoc. OK > > void (*init)(void); > > + bool using_shared_ext_flags; > > I'd s/using_shared_ext_flags/need_shared_flags. Sure > > + * Note: that if none of the users of page_ext is setting > > I'd drop that ^that OK Thanks for the review, I will send a new version soon. Pasha
