Tejun Heo <tj@xxxxxxxxxx> writes: > Hey, Jeff. > > On Mon, Jun 08, 2015 at 03:49:56PM -0400, Jeff Moyer wrote: >> Assuming you're ok with dropping patch 5, I'll review patches 6-8 once > > I'll update it to use GPF_NOWAIT. Can't drop it easily as that would > complicate group lookup more complicated w/ pinning and all. > >> they've been reworked to account for that. I took a look at them, and >> they look OK to me. But, if they are going to change, I'd rather wait >> to ack the final versions. > > Hmm... the changes will be simple s/GFP_ATOMIC/GFP_NOWAIT. Shouldn't > affect reviewing, I think. Yeah, I sent this email before realizing that GFP_NOWAIT would work out. I've reviewed the rest of the series. Looks good to me! You did a nice job splitting things up into easily reviewable pieces, so thanks for that. Cheers, Jeff -- To unsubscribe from this list: send the line "unsubscribe cgroups" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
