RE: [PATCH V2 0/3] Use mm_struct and switch_mm() instead of manually

> So, in summary it seems that the primary kernel boot _fails_ with your > v2 patchset on the real hardware for me irrespective of whether I use Matt's > tree or Linus's tree: > > a) I would suggest that you perform some more checks on real hardware as > qemu boot tests sometimes do not expose

RE: [PATCH V2 0/3] Use mm_struct and switch_mm() instead of manually

> So, in summary it seems that the primary kernel boot _fails_ with your > v2 patchset on the real hardware for me irrespective of whether I use Matt's > tree or Linus's tree: > > a) I would suggest that you perform some more checks on real hardware as > qemu boot tests sometimes do not expose

Re: [PATCH V2 0/3] Use mm_struct and switch_mm() instead of manually

lt;m...@codeblueprint.co.uk>; Ard Biesheuvel <ard.biesheu...@linaro.org>; >> j...@suse.com; Borislav Petkov <b...@alien8.de>; Luck, Tony >> <tony.l...@intel.com>; l...@kernel.org; m...@redhat.com; Neri, Ricardo >> <ricardo.n...@intel.com>; Shankar,

Re: [PATCH V2 0/3] Use mm_struct and switch_mm() instead of manually

Tony >> ; l...@kernel.org; m...@redhat.com; Neri, Ricardo >> ; Shankar, Ravi V >> Subject: Re: [PATCH V2 0/3] Use mm_struct and switch_mm() instead of >> manually >> >> On Tue, 2017-09-05 at 19:21 -0700, Sai Praneeth Prakhya wrote: >> > > I get

RE: [PATCH V2 0/3] Use mm_struct and switch_mm() instead of manually

.@codeblueprint.co.uk>; Ard Biesheuvel <ard.biesheu...@linaro.org>; > j...@suse.com; Borislav Petkov <b...@alien8.de>; Luck, Tony > <tony.l...@intel.com>; l...@kernel.org; m...@redhat.com; Neri, Ricardo > <ricardo.n...@intel.com>; Shankar, Ravi V <rav

RE: [PATCH V2 0/3] Use mm_struct and switch_mm() instead of manually

.com; Borislav Petkov ; Luck, Tony > ; l...@kernel.org; m...@redhat.com; Neri, Ricardo > ; Shankar, Ravi V > Subject: Re: [PATCH V2 0/3] Use mm_struct and switch_mm() instead of > manually > > On Tue, 2017-09-05 at 19:21 -0700, Sai Praneeth Prakhya wrote: > > > I get a

Re: [PATCH V2 0/3] Use mm_struct and switch_mm() instead of manually

On Tue, 2017-09-05 at 19:21 -0700, Sai Praneeth Prakhya wrote: > > I get a similar crash on Qemu with linus's master branch and the V2 > > applied on top of it. Here are the details of my test environment: > > > > 1. I use the OVMF (EDK2) EFI firmware to launch the kernel: > > edk2.git/ovmf-x64 >

Re: [PATCH V2 0/3] Use mm_struct and switch_mm() instead of manually

On Tue, 2017-09-05 at 19:21 -0700, Sai Praneeth Prakhya wrote: > > I get a similar crash on Qemu with linus's master branch and the V2 > > applied on top of it. Here are the details of my test environment: > > > > 1. I use the OVMF (EDK2) EFI firmware to launch the kernel: > > edk2.git/ovmf-x64 >

Re: [PATCH V2 0/3] Use mm_struct and switch_mm() instead of manually

> I get a similar crash on Qemu with linus's master branch and the V2 > applied on top of it. Here are the details of my test environment: > > 1. I use the OVMF (EDK2) EFI firmware to launch the kernel: > edk2.git/ovmf-x64 > > 2. I used linus's master branch (HEAD - commit: >

Re: [PATCH V2 0/3] Use mm_struct and switch_mm() instead of manually

> I get a similar crash on Qemu with linus's master branch and the V2 > applied on top of it. Here are the details of my test environment: > > 1. I use the OVMF (EDK2) EFI firmware to launch the kernel: > edk2.git/ovmf-x64 > > 2. I used linus's master branch (HEAD - commit: >

Re: [PATCH V2 0/3] Use mm_struct and switch_mm() instead of manually

Hi Sai, On Sun, Sep 3, 2017 at 1:16 PM, Prakhya, Sai Praneeth wrote: >> > >> > Thanks for this v2. >> > Introducing the 'efi_switch_mm() ' helper instead of manually >> > twiddling with %cr3 seems more cleaner. >> > >> > I have tested this patchset on a x86_64

Re: [PATCH V2 0/3] Use mm_struct and switch_mm() instead of manually

Hi Sai, On Sun, Sep 3, 2017 at 1:16 PM, Prakhya, Sai Praneeth wrote: >> > >> > Thanks for this v2. >> > Introducing the 'efi_switch_mm() ' helper instead of manually >> > twiddling with %cr3 seems more cleaner. >> > >> > I have tested this patchset on a x86_64 machine with the following >> >

RE: [PATCH V2 0/3] Use mm_struct and switch_mm() instead of manually

> > > > Thanks for this v2. > > Introducing the 'efi_switch_mm() ' helper instead of manually > > twiddling with %cr3 seems more cleaner. > > > > I have tested this patchset on a x86_64 machine with the following > > configurations: > > > > 1. Primary kernel boot with efi=old_map 2. Primary kernel

RE: [PATCH V2 0/3] Use mm_struct and switch_mm() instead of manually

> > > > Thanks for this v2. > > Introducing the 'efi_switch_mm() ' helper instead of manually > > twiddling with %cr3 seems more cleaner. > > > > I have tested this patchset on a x86_64 machine with the following > > configurations: > > > > 1. Primary kernel boot with efi=old_map 2. Primary kernel

Re: [PATCH V2 0/3] Use mm_struct and switch_mm() instead of manually

On Sat, Sep 2, 2017 at 7:38 PM, Bhupesh Sharma wrote: > Hi Sai, > > On Tue, Aug 29, 2017 at 5:07 AM, Sai Praneeth Prakhya > wrote: >> From: Sai Praneeth >> >> Presently, in x86, to invoke any efi function like

Re: [PATCH V2 0/3] Use mm_struct and switch_mm() instead of manually

On Sat, Sep 2, 2017 at 7:38 PM, Bhupesh Sharma wrote: > Hi Sai, > > On Tue, Aug 29, 2017 at 5:07 AM, Sai Praneeth Prakhya > wrote: >> From: Sai Praneeth >> >> Presently, in x86, to invoke any efi function like >> efi_set_virtual_address_map() or any efi_runtime_service() the code path >>

Re: [PATCH V2 0/3] Use mm_struct and switch_mm() instead of manually

Hi Sai, On Tue, Aug 29, 2017 at 5:07 AM, Sai Praneeth Prakhya wrote: > From: Sai Praneeth > > Presently, in x86, to invoke any efi function like > efi_set_virtual_address_map() or any efi_runtime_service() the code path > typically

Re: [PATCH V2 0/3] Use mm_struct and switch_mm() instead of manually

Hi Sai, On Tue, Aug 29, 2017 at 5:07 AM, Sai Praneeth Prakhya wrote: > From: Sai Praneeth > > Presently, in x86, to invoke any efi function like > efi_set_virtual_address_map() or any efi_runtime_service() the code path > typically involves read_cr3() (save previous pgd), write_cr3() > (write