Merge "Removed unused fields"
[openflowplugin.git] / openflowplugin / src / main / java / org / opendaylight / openflowplugin / openflow / md / core / sal / convertor / match / MatchConvertorV10Impl.java
index 323c1f050afd69c8e166f2c01bcb806a0a545283..848084573be7113e5f4e5d1cc1fa25a1046db658 100644 (file)
@@ -1,5 +1,5 @@
 /**
- * Copyright (c) 2013 Cisco Systems, Inc. and others.  All rights reserved.
+ * Copyright (c) 2013, 2015 Cisco Systems, Inc. and others.  All rights reserved.
  *
  * This program and the accompanying materials are made available under the
  * terms of the Eclipse Public License v1.0 which accompanies this distribution,
@@ -9,13 +9,18 @@
 package org.opendaylight.openflowplugin.openflow.md.core.sal.convertor.match;
 
 import java.math.BigInteger;
+import java.util.Iterator;
 
+import org.opendaylight.openflowplugin.openflow.md.core.sal.convertor.IpConversionUtil;
+import org.opendaylight.openflowplugin.openflow.md.util.ActionUtil;
 import org.opendaylight.openflowplugin.openflow.md.util.InventoryDataServiceUtil;
+import org.opendaylight.openflowplugin.api.openflow.md.util.OpenflowVersion;
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.inet.types.rev100924.Ipv4Address;
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.types.rev100924.MacAddress;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.NodeConnectorId;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.model.match.types.rev131026.Match;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.model.match.types.rev131026.match.EthernetMatch;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.model.match.types.rev131026.match.Icmpv4Match;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.model.match.types.rev131026.match.IpMatch;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.model.match.types.rev131026.match.Layer3Match;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.model.match.types.rev131026.match.Layer4Match;
@@ -24,19 +29,25 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.model.match.types.rev131026
 import org.opendaylight.yang.gen.v1.urn.opendaylight.model.match.types.rev131026.match.layer._4.match.TcpMatch;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.model.match.types.rev131026.match.layer._4.match.UdpMatch;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.common.types.rev130731.FlowWildcardsV10;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.oxm.rev130731.match.v10.grouping.MatchV10;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.oxm.rev130731.match.v10.grouping.MatchV10Builder;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.oxm.rev150225.match.v10.grouping.MatchV10;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.oxm.rev150225.match.v10.grouping.MatchV10Builder;
 
 /**
- * 
+ *
  */
 public class MatchConvertorV10Impl implements MatchConvertor<MatchV10> {
-    
+
     /** default MAC */
     public static final MacAddress zeroMac = new MacAddress("00:00:00:00:00:00");
     /** default IPv4 */
     public static final Ipv4Address zeroIPv4 = new Ipv4Address("0.0.0.0");
 
+    /*
+     * The value 0xffff (OFP_VLAN_NONE) is used to indicate
+     * that no VLAN ID is set for OF Flow.
+     */
+    private static final Integer OFP_VLAN_NONE = 0xffff;
+
     /**
      * Method builds openflow 1.0 specific match (MatchV10) from MD-SAL match.
      * @param match MD-SAL match
@@ -44,7 +55,7 @@ public class MatchConvertorV10Impl implements MatchConvertor<MatchV10> {
      * @author avishnoi@in.ibm.com
      */
     @Override
-    public MatchV10 convert(Match match,BigInteger datapathid) {
+    public MatchV10 convert(final Match match,final BigInteger datapathid) {
         MatchV10Builder matchBuilder = new MatchV10Builder();
         boolean _dLDST = true;
         boolean _dLSRC = true;
@@ -56,12 +67,12 @@ public class MatchConvertorV10Impl implements MatchConvertor<MatchV10> {
         boolean _nWTOS = true;
         boolean _tPDST = true;
         boolean _tPSRC = true;
-        
+
         matchBuilder.setInPort(0);
         matchBuilder.setDlDst(zeroMac);
         matchBuilder.setDlSrc(zeroMac);
         matchBuilder.setDlType(0);
-        matchBuilder.setDlVlan(0);
+        matchBuilder.setDlVlan(OFP_VLAN_NONE);
         matchBuilder.setDlVlanPcp((short) 0);
         matchBuilder.setNwDst(zeroIPv4);
         matchBuilder.setNwDstMask((short) 0);
@@ -71,53 +82,69 @@ public class MatchConvertorV10Impl implements MatchConvertor<MatchV10> {
         matchBuilder.setNwTos((short) 0);
         matchBuilder.setTpSrc(0);
         matchBuilder.setTpDst(0);
-        
-        EthernetMatch ethernetMatch = match.getEthernetMatch();
-        if(ethernetMatch!= null){
-            _dLDST = convertEthernetDlDst(matchBuilder, ethernetMatch);
-            _dLSRC = convertEthernetDlSrc(matchBuilder, ethernetMatch);
-            _dLTYPE = convertEthernetDlType(matchBuilder, ethernetMatch);
-        }
-        VlanMatch vlanMatch = match.getVlanMatch();
-        if(vlanMatch!= null){
-            _dLVLAN = convertDlVlan(matchBuilder, vlanMatch);
-            _dLVLANPCP = convertDlVlanPcp(matchBuilder, vlanMatch);
-        }
-        NodeConnectorId inPort = match.getInPort();
-        if(inPort!=null){
-            _iNPORT = convertInPortMatch(matchBuilder, inPort);
-        }
-        Layer3Match l3Match = match.getLayer3Match();
-        if(l3Match != null){
-            if(l3Match instanceof Ipv4Match){
-                Ipv4Match ipv4 = (Ipv4Match)l3Match;
-                _tPSRC = convertL3Ipv4SrcMatch(matchBuilder, ipv4);
-                _tPDST = convertL3Ipv4DstMatch(matchBuilder, ipv4);
+
+        if (match != null) {
+            EthernetMatch ethernetMatch = match.getEthernetMatch();
+            if(ethernetMatch!= null){
+                _dLDST = convertEthernetDlDst(matchBuilder, ethernetMatch);
+                _dLSRC = convertEthernetDlSrc(matchBuilder, ethernetMatch);
+                _dLTYPE = convertEthernetDlType(matchBuilder, ethernetMatch);
             }
-        }
-        IpMatch ipMatch = match.getIpMatch();
-        if(ipMatch!=null){
-            _nWPROTO = convertNwProto(matchBuilder, ipMatch);
-            _nWTOS = convertNwTos(matchBuilder, ipMatch);
-        }
-        Layer4Match layer4Match = match.getLayer4Match();
-        if (layer4Match != null) {
-            if (layer4Match instanceof TcpMatch) {
-                TcpMatch tcpMatch = (TcpMatch) layer4Match;
-                _tPSRC = convertL4TpSrcMatch(matchBuilder, tcpMatch);
-                _tPDST = convertL4TpDstMatch(matchBuilder, tcpMatch);
-            } else if (layer4Match instanceof UdpMatch) {
-                UdpMatch udpMatch = (UdpMatch) layer4Match;
-                _tPSRC = convertL4UdpSrcMatch(matchBuilder, udpMatch);
-                _tPDST = convertL4UdpDstMatch(matchBuilder, udpMatch);
+            VlanMatch vlanMatch = match.getVlanMatch();
+            if(vlanMatch!= null){
+                _dLVLAN = convertDlVlan(matchBuilder, vlanMatch);
+                _dLVLANPCP = convertDlVlanPcp(matchBuilder, vlanMatch);
+            }
+            NodeConnectorId inPort = match.getInPort();
+            if(inPort!=null){
+                _iNPORT = convertInPortMatch(matchBuilder, inPort);
+            }
+            Layer3Match l3Match = match.getLayer3Match();
+            if(l3Match != null){
+                if(l3Match instanceof Ipv4Match){
+                    Ipv4Match ipv4 = (Ipv4Match)l3Match;
+                    convertL3Ipv4SrcMatch(matchBuilder, ipv4);
+                    convertL3Ipv4DstMatch(matchBuilder, ipv4);
+                }
+            }
+            IpMatch ipMatch = match.getIpMatch();
+            if(ipMatch!=null){
+                _nWPROTO = convertNwProto(matchBuilder, ipMatch);
+                _nWTOS = convertNwTos(matchBuilder, ipMatch);
+            }
+            Layer4Match layer4Match = match.getLayer4Match();
+            if (layer4Match != null) {
+                if (layer4Match instanceof TcpMatch) {
+                    TcpMatch tcpMatch = (TcpMatch) layer4Match;
+                    _tPSRC = convertL4TpSrcMatch(matchBuilder, tcpMatch);
+                    _tPDST = convertL4TpDstMatch(matchBuilder, tcpMatch);
+                } else if (layer4Match instanceof UdpMatch) {
+                    UdpMatch udpMatch = (UdpMatch) layer4Match;
+                    _tPSRC = convertL4UdpSrcMatch(matchBuilder, udpMatch);
+                    _tPDST = convertL4UdpDstMatch(matchBuilder, udpMatch);
+                }
+            } else {
+                Icmpv4Match icmpv4Match = match.getIcmpv4Match();
+                if (icmpv4Match != null) {
+                    Short type = icmpv4Match.getIcmpv4Type();
+                    if (type != null) {
+                        matchBuilder.setTpSrc(type.intValue());
+                        _tPSRC = false;
+                    }
+                    Short code = icmpv4Match.getIcmpv4Code();
+                    if (code != null) {
+                        matchBuilder.setTpDst(code.intValue());
+                        _tPDST = false;
+                    }
+                }
             }
         }
-        
+
         FlowWildcardsV10 wildCards = new FlowWildcardsV10(
-                _dLDST, _dLSRC, _dLTYPE, _dLVLAN, 
+                _dLDST, _dLSRC, _dLTYPE, _dLVLAN,
                 _dLVLANPCP, _iNPORT, _nWPROTO, _nWTOS, _tPDST, _tPSRC);
         matchBuilder.setWildcards(wildCards);
-        
+
         return matchBuilder.build();
     }
 
@@ -126,8 +153,8 @@ public class MatchConvertorV10Impl implements MatchConvertor<MatchV10> {
      * @param udpMatch
      * @return is wildCard
      */
-    private static boolean convertL4UdpDstMatch(MatchV10Builder matchBuilder,
-            UdpMatch udpMatch) {
+    private static boolean convertL4UdpDstMatch(final MatchV10Builder matchBuilder,
+            final UdpMatch udpMatch) {
         if (udpMatch.getUdpDestinationPort() != null) {
             matchBuilder.setTpDst(udpMatch.getUdpDestinationPort().getValue());
             return false;
@@ -140,8 +167,8 @@ public class MatchConvertorV10Impl implements MatchConvertor<MatchV10> {
      * @param udpMatch
      * @return is wildCard
      */
-    private static boolean convertL4UdpSrcMatch(MatchV10Builder matchBuilder,
-            UdpMatch udpMatch) {
+    private static boolean convertL4UdpSrcMatch(final MatchV10Builder matchBuilder,
+            final UdpMatch udpMatch) {
         if (udpMatch.getUdpSourcePort() != null) {
             matchBuilder.setTpSrc(udpMatch.getUdpSourcePort().getValue());
             return false;
@@ -154,8 +181,8 @@ public class MatchConvertorV10Impl implements MatchConvertor<MatchV10> {
      * @param tcpMatch
      * @return is wildCard
      */
-    private static boolean convertL4TpDstMatch(MatchV10Builder matchBuilder,
-            TcpMatch tcpMatch) {
+    private static boolean convertL4TpDstMatch(final MatchV10Builder matchBuilder,
+            final TcpMatch tcpMatch) {
         if (tcpMatch.getTcpDestinationPort() != null) {
             matchBuilder.setTpDst(tcpMatch.getTcpDestinationPort().getValue());
             return false;
@@ -168,8 +195,8 @@ public class MatchConvertorV10Impl implements MatchConvertor<MatchV10> {
      * @param tcpMatch
      * @return is wildCard
      */
-    private static boolean convertL4TpSrcMatch(MatchV10Builder matchBuilder,
-            TcpMatch tcpMatch) {
+    private static boolean convertL4TpSrcMatch(final MatchV10Builder matchBuilder,
+            final TcpMatch tcpMatch) {
         if (tcpMatch.getTcpSourcePort() != null) {
             matchBuilder.setTpSrc(tcpMatch.getTcpSourcePort().getValue());
             return false;
@@ -182,10 +209,10 @@ public class MatchConvertorV10Impl implements MatchConvertor<MatchV10> {
      * @param ipMatch
      * @return is wildCard
      */
-    private static boolean convertNwTos(MatchV10Builder matchBuilder,
-            IpMatch ipMatch) {
+    private static boolean convertNwTos(final MatchV10Builder matchBuilder,
+            final IpMatch ipMatch) {
         if (ipMatch.getIpDscp() != null) {
-            matchBuilder.setNwTos(ipMatch.getIpDscp().getValue());
+            matchBuilder.setNwTos(ActionUtil.dscpToTos(ipMatch.getIpDscp().getValue()));
             return false;
         }
         return true;
@@ -196,7 +223,7 @@ public class MatchConvertorV10Impl implements MatchConvertor<MatchV10> {
      * @param ipMatch
      * @return is wildCard
      */
-    private static boolean convertNwProto(MatchV10Builder matchBuilder, IpMatch ipMatch) {
+    private static boolean convertNwProto(final MatchV10Builder matchBuilder, final IpMatch ipMatch) {
         if (ipMatch.getIpProtocol() != null) {
             matchBuilder.setNwProto(ipMatch.getIpProtocol());
             return false;
@@ -205,51 +232,48 @@ public class MatchConvertorV10Impl implements MatchConvertor<MatchV10> {
     }
 
     /**
+     * Method splits the IP address and its mask and set their respective values in MatchV10Builder instance.
+     * Wildcard value of the IP mask will be determined by Openflow java encoding library.
      * @param matchBuilder
      * @param ipv4
-     * @return is wildCard
      */
-    private static boolean convertL3Ipv4DstMatch(MatchV10Builder matchBuilder,
-            Ipv4Match ipv4) {
+    private static void convertL3Ipv4DstMatch(final MatchV10Builder matchBuilder,
+            final Ipv4Match ipv4) {
         if(ipv4.getIpv4Destination()!=null){
-            String[] addressParts = ipv4.getIpv4Destination().getValue().split("/");
+            Iterator<String> addressParts = IpConversionUtil.PREFIX_SPLITTER.split(ipv4.getIpv4Destination().getValue()).iterator();
+            Ipv4Address ipv4Address = new Ipv4Address(addressParts.next());
             Integer prefix = buildPrefix(addressParts);
-
-            Ipv4Address ipv4Address = new Ipv4Address(addressParts[0]);
             matchBuilder.setNwDst(ipv4Address);
             matchBuilder.setNwDstMask(prefix.shortValue());
-            return false;
         }
-        return true;
     }
 
     /**
+     * Method splits the IP address and its mask and set their respective values in MatchV10Builder instance.
+     * Wildcard value of the IP mask will be determined by Openflow java encoding library.
      * @param matchBuilder
      * @param ipv4
-     * @return is wildCard
      */
-    private static boolean convertL3Ipv4SrcMatch(MatchV10Builder matchBuilder,
-            Ipv4Match ipv4) {
+    private static void convertL3Ipv4SrcMatch(final MatchV10Builder matchBuilder,
+            final Ipv4Match ipv4) {
         if(ipv4.getIpv4Source()!=null){
-            String[] addressParts = ipv4.getIpv4Source().getValue().split(MatchConvertorImpl.PREFIX_SEPARATOR);
-            Ipv4Address ipv4Address = new Ipv4Address(addressParts[0]);
+            Iterator<String> addressParts = IpConversionUtil.PREFIX_SPLITTER.split(ipv4.getIpv4Source().getValue()).iterator();
+            Ipv4Address ipv4Address = new Ipv4Address(addressParts.next());
             int prefix = buildPrefix(addressParts);
-            
+
             matchBuilder.setNwSrc(ipv4Address);
             matchBuilder.setNwSrcMask((short) prefix);
-            return false;
         }
-        return true;
     }
 
     /**
      * @param addressParts
      * @return
      */
-    private static int buildPrefix(String[] addressParts) {
-        int prefix = 0;
-        if (addressParts.length > 1) {
-            prefix = Integer.parseInt(addressParts[1]);
+    private static int buildPrefix(final Iterator<String> addressParts) {
+        int prefix = 32;
+        if (addressParts.hasNext()) {
+            prefix = Integer.parseInt(addressParts.next());
         }
         return prefix;
     }
@@ -257,10 +281,10 @@ public class MatchConvertorV10Impl implements MatchConvertor<MatchV10> {
     /**
      * @param matchBuilder
      * @param vlanMatch
-     * @return 
+     * @return
      */
-    private static boolean convertDlVlanPcp(MatchV10Builder matchBuilder,
-            VlanMatch vlanMatch) {
+    private static boolean convertDlVlanPcp(final MatchV10Builder matchBuilder,
+            final VlanMatch vlanMatch) {
         if (vlanMatch.getVlanPcp() != null) {
             matchBuilder.setDlVlanPcp(vlanMatch.getVlanPcp().getValue());
             return false;
@@ -271,11 +295,12 @@ public class MatchConvertorV10Impl implements MatchConvertor<MatchV10> {
     /**
      * @param matchBuilder
      * @param vlanMatch
-     * @return 
+     * @return
      */
-    private static boolean convertDlVlan(MatchV10Builder matchBuilder, VlanMatch vlanMatch) {
+    private static boolean convertDlVlan(final MatchV10Builder matchBuilder, final VlanMatch vlanMatch) {
         if (vlanMatch.getVlanId() != null) {
-            matchBuilder.setDlVlan(vlanMatch.getVlanId().getVlanId().getValue());
+            int vlanId = vlanMatch.getVlanId().getVlanId().getValue();
+            matchBuilder.setDlVlan((vlanId == 0 ? OFP_VLAN_NONE : vlanId));
             return false;
         }
         return true;
@@ -286,8 +311,8 @@ public class MatchConvertorV10Impl implements MatchConvertor<MatchV10> {
      * @param ethernetMatch
      * @return is wildCard
      */
-    private static boolean convertEthernetDlType(MatchV10Builder matchBuilder,
-            EthernetMatch ethernetMatch) {
+    private static boolean convertEthernetDlType(final MatchV10Builder matchBuilder,
+            final EthernetMatch ethernetMatch) {
         if (ethernetMatch.getEthernetType() != null) {
             matchBuilder.setDlType(ethernetMatch.getEthernetType().getType().getValue().intValue());
             return false;
@@ -300,8 +325,8 @@ public class MatchConvertorV10Impl implements MatchConvertor<MatchV10> {
      * @param ethernetMatch
      * @return is wildCard
      */
-    private static boolean convertEthernetDlSrc(MatchV10Builder matchBuilder,
-            EthernetMatch ethernetMatch) {
+    private static boolean convertEthernetDlSrc(final MatchV10Builder matchBuilder,
+            final EthernetMatch ethernetMatch) {
         if (ethernetMatch.getEthernetSource() != null) {
             matchBuilder.setDlSrc(ethernetMatch.getEthernetSource().getAddress());
             return false;
@@ -314,8 +339,8 @@ public class MatchConvertorV10Impl implements MatchConvertor<MatchV10> {
      * @param ethernetMatch
      * @return is wildCard
      */
-    private static boolean convertEthernetDlDst(MatchV10Builder matchBuilder,
-            EthernetMatch ethernetMatch) {
+    private static boolean convertEthernetDlDst(final MatchV10Builder matchBuilder,
+            final EthernetMatch ethernetMatch) {
         if (ethernetMatch.getEthernetDestination() != null) {
             matchBuilder.setDlDst(ethernetMatch.getEthernetDestination().getAddress());
             return false;
@@ -327,9 +352,9 @@ public class MatchConvertorV10Impl implements MatchConvertor<MatchV10> {
      * @param matchBuilder
      * @param inPort
      */
-    private static boolean convertInPortMatch(MatchV10Builder matchBuilder, NodeConnectorId inPort) {
+    private static boolean convertInPortMatch(final MatchV10Builder matchBuilder, final NodeConnectorId inPort) {
         if (inPort != null) {
-            matchBuilder.setInPort(InventoryDataServiceUtil.portNumberfromNodeConnectorId(inPort).intValue());
+            matchBuilder.setInPort(InventoryDataServiceUtil.portNumberfromNodeConnectorId(OpenflowVersion.OF10, inPort).intValue());
             return false;
         }
         return true;