Open-Source Projects: Issueshttps://forge.ispras.ru/https://forge.ispras.ru/favicon.ico?16490126692020-03-02T08:28:05ZOpen-Source Projects
Redmine Retrascope - Task #10139 (Closed): fix coding issues at *BenchTest classeshttps://forge.ispras.ru/issues/101392020-03-02T08:28:05ZSergey Smolovsmolov@ispras.ru
<p>The following issues should be fixed at "*BenchTest" test classes:</p>
<ol>
<li>Substitute copy-pasted Javadoc header comments by relevant ones</li>
<li><strong>ru.ispras.retrascope.engine.cfg.extractor.iface.VhdlCfgExtractorIfaceBenchTest</strong>: parameter <code>getStatistics</code> is never used at <code>runTest</code> method - refactoring is needed</li>
<li>There is a branch order for "if" operator that is used in the project. Following the rule, this form:<br /><pre><code class="java syntaxhl" data-language="java"><span class="k">if</span> <span class="o">(</span><span class="n">cond</span><span class="o">)</span> <span class="o">{</span>
<span class="o">...</span>
<span class="o">}</span> <span class="k">else</span> <span class="o">{</span>
<span class="o">..</span>
<span class="o">}</span>
</code></pre><br />is better than:<br /><pre><code class="java syntaxhl" data-language="java"><span class="k">if</span> <span class="o">(!</span><span class="n">cond</span><span class="o">)</span> <span class="o">{</span>
<span class="o">...</span>
<span class="o">}</span> <span class="k">else</span> <span class="o">{</span>
<span class="o">..</span>
<span class="o">}</span>
</code></pre><br />Take a look at "*BenchTest" classes to fix this rule's violations.</li>
<li>Fix "Local variable 'argPaths' is redundant" warning that appears while opening <strong>ru.ispras.retrascope.engine.cfg.transformer.gadd.VerilogCfgGaddBenchTest</strong> class in IntelliJ IDEA. This warning also appears at other "*BenchTest" classes too. Other warnings that are produced by IntelliJ IDEA are also worth to be fixed.</li>
<li>Having expressions like "!x.endsWith(y) && !x.endsWith(z) && !x.endsWith(w) && ..." you should rewrite them with single "!x.matches(..)" expression. Take a look at regular expressions in Java.</li>
</ol> Retrascope - Task #10073 (Closed): fix checkstyle warningshttps://forge.ispras.ru/issues/100732020-01-27T12:01:41ZSergey Smolovsmolov@ispras.ru
<pre>
$ ./gradlew checkstyleMain checkstyleTest
Starting a Gradle Daemon (subsequent builds will be faster)
> Task :checkstyleTest
[ant:checkstyle] retrascope/src/test/java/ru/ispras/retrascope/engine/cfg/printer/c/VhdlCfgPrinterCTestCase.java:14: warning: 'package' should be separated from previous statement.
[ant:checkstyle] retrascope/src/test/java/ru/ispras/retrascope/engine/cfg/printer/c/VhdlPrinterCBenchTest.java:14: warning: 'package' should be separated from previous statement.
[ant:checkstyle] retrascope/src/test/java/ru/ispras/retrascope/engine/hldd/printer/smv/formula/sample/HlddSmvFormulaPrinterVerilogTest.java:36: warning: Line is longer than 100 characters (found 104).
[ant:checkstyle] retrascope/src/test/java/ru/ispras/retrascope/parser/verilog/VerilogParserTestCase.java:19: warning: Wrong lexicographical order for 'junitparams.JUnitParamsRunner' import. Should be before 'org.junit.runner.RunWith'.
</pre> Verilog Translator - Bug #9915 (Closed): "Cycle inclusion has been detected in fine <filename>" e...https://forge.ispras.ru/issues/99152019-11-13T12:01:33ZSergey Smolovsmolov@ispras.ru
<p>The tool reports "Cycle inclusion has been detected in fine <filename>" error for the case when "a.v" and "b.v" modules include "c.v" module.</p>
<p>To reproduce the bug, checkout to <a class="changeset" title="hdl-benchmark submodule update Signed-off-by: chudnovmaxim <chudnov@ispras.ru>" href="https://forge.ispras.ru/projects/veritrans/repository/veritrans/revisions/5ca788cdbc460bf393ccdef4b9cd6451f71acdd0">5ca788cd</a> commit and run <strong>ru.ispras.verilog.parser.VerilogQuipTestCase</strong>. It should be fail-free, but it is not.</p>
<p>IMPORTANT: please run all the project tests before push and compare your results with Jenkins!</p> Retrascope - Task #9911 (Closed): merge "*/sample/*TestCase" Java test cases https://forge.ispras.ru/issues/99112019-11-12T08:52:05ZSergey Smolovsmolov@ispras.ru
<p>There are separate "*/sample/*TestCase" Java classes in the project. They contain duplicated code and should be merged the same way as benchmark-related test case collections at Verilog Translator project. See <strong>ru.ispras.verilog.parser.VerilogQuipTestCase</strong> for example.</p> Verilog Translator - Task #9811 (Closed): macro with parametershttps://forge.ispras.ru/issues/98112019-09-05T08:39:23ZSergey Smolovsmolov@ispras.ru
<p>Macro with parameters are used, for example, in RISC-V cores like picorv32 (see Retrascope for RISC-V).</p> Retrascope - Task #9766 (Closed): remove 'vhdl.record' Git branch from remote repohttps://forge.ispras.ru/issues/97662019-07-23T11:47:21ZSergey Smolovsmolov@ispras.ru
<p>The branch is merged into master so it should be removed from the repository.</p> Retrascope IDE - Task #9764 (New): migrate to Eclipse 2019https://forge.ispras.ru/issues/97642019-07-22T13:37:19ZSergey Smolovsmolov@ispras.ruRetrascope - Task #9762 (Closed): prepare to 1.1.1 releasehttps://forge.ispras.ru/issues/97622019-07-20T18:10:43ZSergey Smolovsmolov@ispras.ruRetrascope Test Suite - Task #9670 (New): add 'ar.v' module to the test suite when SVA support wi...https://forge.ispras.ru/issues/96702019-05-22T11:24:19ZSergey Smolovsmolov@ispras.ru
<p>See <strong>Ver2SmvBenchmarks</strong> test class.</p> Verilog Translator - Task #9311 (Closed): type casting of expression operandshttps://forge.ispras.ru/issues/93112018-10-05T15:09:31ZSergey Smolovsmolov@ispras.ru
<p>It is a common case for expressions in Verilog designs to have operands that become wrong via transforming to SMT\SMV format. Here are some examples.</p>
<p>1. In <em>vcegar-tests/mpeg/mpeg_1.v</em>:<br /><pre>
reg [15:0] NumBytes;
wire [3:0] timeStampBytes;
...
NumBytes = NumBytes - timeStampBytes;
</pre><br />Operands of "-" have different data types, so SMT solver reports an error.</p>
<p>2. In <em>verilog2smv-vis-tests/Vsa16/vsa16a.v</em>:<br /><pre>
output [15:0] ALUOutput;
reg [11:0] NPC;
wire [15:0] Imm = {{8{immFld[8]}},immFld[7:0]};
ALUOutput <= {4'd0,NPC} + {Imm,1'b0};
</pre><br />This expression produces the following error in nuXmv model checker:<br /><pre><code class="text syntaxhl" data-language="text">TYPE ERROR file vsa16a.smv: line 593 : illegal operand types of "+" : unsigned word[16] and unsigned word[17]
</code></pre><br />3. In <em>twoFifo1.v</em>:<br /><pre>
parameter LOGLENGTH = 2;
...
reg [LOGLENGTH-1:0] writehead;
...
for (i = 0; i <= LENGTH-1; i = i + 1) begin
if (((writehead < writetail) && (i >= writehead) &&
(i < writetail)) ||
((writehead > writetail) && ((i >= writehead) ||
(i < writetail)))) begin
if ((readempty == 0) &&
(readfifo[readhead] == writefifo[i])) begin
match <= 1;
storeaddr <= readhead;
end
end
</pre><br />The unrolled "for" loop produces the error in nuXmv model checker (probably the <em>i</em> counter value has 32-bit vector type):<br /><pre><code class="text syntaxhl" data-language="text">TYPE ERROR file twoFifo1.smv: line 762 : illegal operand types of "<" : unsigned word[32] and unsigned word[2]
</code></pre><br />4. In <em>verilog2smv-vis-tests/Ibuf/ibuf.v</em>:<br /><pre>
input [0:2] flush;
output [0:1] load0;
output [0:2] issue0;
output [0:2] issue1;
output [0:2] valid;
...
wire nv0 = ~flush[0] & (valid[0] & ~(issue0[0] | issue1[0]) | load0);
</pre><br />This expression produces the following error in nuXmv model checker:<br /><pre><code class="text syntaxhl" data-language="text">TYPE ERROR file ibuf.smv: line 164 : illegal operand types of "|" : unsigned word[1] and unsigned word[2]
</code></pre><br />5. In <em>verilog2smv-vis-tests/BufAl/bufferAlloc.v</em>:<br /><pre>
output nack;
reg [4:0] count;
reg alloc, free;
...
count = count + (alloc & ~nack) - (free & busy[free_addr]);
</pre><br />This expression produces the following error in nuXmv model checker:<br /><pre><code class="text syntaxhl" data-language="text">TYPE ERROR file bufferAlloc.smv: line 964 : illegal operand types of "+" : unsigned word[5] and unsigned word[1]
</code></pre></p> QEMU4V - Bug #9288 (Closed): /target/mips/translate.c:2617:9: error: ‘else’ without a previous ‘if’https://forge.ispras.ru/issues/92882018-09-28T08:44:54ZSergey Smolovsmolov@ispras.ru
<p>Compilation error:</p>
<pre>
/srv/****/workspace/QEMU4V/target/mips/translate.c: In function ‘gen_logic_imm’:
/srv/****/workspace/QEMU4V/target/mips/translate.c:2617:9: error: ‘else’ without a previous ‘if’
else {
^~~~
/srv/****/workspace/QEMU4V/rules.mak:69: recipe for target 'target/mips/translate.o' failed
make[1]: *** [target/mips/translate.o] Error 1
Makefile:481: recipe for target 'subdir-mips-softmmu' failed
make: *** [subdir-mips-softmmu] Error 2
:make FAILED
</pre> Retrascope - Task #6367 (Closed): Fortress expressions printing in an SMV formathttps://forge.ispras.ru/issues/63672015-10-25T18:19:41ZSergey Smolovsmolov@ispras.ru
<p>We need utility methods for Fortress expressions printing in an SMV format.</p>
<p>The most wanted use case is:</p>
<p>we have a collection of Fortress expressions: <code>e[1]</code>, <code>e[2]</code>, ... , <code>e[n]</code>;<br />we want to produce and SMV file of the following structure (it is supposed to be so):</p>
<pre>
declarations(e[1])
...
declarations(e[n])
formula(e[1])
...
formula(e[n])
</pre>
<p>where i = 1, ... , n; <code>declarations(e[i])</code> is a list of variable declarations that are used in <code>e[i]</code> expression; <code>formula(e[i])</code> is an SMV equivalent for <code>e[i]</code> expression.</p> Retrascope - Bug #5719 (Closed): EFSM Test Generator hangs on b11https://forge.ispras.ru/issues/57192015-03-18T07:20:55ZSergey Smolovsmolov@ispras.ru
<p>The EFSM Test Generator that is run at <strong>EfsmTestGeneratorVhdlTestCase</strong> hangs on b11 VHDL design (or this testcase continues more than <strong>8 hours</strong> - it is very suspicious).</p>
<p>The log fragment is attached below.</p> Retrascope - Bug #5680 (Closed): [efsm][generator][test][fate] DirectedFateGenerator.generateSequ...https://forge.ispras.ru/issues/56802015-03-04T08:01:26ZSergey Smolovsmolov@ispras.ru
<p>The error appears upon b07 design processing.</p>
<p>The stack trace:</p>
<p>[stack]<br />java.lang.NullPointerException<br /> at ru.ispras.retrascope.engine.efsm.generator.test.fate.DirectedFateGenerator.generateSequence(DirectedFateGenerator.java:226)<br /> at ru.ispras.retrascope.engine.efsm.generator.test.fate.DirectedFateGenerator.getNextSequenceIterator(DirectedFateGenerator.java:163)<br /> at ru.ispras.retrascope.engine.efsm.generator.test.fate.EfsmFateTestGenerator.start(EfsmFateTestGenerator.java:315)<br /> at ru.ispras.retrascope.engine.efsm.generator.test.fate.EfsmFateTestGenerator.start(EfsmFateTestGenerator.java:52)<br /> at ru.ispras.retrascope.basis.Engine.start(Engine.java:200)<br /> at ru.ispras.retrascope.basis.ToolChain.start(ToolChain.java:106)<br /> at ru.ispras.retrascope.basis.Engine.start(Engine.java:200)<br /> at ru.ispras.retrascope.Retrascope$Run.start(Retrascope.java:116)<br /> at ru.ispras.retrascope.Retrascope.main(Retrascope.java:333)<br /> at ru.ispras.retrascope.Retrascope.main(Retrascope.java:355)<br /> at ru.ispras.retrascope.util.VhdlUtilTest.runRetrascope(VhdlUtilTest.java:148)<br /> at ru.ispras.retrascope.util.VhdlUtilTest.runVhdl(VhdlUtilTest.java:73)<br /> at ru.ispras.retrascope.util.HdlUtilTest.runVhdl(HdlUtilTest.java:94)<br /> at ru.ispras.retrascope.engine.efsm.generator.test.fate.EfsmFateTestGeneratorVhdlTestCase.generate(EfsmFateTestGeneratorVhdlTestCase.java:32)<br />[/stack]</p>
<p>Full log is attached below.</p> С++TESK Development Environment - Task #3756 (New): Генерация C++ кода для модели сообщенийhttps://forge.ispras.ru/issues/37562012-12-05T15:32:06ZSergey Smolovsmolov@ispras.ru
<p>Небходимо разработать метод генерации C++ кода для модели сообщений.</p>
<p>На вход методу подается несколько объектов класса Adapter. В виде какой структуры данных эти "несколько" будут подаваться - на твое усмотрение. Например, можно взять ту же, что использовалась<br />в инструменте signalsGrouper для хранения набора накликанных "интерфейсов".<br />Т.к. все адаптеры между собой различны и полных совпадений между ними быть не должно, то из самых общих соображений могу предложить использовать java.util.Set.</p>
<p>Суть метода такова: проходим по всем адаптерам и извлекаем из них объекты MessageType и помещаем их в промежуточное хранилище (возможно, тот же Set). При этом необходимо проверять, что в хранилище ещё нет такого же типа сообщений (а при разработке адаптеров для разных интерфейсов вполне реально, что они будут использовать сообщения одного типа)- делать такую проверку лучше всего посредством разработки метода сравнения equals в классе MessageType.</p>
<p><strong>Шаблон для *.h-файла</strong></p>
<pre>
#pragma once
#include <hw/message.hpp>
namespace имя_пространства_имен {
</pre> Про извлечение название пространства имен смотри <a class="issue tracker-2 status-1 priority-4 priority-default" title="Task: namespace name for test system prototypes (New)" href="https://forge.ispras.ru/issues/3755">#3755</a>
<p>Для каждого типа сообщений далее генерируем следующий код:<br /><pre>
MESSAGE(имя_типа_сообщений)
{
public:
имя_типа_сообщений();
virtual ~имя_типа_сообщений();
SUPPORT_CLONE(имя_типа_сообщений);
</pre></p>
<p>Далее для всех полей сообщения данного типа генерируем вызов соответствующего макроса. Макросы бывают следующие:<br />1) Если размер поля больше 64 бит, то нужно использовать<br /> - CPPTESK_DECLARE_FIELD_ARRAY(имя_поля, размер_массива, размер_поля); - если маска не задана<br /> - CPPTESK_DECLARE_MASKED_FIELD_ARRAY(имя_поля, размер_массива, размер_поля, маска_поля); - если маска задана</p>
<p>В силу особенностей реализации параметр размер_поля делаем равным 64, а параметр размер_массива делаем таким, чтобы удовлетворялось следующее неравенство:</p>
<p>capacity <= размер_поля*размер_массива</p>
<p>где capacity - одноименное поле соответствующего экземпляра класса MessageField.</p>
<p>2) Если размер поля меньше, или равен 64 бит, то нужно использовать<br /> - CPPTESK_DECLARE_FIELD(имя_поля, размер_поля);<br /> - CPPTESK_DECLARE_MASKED_FIELD(имя_поля, размер_поля, маска_поля);<br /> - CPPTESK_DECLARE_BIT(имя_поля); - если размер поля равен 1</p>
<pre>
};
}
</pre>
<p>Заголовочный файл называем имя_пространства_имен_msg.h</p>
<p><strong>Шаблон для .cpp файла</strong></p>
<pre>
include <имя_пространства_имен_msg.h>
namespace имя_пространства_имен {
</pre>
<p>Для каждого из типов сообщений генерируем следующий код конструктора и деструктора</p>
<pre>
имя_типа_сообщений::имя_типа_сообщений(void)
{
</pre>
<p>Для всех полей сообщения, у которых incomparable равно false (см. <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Task: флаг incomparable в полях сообщений (Closed)" href="https://forge.ispras.ru/issues/3754">#3754</a>):<br /><pre>
ADD_FIELD(имя_типа_сообщений::имя_поля);
</pre></p>
<p>Для всех полей сообщения, у которых incomparable равно true (см. <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Task: флаг incomparable в полях сообщений (Closed)" href="https://forge.ispras.ru/issues/3754">#3754</a>):<br /><pre>
ADD_INCOMPARABLE_FIELD(имя_типа_сообщений::имя_поля);
</pre></p>
<p>Рандомизируем значения полей - только если оно соответствует входному интерфейсу!<br /><pre>
RANDOMIZE_MESSAGE(*this);
}
имя_типа_сообщений::~имя_типа_сообщений(void) {}
}
</pre></p>
<p>Рекомендация - данную задачу стоить решать посредством разработки нескольких относительно простых методов в соответствующих классах, а не одного сложного.</p>