On Thu, Nov 03, 2016 at 11:17:59AM -0400, Daniel De Graaf wrote: > On 10/28/2016 11:17 AM, Wei Liu wrote: > >The flask policy can be build twice -- one for hypervisor and one for > >tools. > > > >Before this patch, everything is built inside tools/flask/policy > >directory. It is possible to have a race to write to the same output > >file when running parallel builds. > > > >Prepend output file names with FLASK_BUILD_DIR. Hypervisor and tools > >build will set that variable to different directories, so that we can > >be safe from races. > > > >Adjust other bits of the build system as needed. > > > >Signed-off-by: Wei Liu <wei.l...@citrix.com> > > Acked-by: Daniel De Graaf <dgde...@tycho.nsa.gov> >
Thanks. > Pulling the definition of POLICY_FILENAME out of Makefile.common might > remove the need for the cmp||cp line in the xen-side Makefile, but that > probably belongs in another patch. > Yes, I think that's better done with another patch. I will remove the redundant "tmp" in Makefile.common as discussed with Jan and commit the updated patch with your ack. Wei. > -- > Daniel De Graaf > National Security Agency _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel
