Re: [Tails-dev] Please review'n'merge test/5959-antitest-mem…

Borrar esta mensaxe

Responder a esta mensaxe
Autor: anonym
Data:  
Para: The Tails public development discussion list
Asunto: Re: [Tails-dev] Please review'n'merge test/5959-antitest-memory-erasure
01/03/14 19:58, intrigeri wrote:
> Hi,
>
> anonym wrote (28 Feb 2014 14:44:00 GMT) :
>> 28/12/13 23:50, intrigeri wrote:
>> For whatever reason, the TailsToaster (i.e. `@domain_name`) domain isn't
>> present for me (also checked with `virsh list`), [...]
>
> Fixed by commit 0caac4a2, thanks.


Ah, that fixes it.

>> Then I have two probably unrelated problems:
>
>> 1. I consistently get:
>
>>>     And at least 8 GiB of RAM was detected #
>>> features/step_definitions/erase_memory.rb:46
>>>       Detected 8393662464 bytes of RAM
>>>       Didn't detect enough RAM (RuntimeError)
>>>       ./features/step_definitions/erase_memory.rb:53:in `/^at least (\d+)
>>> ([[:alpha:]]+) of RAM was detected$/'
>>>       features/erase_memory.feature:16:in `And at least 8 GiB of RAM was detected'

>
>> We currently allow a 128 MiB "gap", but now the gap is 187 MiB. I
>> suppose the switch from a i686-pae to a amd64 kernel is the cause. Can
>> you reproduce this?
>
> I can. Applied your fix in 896fdaa.
>
>> 2. I consistently get:
>
>>>   Scenario: Memory erasure on an old computer # features/erase_memory.feature:55
>> [...]
>>>     Then I find very few patterns in the guest's memory           # features/step_definitions/erase_memory.rb:141
>>>       Pattern coverage: 0.164% (6097552 bytes)
>>>       0.164% of the memory is filled with the pattern, but less than 0.100% was expected (RuntimeError)
>>>       ./features/step_definitions/erase_memory.rb:145:in `/^I find very few patterns in the guest's memory$/'
>>>       features/erase_memory.feature:69:in `Then I find very few patterns in the guest's memory'

>
>> The exact pattern coverage of course varies but it's always been
>> slightly above 0.1%. We currently only allow 0.001 = 0.1% coverage after
>> memory erasure, so we may want to bump it to 0.5%, which still I guess
>> is pretty ok (I guess? It feels like we're completely arbitrary here :)).
>
> I've raised it to 0.25% instead (f5558ad), so that we can detect
> a bump from 0.15% to 0.49% in the future.
>
> So I think you should now be happy to merge this branch :)


Agreed. Merged!

>>> Then, only #6483 will remain a blocker to drop this part of our manual
>>> test suite, eventually.
>
>> Excellent! Since we now use an amd64 kernel we can close that ticket
>> once 0.23 is out.
>
> I've closed it right away, since we're going to use the devel branch
> to run the automated test suite when releasing 0.23.


Good news indeed!

Cheers!