On Mon, Jul 13, 2020 at 8:28 PM Casey Schaufler <casey(a)schaufler-ca.com> wrote:
 On 7/13/2020 5:11 PM, Paul Moore wrote:
 > On Mon, Jul 13, 2020 at 7:09 PM Casey Schaufler <casey(a)schaufler-ca.com>
wrote:
 >> ... but it does appear that I could switch to using your audit_alloc_local().
 > In my opinion, linking the audit container ID and LSM stacking
 > patchsets would seem like a very big mistake, especially since the
 > consolidation you are describing could be done after the fact without
 > any disruption to the kernel/userspace interface.  I would strongly
 > encourage both patchsets to remain self-contained if at all possible
 > so as to not jeopardize each other.
 Whatever helps the review/ack process best works for me. I will leave
 audit_stamp_context() as is unless there is other feedback that leads
 it to be changed. 
Thanks, I think that is the best path forward for both patchsets.
If/when both patchsets land in the kernel I'm happy to revisit this.
-- 
paul moore
www.paul-moore.com