optee: allow to work without static shared memory

linaro-swg / linux

Linux kernel source tree

Other

41 stars 79 forks source link

optee: allow to work without static shared memory #63

Closed lorc closed 3 years ago

lorc commented 6 years ago

On virtualized systems it is possible that OP-TEE will provide only dynamic shared memory support. So it is fine to boot without static SHM enabled if dymanic one is supported.

Signed-off-by: Volodymyr Babchuk vlad.babchuk@gmail.com

lorc commented 6 years ago

@jenswi-linaro thanks for review. I restructured function, now it indeed looks cleaner.

jenswi-linaro commented 6 years ago

I suspect that you haven't pushed the result...

lorc commented 6 years ago

Yes, sorry, I have pushed wrong branch :(

Should be okay now.

lorc commented 6 years ago

I decided that is better to leave optee_config_shm_memremap to initialize only static SHM and to add function optee_config_dyn_shm(). optee_probe() can now try to initialize dynamic SHM first (if available) and then fall back to static SHM if there was a problem. I think it is much better in this way.

lorc commented 6 years ago

Updated. Also sent updated patch to ML. Do you want to merge this PR into linaro-swg tree? If not - I can close it now.

lorc commented 3 years ago

Closing as already merged