On 17 June 2014 12:18, Vishal Bhoj <vishal.bhoj@linaro.org> wrote:

On 17 June 2014 16:41, Fathi Boudra <fathi.boudra@linaro.org> wrote:

Vishal,

1. against which tree should it be applied?
I have tested it against TC2 with the LSK tree. I was not sure if the patches directly go to LSK. I thought it should first go into config fragment tree.

I don't know what the "config fragment tree" is but the LSK doesn't use it and you've not CCed - I guess you should send this to Andrey too for inclusion in linux-linaro? I've added him.

2. do you want it enabled by default for all android builds?

Moving ahead SELinux is mandatory for Android so it needs to be enabled across all the platforms we care for. Linux-linaro and LSK are validated on VExpress platform so I am enabling the userspace bits for VExpress to start with.

OK, I'll a change to the LSK which enables these options - I can't see any reason to restrict them to Android so I won't just apply this as-is. Please include information like the above in the changelogs for patches, this is just generally good practice and will help people figure out why the change is being made and if it is sensible to apply it.