Hopefully, when the KDP is working Rob will merge (and if necessary debug) my changes & commit them to fix both problems. Ideally adding the page optimization, but beggars can't be choosers. If he doesn't I'll try to get to that later (if I have commit access to simh on github).

I suppose I should see about building a current version, as I seem to be in the middle of debugging several things by Braille... (Or is this simply a memory test for me :-)

This communication may not represent my employer's views,
if any, on the matters discussed.

On 27-May-13 11:48, Mark Pizzolato - Info Comm wrote:
On Monday, May 27, 2013 at 6:19 AM, Timothe Litt wrote:
This thread is getting far too messy.

The increment problem found by Rob is NOT subtle.  It's just plain broken.
I agree 100%.

The code, as written, would write twice as much simulated memory as intended 
which usually would crash things nicely (and did when Rob tried with the DMC).

I found that the ONLY current use (prior to Rob's DMC) of this routine was in 
the CD11 simulator and that use either might never actually get executed (if 
the CDCSR_V_PACK bit not been set), OR if it was actually executed it would 
have been with the constant byte count of 2 which should have written 1 16/18 
bit word, but actually wrote 2 due to the bug.  This may have never been 
exercised, OR writing the second word may have merely written the high bytes 18 
bits of a word which was never referenced by the program...

The mishandling of the high bits is what I meant when I wrote 'subtle'.
Understood.



Attachment: smime.p7s
Description: S/MIME Cryptographic Signature

_______________________________________________
Simh mailing list
[email protected]
http://mailman.trailing-edge.com/mailman/listinfo/simh

Reply via email to