ONOS-635 Adding ICMPv6 type and code builder for HostMonitor.
aovid duplicate call of toString.
Apply changes to FlowEntryBuilder and FlowModBuilder as well.
Fix missing break; issue.
Change-Id: I1a2e59ca943bbcc9677b9fc0cb9eb2f7cb3d7aa3
diff --git a/core/api/src/main/java/org/onosproject/net/flow/DefaultTrafficSelector.java b/core/api/src/main/java/org/onosproject/net/flow/DefaultTrafficSelector.java
index 756fdd7..e5f552d 100644
--- a/core/api/src/main/java/org/onosproject/net/flow/DefaultTrafficSelector.java
+++ b/core/api/src/main/java/org/onosproject/net/flow/DefaultTrafficSelector.java
@@ -194,6 +194,16 @@
}
@Override
+ public Builder matchIcmpv6Type(Byte icmpv6Type) {
+ return add(Criteria.matchIcmpv6Type(icmpv6Type));
+ }
+
+ @Override
+ public Builder matchIcmpv6Code(Byte icmpv6Code) {
+ return add(Criteria.matchIcmpv6Code(icmpv6Code));
+ }
+
+ @Override
public Builder matchMplsLabel(Integer mplsLabel) {
return add(Criteria.matchMplsLabel(mplsLabel));
}
diff --git a/core/api/src/main/java/org/onosproject/net/flow/TrafficSelector.java b/core/api/src/main/java/org/onosproject/net/flow/TrafficSelector.java
index 267fb51..e483043 100644
--- a/core/api/src/main/java/org/onosproject/net/flow/TrafficSelector.java
+++ b/core/api/src/main/java/org/onosproject/net/flow/TrafficSelector.java
@@ -163,6 +163,22 @@
public Builder matchIPv6Dst(IpPrefix ip);
/**
+ * Matches a ICMPv6 type.
+ *
+ * @param icmpv6Type a ICMPv6 type
+ * @return a selection builder
+ */
+ public Builder matchIcmpv6Type(Byte icmpv6Type);
+
+ /**
+ * Matches a ICMPv6 code.
+ *
+ * @param icmpv6Code a ICMPv6 code
+ * @return a selection builder
+ */
+ public Builder matchIcmpv6Code(Byte icmpv6Code);
+
+ /**
* Matches on a MPLS label .
* @param mplsLabel a MPLS label.
* @return a selection builder
diff --git a/core/api/src/main/java/org/onosproject/net/flow/criteria/Criteria.java b/core/api/src/main/java/org/onosproject/net/flow/criteria/Criteria.java
index e725143..5ef50c4 100644
--- a/core/api/src/main/java/org/onosproject/net/flow/criteria/Criteria.java
+++ b/core/api/src/main/java/org/onosproject/net/flow/criteria/Criteria.java
@@ -167,6 +167,26 @@
return new IPCriterion(ip, Type.IPV6_DST);
}
+ /*
+ * Creates a match on ICMPv6 type field using the specified value.
+ *
+ * @param icmpv6Type ICMPv6 type
+ * @return match criterion
+ */
+ public static Criterion matchIcmpv6Type(Byte icmpv6Type) {
+ return new Icmpv6TypeCriterion(icmpv6Type);
+ }
+
+ /**
+ * Creates a match on ICMPv6 code field using the specified value.
+ *
+ * @param icmpv6Code ICMPv6 code
+ * @return match criterion
+ */
+ public static Criterion matchIcmpv6Code(Byte icmpv6Code) {
+ return new Icmpv6CodeCriterion(icmpv6Code);
+ }
+
/**
* Creates a match on MPLS label.
* @param mplsLabel MPLS label
@@ -588,6 +608,96 @@
}
/**
+ * Implementation of ICMPv6 type criterion.
+ */
+ public static final class Icmpv6TypeCriterion implements Criterion {
+
+ private final Byte icmpv6Type;
+
+ public Icmpv6TypeCriterion(Byte icmpv6Type) {
+ this.icmpv6Type = icmpv6Type;
+ }
+
+ @Override
+ public Type type() {
+ return Type.ICMPV6_TYPE;
+ }
+
+ public Byte icmpv6Type() {
+ return icmpv6Type;
+ }
+
+ @Override
+ public String toString() {
+ return toStringHelper(type().toString())
+ .add("icmpv6Type", icmpv6Type & 0xff).toString();
+ }
+
+ @Override
+ public int hashCode() {
+ return Objects.hash(icmpv6Type, type());
+ }
+
+ @Override
+ public boolean equals(Object obj) {
+ if (this == obj) {
+ return true;
+ }
+ if (obj instanceof Icmpv6TypeCriterion) {
+ Icmpv6TypeCriterion that = (Icmpv6TypeCriterion) obj;
+ return Objects.equals(icmpv6Type, that.icmpv6Type) &&
+ Objects.equals(this.type(), that.type());
+ }
+ return false;
+ }
+ }
+
+ /**
+ * Implementation of ICMPv6 code criterion.
+ */
+ public static final class Icmpv6CodeCriterion implements Criterion {
+
+ private final Byte icmpv6Code;
+
+ public Icmpv6CodeCriterion(Byte icmpv6Code) {
+ this.icmpv6Code = icmpv6Code;
+ }
+
+ @Override
+ public Type type() {
+ return Type.ICMPV6_CODE;
+ }
+
+ public Byte icmpv6Code() {
+ return icmpv6Code;
+ }
+
+ @Override
+ public String toString() {
+ return toStringHelper(type().toString())
+ .add("icmpv6Code", icmpv6Code & 0xff).toString();
+ }
+
+ @Override
+ public int hashCode() {
+ return Objects.hash(icmpv6Code, type());
+ }
+
+ @Override
+ public boolean equals(Object obj) {
+ if (this == obj) {
+ return true;
+ }
+ if (obj instanceof Icmpv6CodeCriterion) {
+ Icmpv6CodeCriterion that = (Icmpv6CodeCriterion) obj;
+ return Objects.equals(icmpv6Code, that.icmpv6Code) &&
+ Objects.equals(this.type(), that.type());
+ }
+ return false;
+ }
+ }
+
+ /**
* Implementation of MPLS tag criterion.
*/
public static final class MplsCriterion implements Criterion {