On Fri, Mar 13, 2020 at 3:07 PM Stephen Smalley <stephen.smalley.work@xxxxxxxxx> wrote: > > On Fri, Mar 13, 2020 at 7:53 AM Ondrej Mosnacek <omosnace@xxxxxxxxxx> wrote: > > > > On Thu, Feb 27, 2020 at 5:02 PM Ondrej Mosnacek <omosnace@xxxxxxxxxx> wrote: > > > This series contains two small changes (these don't seem to affect > > > performance measurably, but are nonetheless logical) and a patch that > > > changes how the policy optimization "type_map" helper structure is > > > represented, which speeds up the whole process. > > > > > > Ondrej Mosnacek (3): > > > libsepol: skip unnecessary check in build_type_map() > > > libsepol: optimize inner loop in build_type_map() > > > libsepol: speed up policy optimization > > > > > > libsepol/src/optimize.c | 119 +++++++++++++++++++++++++++++++--------- > > > 1 file changed, 94 insertions(+), 25 deletions(-) > > > > > > -- > > > 2.24.1 > > > > I can see this series marked as "Changes Requested" in patchwork - is > > there anything requested other than a test for policy optimization? > > After 692716fc5fd5 ("libsepol/cil: raise default attrs_expand_size to > > 2") the second no longer leads to a different output (with expand size > > >=2). > > I suppose you could move it back to New. I can confirm that it no longer yields a different kernel policy. Tested-by: Stephen Smalley <sds@xxxxxxxxxxxxx>