> > Just one minor question why not make following functions > > EXPORT_SYMBOL_GPL() ? > > > > 1. iov_iter_get_pages2() > > 2. iov_iter_get_pages_alloc2() > > They previously were not GPL, so I didn't think that should be changed > in this patch. Yes. While they should have been _GPL from the start, rocking that boat is a bit pointless now. We just need to make sure to do the right thing for the pinning variants that are going to replace them soon.