Hi Peter,

I have attached the initial test logs from your latest patchset. 

The perf command does not crash when /sys/devices/cpu/allow_tsx_force_abort is set to 0.
 
Thanks,
Nelson

On 2/19/19 8:10 AM, speck for Peter Zijlstra wrote:
> On Tue, Feb 19, 2019 at 04:58:11PM +0100, speck for Peter Zijlstra wrote:
>> +	if (!allow_tsx_force_abort && test_bit(3, c->idxmsk)) {
>> +		c = dyn_constraint(cpuc, c, idx);
>> +		c->idxmsk64 &= ~(1ULL << 3);
>> +		c->weight = c->idxmsk64 - 1;
> 
> Damn; that should be:
> 
> 		c->weight--;
>