Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a p4-constraints and @refers_to parser to P4Tools. #3568

Merged
merged 33 commits into from
Nov 10, 2022
Merged
Show file tree
Hide file tree
Changes from 13 commits
Commits
Show all changes
33 commits
Select commit Hold shift + click to select a range
57d5b67
Initial commit
OleksandrLetychevskyiIntel Oct 12, 2022
1042558
Fix
OleksandrLetychevskyiIntel Oct 12, 2022
8648a36
Fix format
OleksandrLetychevskyiIntel Oct 12, 2022
f8311d6
Merge branch 'p4lang:main' into Restrictions
OleksandrLetychevskyiIntel Oct 14, 2022
9f25917
Fix types
OleksandrLetychevskyiIntel Oct 14, 2022
853663b
Merge branch 'Restrictions' of https://github.com/vladyslav-dubina/p4…
OleksandrLetychevskyiIntel Oct 14, 2022
f57bf7c
Fix types
OleksandrLetychevskyiIntel Oct 14, 2022
c3e85f1
Fix types
OleksandrLetychevskyiIntel Oct 14, 2022
cef9ef0
Add new comments and rework
OleksandrLetychevskyiIntel Oct 19, 2022
4e46a64
Merge branch 'main' into Restrictions
OleksandrLetychevskyiIntel Oct 19, 2022
f8c72b5
Fix
OleksandrLetychevskyiIntel Oct 19, 2022
6be3eca
Rework refers_to parser
OleksandrLetychevskyiIntel Oct 19, 2022
0aba162
Merge branch 'p4lang:main' into Restrictions
OleksandrLetychevskyiIntel Oct 21, 2022
0d93e8f
Parser rework
OleksandrLetychevskyiIntel Oct 25, 2022
fadae11
Fix
OleksandrLetychevskyiIntel Oct 25, 2022
7308f64
Fix
OleksandrLetychevskyiIntel Oct 26, 2022
77e2758
Fix Xfails
OleksandrLetychevskyiIntel Oct 26, 2022
550c55d
Fix processing of LNot
OleksandrLetychevskyiIntel Oct 26, 2022
8f952a5
Moving code into a function
OleksandrLetychevskyiIntel Oct 26, 2022
cbc7f48
Fix
OleksandrLetychevskyiIntel Oct 26, 2022
73d7a9a
Fix Xfails
OleksandrLetychevskyiIntel Oct 26, 2022
31e50b7
Add patch
OleksandrLetychevskyiIntel Oct 28, 2022
bd63d1a
Fix
OleksandrLetychevskyiIntel Oct 28, 2022
0694a91
Add new p4 program
OleksandrLetychevskyiIntel Oct 28, 2022
8152926
Add patch and fix makeSingleExpr function
OleksandrLetychevskyiIntel Oct 31, 2022
843418d
Fix
OleksandrLetychevskyiIntel Oct 31, 2022
225539f
Merge branch 'p4lang:main' into Restrictions
OleksandrLetychevskyiIntel Oct 31, 2022
bac9bcd
Merge branch 'main' into Restrictions
OleksandrLetychevskyiIntel Nov 1, 2022
d19496a
Fix
OleksandrLetychevskyiIntel Nov 1, 2022
c868757
Merge branch 'p4lang:main' into Restrictions
OleksandrLetychevskyiIntel Nov 2, 2022
c571ddf
Add patch
OleksandrLetychevskyiIntel Nov 7, 2022
0c22149
Merge branch 'p4lang:main' into Restrictions
OleksandrLetychevskyiIntel Nov 8, 2022
e8ebd32
Fix gtests
OleksandrLetychevskyiIntel Nov 9, 2022
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions backends/p4tools/testgen/CMakeLists.txt
100644 → 100755
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,7 @@ set(
# # XXX These should be in a library.
${P4C_SOURCE_DIR}/test/gtest/helpers.cpp
${P4C_SOURCE_DIR}/test/gtest/gtestp4c.cpp

test/gtest_utils.cpp
test/lib/format_int.cpp
test/lib/taint.cpp
Expand All @@ -56,6 +57,7 @@ set(
test/small-step/unary.cpp
test/small-step/util.cpp
test/small-step/value.cpp
test/small-step/p4_asserts_parser_test.cpp
test/transformations/saturation_arithm.cpp
test/z3-solver/asrt_model.cpp
test/z3-solver/expressions.cpp
Expand Down
2 changes: 2 additions & 0 deletions backends/p4tools/testgen/targets/bmv2/CMakeLists.txt
100644 → 100755
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,8 @@ set(
${CMAKE_CURRENT_SOURCE_DIR}/program_info.cpp
${CMAKE_CURRENT_SOURCE_DIR}/table_stepper.cpp
${CMAKE_CURRENT_SOURCE_DIR}/target.cpp
${CMAKE_CURRENT_SOURCE_DIR}/p4_asserts_parser.cpp
${CMAKE_CURRENT_SOURCE_DIR}/p4_refers_to_parser.cpp
${CMAKE_CURRENT_SOURCE_DIR}/test_backend.cpp
${CMAKE_CURRENT_SOURCE_DIR}/test_spec.cpp
PARENT_SCOPE
Expand Down
673 changes: 673 additions & 0 deletions backends/p4tools/testgen/targets/bmv2/p4_asserts_parser.cpp

Large diffs are not rendered by default.

115 changes: 115 additions & 0 deletions backends/p4tools/testgen/targets/bmv2/p4_asserts_parser.h
Original file line number Diff line number Diff line change
@@ -0,0 +1,115 @@
#ifndef TESTGEN_TARGETS_BMV2_P4_ASSERTS_PARSER_H_
#define TESTGEN_TARGETS_BMV2_P4_ASSERTS_PARSER_H_

#include <iomanip>
#include <iostream>
#include <string>
#include <string_view>
#include <utility>
#include <vector>

#include "ir/ir.h"

namespace P4Tools {
namespace AssertsParser {

class AssertsParser : public Transform {
std::vector<std::vector<const IR::Expression*>>& restrictionsVec;
IR::Vector<IR::KeyElement> keyElements;
cstring tableName;

public:
explicit AssertsParser(std::vector<std::vector<const IR::Expression*>>& output);
/// A function that calls the beginning of the transformation of restrictions from a string into
/// an IR::Expression. Internally calls all other necessary functions, for example
/// combineTokensToNames and the like, to eventually get an IR expression that meets the string
/// constraint
std::vector<const IR::Expression*> genIRStructs(cstring str);
const IR::Node* postorder(IR::P4Table* node) override;
};

class Token {
public:
enum class Kind {
Text,
True,
False,
LineStatementClose, // \n
Id, // this, this.foo
Number, // 1, 2, -1, 5.2, -5.3
Minus, // -
Plus, // +
Dot, // .
FieldAcces, // :: (after key)
MetadataAccess, // :: (otherwise)
LeftParen, // (
RightParen, // )
Equal, // ==
NotEqual, // !=
GreaterThan, // >
GreaterEqual, // >=
LessThan, // <
LessEqual, // <=
LNot, // !
Colon, // :
Semicolon, // ;
Conjunction, // &&
Disjunction, // ||
Implication, // ->
Slash, // /
Percent, // %
Shr, // >>
Shl, // <<
Mul, // *
Comment, // //
Unknown,
EndString,
End,
};

Kind m_kind{};
std::string_view m_lexeme{};
explicit Token(Kind kind) noexcept : m_kind{kind} {}

Token(Kind kind, const char* beg, std::size_t len) noexcept
: m_kind{kind}, m_lexeme(beg, len) {}

Token(Kind kind, const char* beg, const char* end) noexcept
: m_kind{kind}, m_lexeme(beg, std::distance(beg, end)) {}

Kind kind() const noexcept;

void kind(Kind kind) noexcept;

bool is(Kind kind) const noexcept;

bool is_not(Kind kind) const noexcept;

bool is_one_of(Kind k1, Kind k2) const noexcept;

template <typename... Ts>
bool is_one_of(Kind k1, Kind k2, Ts... ks) const noexcept;

std::string_view lexeme() const noexcept;

void lexeme(std::string_view lexeme) noexcept;
};

class Lexer {
public:
explicit Lexer(const char* beg) noexcept : m_beg{beg} {}

Token next() noexcept;
const char* m_beg = nullptr;

private:
Token atom(Token::Kind) noexcept;

char peek() const noexcept;
char prev() noexcept;
char get() noexcept;
};
} // namespace AssertsParser
} // namespace P4Tools

#endif /* TESTGEN_TARGETS_BMV2_P4_ASSERTS_PARSER_H_ */
147 changes: 147 additions & 0 deletions backends/p4tools/testgen/targets/bmv2/p4_refers_to_parser.cpp
Original file line number Diff line number Diff line change
@@ -0,0 +1,147 @@
#include "backends/p4tools/testgen/targets/bmv2/p4_refers_to_parser.h"

#include <boost/optional/optional_io.hpp>

#include "backends/p4tools/common/lib/ir.h"

namespace P4Tools {

namespace RefersToParser {

RefersToParser::RefersToParser(std::vector<std::vector<const IR::Expression*>>& output)
: restrictionsVec(output) {
setName("RefersToParser");
}

/// Builds names for the zombie constant and then creates a zombie constant and builds the refers_to
/// constraints based on them
void RefersToParser::createConstraint(bool table, cstring currentName, cstring currentKeyName,
cstring destKeyName, cstring destTableName,
const IR::Type* type) {
cstring tmp = "";
if (table)
tmp = currentName + "_key_" + currentKeyName;
else
tmp = currentName + currentKeyName;
auto left = IRUtils::getZombieConst(type, 0, tmp);
std::string str = static_cast<std::string>(currentName);
std::vector<std::string> elems;
std::stringstream ss(str);
std::string item;
while (std::getline(ss, item, '.')) {
elems.push_back(item);
}
str = "";
for (long unsigned int i = 0; i < elems.size() - 1; i++) {
str += elems[i] + ".";
}
tmp = str + destTableName + "_key_" + destKeyName;
auto right = IRUtils::getZombieConst(type, 0, tmp);
auto expr = new IR::Equ(left, right);
std::vector<const IR::Expression*> constraint;
constraint.push_back(expr);
restrictionsVec.push_back(constraint);
}

const IR::Node* RefersToParser::postorder(IR::Annotation* annotation) {
const IR::P4Action* prevAction = nullptr;
if (annotation->name.name == "refers_to") {
if (auto action = findOrigCtxt<IR::P4Action>()) {
if (prevAction != action) {
actionVector.push_back(action);
prevAction = action;
}
} else if (auto keys = findOrigCtxt<IR::Key>()) {
auto table = findOrigCtxt<IR::P4Table>();
CHECK_NULL(table);
auto key = findOrigCtxt<IR::KeyElement>();
CHECK_NULL(key);
auto it = find(keys->keyElements.begin(), keys->keyElements.end(), key);
if (it != keys->keyElements.end()) {
int id = it - keys->keyElements.begin();
createRefersToConstraint(key->annotations->annotations, key->expression->type,
table->controlPlaneName(), id, false,
key->expression->toString());
}
}
}
return annotation;
}
// Finds a P4Action in the actionVector according
/// to the specified input argument which is an ActionListElement
const IR::P4Action* RefersToParser::findAction(const IR::ActionListElement* input) {
for (auto element : actionVector) {
if (input->getName().name == element->name.name) {
return element;
}
}
return nullptr;
}

// Builds a variable name from the body of the "refers_to" annotation.
// The build starts at index 2 because 0 is the table name and 1 is ",".
cstring buildName(IR::Vector<IR::AnnotationToken> input) {
cstring result = "";
for (long unsigned int i = 2; i < input.size(); i++) {
result += input[i]->text;
}
return result;
}

/// An intermediate function that determines the type for future variables and partially
/// collects their names for them, after which it calls the createConstraint function,
/// which completes the construction of the constraint
void RefersToParser::createRefersToConstraint(IR::Vector<IR::Annotation> annotations,
const IR::Type* inputType, cstring controlPlaneName,
int id, bool isParameter, cstring inputName) {
cstring currentKeyName = inputName;
const IR::Type* type = nullptr;
for (auto annotation : annotations) {
if (annotation->name.name == "refers_to") {
cstring destTableName = annotation->body[0]->text;
cstring destKeyName = buildName(annotation->body);
if (auto bit = inputType->to<IR::Type_Bits>()) {
type = bit;
} else if (auto varbit = inputType->to<IR::Extracted_Varbits>()) {
type = varbit;
} else if (inputType->to<IR::Type_Unknown>()) {
type = IR::Type_Bits::get(32);
} else {
type = IR::Type_Bits::get(1);
}
if (isParameter) {
currentKeyName = "_param_" + inputName + std::to_string(id);
createConstraint(false, controlPlaneName, currentKeyName, destKeyName,
destTableName, type);
} else {
createConstraint(true, controlPlaneName, currentKeyName, destKeyName, destTableName,
type);
}
}
}
}

const IR::Node* RefersToParser::postorder(IR::ActionListElement* action) {
auto findedAction = findAction(action);
if (!findedAction) {
return action;
}
if (auto table = findOrigCtxt<IR::P4Table>()) {
if (findedAction->parameters) {
int id = 0;
for (auto parameter : findedAction->parameters->parameters) {
if (parameter->annotations) {
createRefersToConstraint(parameter->annotations->annotations, parameter->type,
table->controlPlaneName(), id, true,
findedAction->controlPlaneName());
}
id++;
}
}
}

return action;
}
} // namespace RefersToParser

} // namespace P4Tools
42 changes: 42 additions & 0 deletions backends/p4tools/testgen/targets/bmv2/p4_refers_to_parser.h
Original file line number Diff line number Diff line change
@@ -0,0 +1,42 @@
#ifndef TESTGEN_TARGETS_BMV2_P4_REFERS_TO_PARSER_H_
#define TESTGEN_TARGETS_BMV2_P4_REFERS_TO_PARSER_H_

#include <iomanip>
#include <iostream>
#include <string>
#include <string_view>
#include <utility>
#include <vector>

#include "ir/ir.h"

namespace P4Tools {
namespace RefersToParser {

class RefersToParser : public Transform {
std::vector<std::vector<const IR::Expression*>>& restrictionsVec;
std::vector<const IR::P4Action*> actionVector;
/// Finds a P4Action in the actionVector according
/// to the specified input argument which is an ActionListElement
const IR::P4Action* findAction(const IR::ActionListElement* input);
OleksandrLetychevskyiIntel marked this conversation as resolved.
Show resolved Hide resolved
/// An intermediate function that determines the type for future variables and partially
/// collects their names for them, after which it calls the createConstraint function,
/// which completes the construction of the constraint
void createRefersToConstraint(IR::Vector<IR::Annotation> annotations, const IR::Type* inputType,
cstring controlPlaneName, int id, bool isParameter,
cstring inputName);

public:
explicit RefersToParser(std::vector<std::vector<const IR::Expression*>>& output);
const IR::Node* postorder(IR::ActionListElement* action) override;
const IR::Node* postorder(IR::Annotation* annotation) override;
/// Builds names for the zombie constant and then creates a zombie constant and builds the
/// refers_to constraints based on them
void createConstraint(bool table, cstring currentName, cstring currentKeyName,
cstring destKeyName, cstring destTableName, const IR::Type* type);
};

} // namespace RefersToParser
} // namespace P4Tools

#endif /* TESTGEN_TARGETS_BMV2_P4_REFERS_TO_PARSER_H_ */
22 changes: 19 additions & 3 deletions backends/p4tools/testgen/targets/bmv2/program_info.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,8 @@
#include "backends/p4tools/testgen/core/target.h"
#include "backends/p4tools/testgen/lib/concolic.h"
#include "backends/p4tools/testgen/targets/bmv2/concolic.h"
#include "backends/p4tools/testgen/targets/bmv2/p4_asserts_parser.h"
#include "backends/p4tools/testgen/targets/bmv2/p4_refers_to_parser.h"

namespace P4Tools {

Expand Down Expand Up @@ -50,11 +52,25 @@ BMv2_V1ModelProgramInfo::BMv2_V1ModelProgramInfo(
pipelineSequence.insert(pipelineSequence.end(), subResult.begin(), subResult.end());
++pipeIdx;
}
// Sending a too short packet in BMV2 produces nonsense, so we require the packet size to be
// larger than 32 bits
targetConstraints =
/// Sending a too short packet in BMV2 produces nonsense, so we require the packet size to be
/// larger than 32 bits
const IR::Operation_Binary* constraint =
new IR::Grt(IR::Type::Boolean::get(), ExecutionState::getInputPacketSizeVar(),
IRUtils::getConstant(ExecutionState::getPacketSizeVarType(), 32));
/// Vector containing pairs of restrictions and nodes to which these restrictions apply.
std::vector<std::vector<const IR::Expression*>> restrictionsVec;
/// Defines all "entry_restriction" and then converts restrictions from string to IR
/// expressions, and stores them in restrictionsVec to move targetConstraints further.
program->apply(AssertsParser::AssertsParser(restrictionsVec));
OleksandrLetychevskyiIntel marked this conversation as resolved.
Show resolved Hide resolved
/// Defines all "refers_to" and then converts restrictions from string to IR expressions,
/// and stores them in restrictionsVec to move targetConstraints further.
program->apply(RefersToParser::RefersToParser(restrictionsVec));
for (auto element : restrictionsVec) {
for (auto restriction : element) {
constraint = new IR::LAnd(constraint, restriction);
}
}
targetConstraints = constraint;
}

const ordered_map<cstring, const IR::Type_Declaration*>*
Expand Down
Empty file modified backends/p4tools/testgen/targets/bmv2/program_info.h
100644 → 100755
Empty file.
Loading