> I think it should do:
> 
> if (status & 1)
>         dse.mem_lvl = PERF_MEM_LVL_L1|PERF_MEM_LVL_HIT;
> else
>         dse.mem_lvl = PERF_MEM_LVL_L1|PERF_MEM_LVL_MISS;
> 
> Otherwise you have L1 as the level with no hit/miss info.

Agreed.

BTW the line before is also not always corect, and any event not
explicitely store can only fill in NA


-Andi
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to