[PATCH 2/5] powerpc/mm: split store_updates_sp() in two parts in do_page_fault()

Benjamin Herrenschmidt benh at kernel.crashing.org
Fri Jun 2 22:11:13 AEST 2017


On Fri, 2017-06-02 at 11:39 +0200, Christophe LEROY wrote:
> The difference between get_user() and __get_user() is that get_user() 
> performs an access_ok() in addition.
> 
> Doesn't access_ok() only checks whether addr is below TASK_SIZE to 
> ensure it is a valid user address ?

Do you have a measurable improvement by skipping that check ? I agree
with your reasoning but I'm also paranoid and so I wouldn't change it
unless it's really worth it.

Cheers,
Ben.



More information about the Linuxppc-dev mailing list