Re: [PATCHv4 09/10] mm/usercopy: Switch to using lm_alias

2016-12-07 Thread Mark Rutland
On Tue, Dec 06, 2016 at 12:10:50PM -0800, Kees Cook wrote: > On Tue, Dec 6, 2016 at 10:18 AM, Mark Rutland wrote: > > On Tue, Nov 29, 2016 at 11:39:44AM -0800, Kees Cook wrote: > >> On Tue, Nov 29, 2016 at 10:55 AM, Laura Abbott wrote: > >> > > >> > The

Re: [PATCHv4 09/10] mm/usercopy: Switch to using lm_alias

2016-12-07 Thread Mark Rutland
On Tue, Dec 06, 2016 at 12:10:50PM -0800, Kees Cook wrote: > On Tue, Dec 6, 2016 at 10:18 AM, Mark Rutland wrote: > > On Tue, Nov 29, 2016 at 11:39:44AM -0800, Kees Cook wrote: > >> On Tue, Nov 29, 2016 at 10:55 AM, Laura Abbott wrote: > >> > > >> > The usercopy checking code currently calls

Re: [PATCHv4 09/10] mm/usercopy: Switch to using lm_alias

2016-12-06 Thread Kees Cook
On Tue, Dec 6, 2016 at 10:18 AM, Mark Rutland wrote: > On Tue, Nov 29, 2016 at 11:39:44AM -0800, Kees Cook wrote: >> On Tue, Nov 29, 2016 at 10:55 AM, Laura Abbott wrote: >> > >> > The usercopy checking code currently calls __va(__pa(...)) to check for

Re: [PATCHv4 09/10] mm/usercopy: Switch to using lm_alias

2016-12-06 Thread Kees Cook
On Tue, Dec 6, 2016 at 10:18 AM, Mark Rutland wrote: > On Tue, Nov 29, 2016 at 11:39:44AM -0800, Kees Cook wrote: >> On Tue, Nov 29, 2016 at 10:55 AM, Laura Abbott wrote: >> > >> > The usercopy checking code currently calls __va(__pa(...)) to check for >> > aliases on symbols. Switch to using

Re: [PATCHv4 09/10] mm/usercopy: Switch to using lm_alias

2016-12-06 Thread Mark Rutland
On Tue, Nov 29, 2016 at 10:55:28AM -0800, Laura Abbott wrote: > > The usercopy checking code currently calls __va(__pa(...)) to check for > aliases on symbols. Switch to using lm_alias instead. > > Signed-off-by: Laura Abbott I've given this a go on Juno, which boots

Re: [PATCHv4 09/10] mm/usercopy: Switch to using lm_alias

2016-12-06 Thread Mark Rutland
On Tue, Nov 29, 2016 at 10:55:28AM -0800, Laura Abbott wrote: > > The usercopy checking code currently calls __va(__pa(...)) to check for > aliases on symbols. Switch to using lm_alias instead. > > Signed-off-by: Laura Abbott I've given this a go on Juno, which boots happily. LKDTM triggers as

Re: [PATCHv4 09/10] mm/usercopy: Switch to using lm_alias

2016-12-06 Thread Mark Rutland
On Tue, Nov 29, 2016 at 11:39:44AM -0800, Kees Cook wrote: > On Tue, Nov 29, 2016 at 10:55 AM, Laura Abbott wrote: > > > > The usercopy checking code currently calls __va(__pa(...)) to check for > > aliases on symbols. Switch to using lm_alias instead. > > > > Signed-off-by:

Re: [PATCHv4 09/10] mm/usercopy: Switch to using lm_alias

2016-12-06 Thread Mark Rutland
On Tue, Nov 29, 2016 at 11:39:44AM -0800, Kees Cook wrote: > On Tue, Nov 29, 2016 at 10:55 AM, Laura Abbott wrote: > > > > The usercopy checking code currently calls __va(__pa(...)) to check for > > aliases on symbols. Switch to using lm_alias instead. > > > > Signed-off-by: Laura Abbott > >

Re: [PATCHv4 09/10] mm/usercopy: Switch to using lm_alias

2016-11-29 Thread Kees Cook
On Tue, Nov 29, 2016 at 10:55 AM, Laura Abbott wrote: > > The usercopy checking code currently calls __va(__pa(...)) to check for > aliases on symbols. Switch to using lm_alias instead. > > Signed-off-by: Laura Abbott Acked-by: Kees Cook

Re: [PATCHv4 09/10] mm/usercopy: Switch to using lm_alias

2016-11-29 Thread Kees Cook
On Tue, Nov 29, 2016 at 10:55 AM, Laura Abbott wrote: > > The usercopy checking code currently calls __va(__pa(...)) to check for > aliases on symbols. Switch to using lm_alias instead. > > Signed-off-by: Laura Abbott Acked-by: Kees Cook I should probably add a corresponding alias test to