Uploaded image for project: 'EtherNet/IP Firmware V3'
  1. EtherNet/IP Firmware V3
  2. PSEISV3-371

API Change: Do not delete remanent data on SET_CONFIGURATION_PARAMETERS_REQ with a changed configuration

    Details

    • Type: Change
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: V3.4.1.0, V3.5.0.0
    • Component/s: None
    • Labels:
      None
    • Account:
      SPC EthernetIp Slave (SPCETHERNE)

      Description


      Currently, when we receive packet SET_CONFIGURATION_PARAMETERS_REQ, we checksum it and compare with the persistently stored checksum of the previous such packet.
      If checksums are equal, i.e. if the same packet has been received again, we apply the remanent data, otherwise we delete the remanent data.

      With net90/GCI, this behavior changes as follows:

      • Checksumming is removed, remanent data is _not_ deleted on SET_CONFIGURATION_PARAMETERS_REQ
      • Instead, remanent data always has to be deleted explicitly with the HIL_DELETE_CONFIG_REQ, which also deletes the stored configuration (e.g. due to CIP Identity Reset Type 1)
      • The remanent data, unless deleted explicitly, is always applied on top of the received configuration (this happens in HIL_CHANNEL_INIT_REQ)

      Justification:

      • Consistency with other stacks
      • Choice of packet set (SetConfig vs. Extended) and remanent data storage (either by host or by stack) can become independent concepts (currently under discussion)
      • Checksumming to distinguish SetConfig-packets has a chance of failure, although with very low probability

      Action items:

      • Implement behavior for eip_gci for netX90/eCos
      • Adapt specific AP task accordingly
      • Adapt API reference

       - Mention changes in porting instructions

        Attachments

          Expenses

            Activity

              Status Description

                People

                • Reporter:
                  MBommert Marc Bommert
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: