The optee driver uses specific page table types to verify if a memory
region is normal. These types are not defined in nommu systems. Trying
to compile the driver in these systems results in a build error:
linux/drivers/tee/optee/call.c: In function ‘is_normal_memory’:
linux/drivers/tee/optee/call.c:533:26: error: ‘L_PTE_MT_MASK’ undeclared
(first use in this function); did you mean ‘PREEMPT_MASK’?
return (pgprot_val(p) & L_PTE_MT_MASK) == L_PTE_MT_WRITEALLOC;
^~~~~~~~~~~~~
PREEMPT_MASK
linux/drivers/tee/optee/call.c:533:26: note: each undeclared identifier is
reported only once for each function it appears in
linux/drivers/tee/optee/call.c:533:44: error: ‘L_PTE_MT_WRITEALLOC’ undeclared
(first use in this function)
return (pgprot_val(p) & L_PTE_MT_MASK) == L_PTE_MT_WRITEALLOC;
^~~~~~~~~~~~~~~~~~~
Make the optee driver depend on MMU to fix the compilation issue.
Cc: Jens Wiklander <[email protected]>
Signed-off-by: Vincenzo Frascino <[email protected]>
---
drivers/tee/optee/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/tee/optee/Kconfig b/drivers/tee/optee/Kconfig
index d1ad512e1708..3ca71e3812ed 100644
--- a/drivers/tee/optee/Kconfig
+++ b/drivers/tee/optee/Kconfig
@@ -3,6 +3,7 @@
config OPTEE
tristate "OP-TEE"
depends on HAVE_ARM_SMCCC
+ depends on MMU
help
This implements the OP-TEE Trusted Execution Environment (TEE)
driver.
--
2.24.1
Hi Greg,
I sent the fix below few days ago to the optee maintaners but I did not get any
answer. Could you please pick it up?
On 10/01/2020 12:28, Vincenzo Frascino wrote:
> The optee driver uses specific page table types to verify if a memory
> region is normal. These types are not defined in nommu systems. Trying
> to compile the driver in these systems results in a build error:
>
> linux/drivers/tee/optee/call.c: In function ‘is_normal_memory’:
> linux/drivers/tee/optee/call.c:533:26: error: ‘L_PTE_MT_MASK’ undeclared
> (first use in this function); did you mean ‘PREEMPT_MASK’?
> return (pgprot_val(p) & L_PTE_MT_MASK) == L_PTE_MT_WRITEALLOC;
> ^~~~~~~~~~~~~
> PREEMPT_MASK
> linux/drivers/tee/optee/call.c:533:26: note: each undeclared identifier is
> reported only once for each function it appears in
> linux/drivers/tee/optee/call.c:533:44: error: ‘L_PTE_MT_WRITEALLOC’ undeclared
> (first use in this function)
> return (pgprot_val(p) & L_PTE_MT_MASK) == L_PTE_MT_WRITEALLOC;
> ^~~~~~~~~~~~~~~~~~~
>
> Make the optee driver depend on MMU to fix the compilation issue.
>
> Cc: Jens Wiklander <[email protected]>
> Signed-off-by: Vincenzo Frascino <[email protected]>
> ---
> drivers/tee/optee/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/tee/optee/Kconfig b/drivers/tee/optee/Kconfig
> index d1ad512e1708..3ca71e3812ed 100644
> --- a/drivers/tee/optee/Kconfig
> +++ b/drivers/tee/optee/Kconfig
> @@ -3,6 +3,7 @@
> config OPTEE
> tristate "OP-TEE"
> depends on HAVE_ARM_SMCCC
> + depends on MMU
> help
> This implements the OP-TEE Trusted Execution Environment (TEE)
> driver.
>
--
Regards,
Vincenzo
On Tue, Jan 21, 2020 at 02:23:02PM +0000, Vincenzo Frascino wrote:
> Hi Greg,
>
> I sent the fix below few days ago to the optee maintaners but I did not get any
> answer. Could you please pick it up?
$ ./scripts/get_maintainer.pl --file drivers/tee/optee/Kconfig
Jens Wiklander <[email protected]> (maintainer:OP-TEE DRIVER)
[email protected] (open list:OP-TEE DRIVER)
[email protected] (open list)
This should go through Jens, why me?
greg k-h
Hi Greg,
On 21/01/2020 15:20, Greg Kroah-Hartman wrote:
> On Tue, Jan 21, 2020 at 02:23:02PM +0000, Vincenzo Frascino wrote:
>> Hi Greg,
>>
>> I sent the fix below few days ago to the optee maintaners but I did not get any
>> answer. Could you please pick it up?
>
> $ ./scripts/get_maintainer.pl --file drivers/tee/optee/Kconfig
> Jens Wiklander <[email protected]> (maintainer:OP-TEE DRIVER)
> [email protected] (open list:OP-TEE DRIVER)
> [email protected] (open list)
>
> This should go through Jens, why me?
>
I added Jens and tee-dev list in copy already but as I was mentioning in my
previous email I did not get any answer. I thought that since it is a small fix
you could help. Sorry if I made a mistake.
> greg k-h
>
--
Regards,
Vincenzo
On Tue, Jan 21, 2020 at 03:27:47PM +0000, Vincenzo Frascino wrote:
> Hi Greg,
>
> On 21/01/2020 15:20, Greg Kroah-Hartman wrote:
> > On Tue, Jan 21, 2020 at 02:23:02PM +0000, Vincenzo Frascino wrote:
> >> Hi Greg,
> >>
> >> I sent the fix below few days ago to the optee maintaners but I did not get any
> >> answer. Could you please pick it up?
> >
> > $ ./scripts/get_maintainer.pl --file drivers/tee/optee/Kconfig
> > Jens Wiklander <[email protected]> (maintainer:OP-TEE DRIVER)
> > [email protected] (open list:OP-TEE DRIVER)
> > [email protected] (open list)
> >
> > This should go through Jens, why me?
> >
>
> I added Jens and tee-dev list in copy already but as I was mentioning in my
> previous email I did not get any answer. I thought that since it is a small fix
> you could help. Sorry if I made a mistake.
Give people time to catch up on email, especially for obscure issues
like this.
thanks,
greg k-h
Hi Vincenzo,
On Tue, Jan 21, 2020 at 6:18 PM Greg Kroah-Hartman
<[email protected]> wrote:
>
> On Tue, Jan 21, 2020 at 03:27:47PM +0000, Vincenzo Frascino wrote:
> > Hi Greg,
> >
> > On 21/01/2020 15:20, Greg Kroah-Hartman wrote:
> > > On Tue, Jan 21, 2020 at 02:23:02PM +0000, Vincenzo Frascino wrote:
> > >> Hi Greg,
> > >>
> > >> I sent the fix below few days ago to the optee maintaners but I did not get any
> > >> answer. Could you please pick it up?
> > >
> > > $ ./scripts/get_maintainer.pl --file drivers/tee/optee/Kconfig
> > > Jens Wiklander <[email protected]> (maintainer:OP-TEE DRIVER)
> > > [email protected] (open list:OP-TEE DRIVER)
> > > [email protected] (open list)
> > >
> > > This should go through Jens, why me?
> > >
> >
> > I added Jens and tee-dev list in copy already but as I was mentioning in my
> > previous email I did not get any answer. I thought that since it is a small fix
> > you could help. Sorry if I made a mistake.
>
> Give people time to catch up on email, especially for obscure issues
> like this.
>
> thanks,
>
> greg k-h
I'll pick up this patch.
Thanks,
Jens
Hi Jens,
On 22/01/2020 08:03, Jens Wiklander wrote:
> Hi Vincenzo,
>
> On Tue, Jan 21, 2020 at 6:18 PM Greg Kroah-Hartman
> <[email protected]> wrote:
>>
>> On Tue, Jan 21, 2020 at 03:27:47PM +0000, Vincenzo Frascino wrote:
>>> Hi Greg,
>>>
>>> On 21/01/2020 15:20, Greg Kroah-Hartman wrote:
>>>> On Tue, Jan 21, 2020 at 02:23:02PM +0000, Vincenzo Frascino wrote:
>>>>> Hi Greg,
>>>>>
>>>>> I sent the fix below few days ago to the optee maintaners but I did not get any
>>>>> answer. Could you please pick it up?
>>>>
>>>> $ ./scripts/get_maintainer.pl --file drivers/tee/optee/Kconfig
>>>> Jens Wiklander <[email protected]> (maintainer:OP-TEE DRIVER)
>>>> [email protected] (open list:OP-TEE DRIVER)
>>>> [email protected] (open list)
>>>>
>>>> This should go through Jens, why me?
>>>>
>>>
>>> I added Jens and tee-dev list in copy already but as I was mentioning in my
>>> previous email I did not get any answer. I thought that since it is a small fix
>>> you could help. Sorry if I made a mistake.
>>
>> Give people time to catch up on email, especially for obscure issues
>> like this.
>>
>> thanks,
>>
>> greg k-h
>
> I'll pick up this patch.
>
Thanks for this, since it might break the build in some cases, do you think it
there any chance it can end up in 5.5? I know it might be late.
> Thanks,
> Jens
>
--
Regards,
Vincenzo
Hi Greg,
On 21/01/2020 17:18, Greg Kroah-Hartman wrote:
> On Tue, Jan 21, 2020 at 03:27:47PM +0000, Vincenzo Frascino wrote:
>> Hi Greg,
>>
>> On 21/01/2020 15:20, Greg Kroah-Hartman wrote:
>>> On Tue, Jan 21, 2020 at 02:23:02PM +0000, Vincenzo Frascino wrote:
>>>> Hi Greg,
>>>>
>>>> I sent the fix below few days ago to the optee maintaners but I did not get any
>>>> answer. Could you please pick it up?
>>>
>>> $ ./scripts/get_maintainer.pl --file drivers/tee/optee/Kconfig
>>> Jens Wiklander <[email protected]> (maintainer:OP-TEE DRIVER)
>>> [email protected] (open list:OP-TEE DRIVER)
>>> [email protected] (open list)
>>>
>>> This should go through Jens, why me?
>>>
>>
>> I added Jens and tee-dev list in copy already but as I was mentioning in my
>> previous email I did not get any answer. I thought that since it is a small fix
>> you could help. Sorry if I made a mistake.
>
> Give people time to catch up on email, especially for obscure issues
> like this.
>
This is a good advise. Thanks.
> thanks,
>
> greg k-h
>
--
Regards,
Vincenzo
On Wed, Jan 22, 2020 at 09:37:28AM +0000, Vincenzo Frascino wrote:
> Hi Jens,
>
> On 22/01/2020 08:03, Jens Wiklander wrote:
> > Hi Vincenzo,
> >
> > On Tue, Jan 21, 2020 at 6:18 PM Greg Kroah-Hartman
> > <[email protected]> wrote:
> >>
> >> On Tue, Jan 21, 2020 at 03:27:47PM +0000, Vincenzo Frascino wrote:
> >>> Hi Greg,
> >>>
> >>> On 21/01/2020 15:20, Greg Kroah-Hartman wrote:
> >>>> On Tue, Jan 21, 2020 at 02:23:02PM +0000, Vincenzo Frascino wrote:
> >>>>> Hi Greg,
> >>>>>
> >>>>> I sent the fix below few days ago to the optee maintaners but I did not get any
> >>>>> answer. Could you please pick it up?
> >>>>
> >>>> $ ./scripts/get_maintainer.pl --file drivers/tee/optee/Kconfig
> >>>> Jens Wiklander <[email protected]> (maintainer:OP-TEE DRIVER)
> >>>> [email protected] (open list:OP-TEE DRIVER)
> >>>> [email protected] (open list)
> >>>>
> >>>> This should go through Jens, why me?
> >>>>
> >>>
> >>> I added Jens and tee-dev list in copy already but as I was mentioning in my
> >>> previous email I did not get any answer. I thought that since it is a small fix
> >>> you could help. Sorry if I made a mistake.
> >>
> >> Give people time to catch up on email, especially for obscure issues
> >> like this.
> >>
> >> thanks,
> >>
> >> greg k-h
> >
> > I'll pick up this patch.
> >
>
> Thanks for this, since it might break the build in some cases, do you think it
> there any chance it can end up in 5.5? I know it might be late.
I've just sent a pull request with this commit to arm-soc. It's their
decision if it's passed on for 5.5 or if it's too late.
Cheers,
Jens