Skip to content
New issue

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

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

Already on GitHub? # to your account

Open simh master #1

Merged
merged 3 commits into from
Jun 5, 2022
Merged

Open simh master #1

merged 3 commits into from
Jun 5, 2022

Conversation

markpizz
Copy link
Contributor

@markpizz markpizz commented Jun 5, 2022

This pull request contains the commits made by others (Trevor Warwick @trevorwarwick in particular) since the addition of the LICENSE.txt in the simh/simh master branch

- This involves adding some trailing characters to received DDCMP frames
  to meet host driver expectations (simulation then behaves more like a
  real synchronous device). 1 SYN is enough for VMS, but RSX needs a few
  when there is no abutting frame.
- The VMS driver implements the CRC performance option in section 5.1.2
  of the DDCMP spec., and thus relies on receiving a valid character after
  the CRC on a received frame.
- When the DUP TXCSR DEVICE_RESET bit is set, which clears the MAISEL bits, any
existing tmxr loopback must also be removed.
@pkoning2
Copy link
Member

pkoning2 commented Jun 5, 2022

Thanks Mark. It looks like simh/simh@ac73e7b is also needed for all this to work. Is that right?

These devices start disabled and will be that way in essentially all
working systems, but there apparently was a DECnet Phase V
support for this device, so it is added to all systems. The DPV
should now be readily testable.

As mentioned in #1152.  That PR will fix the DUP device.

This commit is explicitly released from any license restriction
mentioned in the LICENSE.txt of the github.com/simh/simh
master branch changes.
@markpizz
Copy link
Contributor Author

markpizz commented Jun 5, 2022

The commits could have been digested with no ill effects to any build, but only the PDP11 (and PDP10) would have included @trevorwarwick modified DUP code since only those simulators included the DUP device. I have explicitly released the changes in the commit you reference from any of the previously discussed LICENSE.txt constraints on my future commits.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants