PCM Hammer Suite development

They go by many names, P01, P59, VPW, '0411 etc. Also covering E38 and newer here.
Post Reply
User avatar
Gampy
Posts: 2330
Joined: Sat Dec 15, 2018 7:38 am

Re: PCM Hammer Suite development

Post by Gampy »

Something else that boiled up with all this AllPro testing,
Elapsed time 00:10:14.5380859
Elapsed time 00:10:15.3017578
Elapsed time 00:10:31.4160156
Elapsed time 00:11:03.1767578
Elapsed time 00:11:29.5888672
Elapsed time 00:11:49.3994141
Elapsed time 00:12:14.4736328
Elapsed time 00:12:38.1113281
Elapsed time 00:13:03.6748047
Elapsed time 00:13:25.7861328
However, Re-Intialize Device and you'll get,
Elapsed time 00:09:43.9550781
good speed again ...

Yes, this is different then the other issue, the other issue causes retries, there were Zero retries in the above (or below) back to back runs!
The other issue does not go away with a Re-Initizaltion, it has to be disconnected and left for a period of time.
Yes, all the above runs are with the SparkFun cobbled onto the bus ...

This also happens with the SparkFun (STN1110),
Elapsed time 00:22:18.3759766
Elapsed time 00:22:45.8564453
Elapsed time 00:23:04.2148437
Elapsed time 00:23:20.3984375
Elapsed time 00:23:48.9541016
Elapsed time 00:24:03.8076172
Elapsed time 00:24:30.9707031
Elapsed time 00:24:58.9423828
Elapsed time 00:25:18.5634766
Elapsed time 00:25:42.8623047
Re-Initialize device,
Elapsed time 00:22:05.4882812
and the speed returns.

I have not checked this out on uploads (writes) yet, I will, I want to modify for full clone test writes (non public) for longer run times.

Not that typical users are going to do multiples of back to back operations ...
Intelligence is in the details!

It is easier not to learn bad habits, then it is to break them!

If I was here to win a popularity contest, their would be no point, so I wouldn't be here!
User avatar
NSFW
Posts: 679
Joined: Fri Feb 02, 2018 3:13 pm

Re: PCM Hammer Suite development

Post by NSFW »

Wow, that's weird. You might be on to something with that heat theory.
Please don't PM me with technical questions - start a thread instead, and send me a link to it. That way I can answer in public, and help other people who have the same question. Thanks!
User avatar
antus
Site Admin
Posts: 8237
Joined: Sat Feb 28, 2009 8:34 pm
cars: TX Gemini 2L Twincam
TX Gemini SR20 18psi
Datsun 1200 Ute
Subaru Blitzen '06 EZ30 4th gen, 3.0R Spec B
Contact:

Re: PCM Hammer Suite development

Post by antus »

Does the device reset also clear the log windows in pcmhammer? It could be the size of the buffers for scrollback.
Have you read the FAQ? For lots of information and links to significant threads see here: http://pcmhacking.net/forums/viewtopic.php?f=7&t=1396
User avatar
Gampy
Posts: 2330
Joined: Sat Dec 15, 2018 7:38 am

Re: PCM Hammer Suite development

Post by Gampy »

I use the "Getting Hot" to describe what it acts like ... Not suggesting it's actually getting hot. Adding another device would not affect a over heating issue IMO.
Since cobbling on the SparkFun I have left them hooked up and powered for days without ill affect.

Log size ... Hmmm, could be, I'll sheck it out.
I'll add a button to clear just the log between runs ...
Intelligence is in the details!

It is easier not to learn bad habits, then it is to break them!

If I was here to win a popularity contest, their would be no point, so I wouldn't be here!
User avatar
Gampy
Posts: 2330
Joined: Sat Dec 15, 2018 7:38 am

Re: PCM Hammer Suite development

Post by Gampy »

Preliminary testing says: You NAILED IT Antus!

Hijacked the Re-Initialize Device button,

Code: Select all

            //await this.InitializeCurrentDevice();
            this.ResetLogs();
3 runs to show the rise ...
Elapsed time 00:09:44.8867188
Elapsed time 00:10:05.5205078
Elapsed time 00:10:31.7890625
--- Reset Logs.
Elapsed time 00:09:31.5976563
Intelligence is in the details!

It is easier not to learn bad habits, then it is to break them!

If I was here to win a popularity contest, their would be no point, so I wouldn't be here!
User avatar
NSFW
Posts: 679
Joined: Fri Feb 02, 2018 3:13 pm

Re: PCM Hammer Suite development

Post by NSFW »

Nice job tracking this down, guys.

Could probably fix this for the Results tab by adding a status bar, so it wouldn't append a new line of text with every read or write message.

For the debug tab, when it's not visible we could just add new messages to a linked list when the tab is in the background. And only update the text control when the tab is selected.

I'll see if I can get it done this weekend.
Please don't PM me with technical questions - start a thread instead, and send me a link to it. That way I can answer in public, and help other people who have the same question. Thanks!
User avatar
Gampy
Posts: 2330
Joined: Sat Dec 15, 2018 7:38 am

Re: PCM Hammer Suite development

Post by Gampy »

Thought I would give a bit-O-info on PR#175

Firstly, what is does.
It allows building the kernel with a command line specified kernel base address.
It allows for system dependent configuration by accepting named parameters, thus it can be wrapped with a system dependent wrapper or simply used as is.

IE: build.cmd -aFF9090
.cmd --^^^ Very important.

I intentionally called it Build.cmd and left build.bat with a minor change (build.bat wraps Build.cmd) for I do not know what other developers are doing and I don't want to break them, they can migrate as they wish.
Windows execution order is .BAT before .CMD, thus the original build.bat is the default when executing with "build" (no extension).

That means to use this new parameterized Build.cmd you must explicitly execute "Build.cmd".

It's pretty self explanatory, type: "Build.cmd -h" for help.

TIP: Use -h as the last parameter and it will display the current parameters in the help screen and exit.
EXAMPLE: Build.cmd -aFF9090 -h
Results in,

Code: Select all

  build.cmd -a<address> -c -d -g<path> -p<path>

    -a<address>
      Set base address for the kernel. (no space, no 0x)
      Value: 0xFF9090

    -c
      Set flag to copy kernel.bin to PcmHammer's build directory or not.
      Value: True

    -d
      Set flag to dump kernel.elf > kernel.disassembly or not.
      Value: False

    -g<path>
      Set path to GNU m68k bin dir. (no space)
      Value: C:\SysGCC\m68k-elf\bin\

    -p<path>
      Set path\<filename> where to copy kernel.bin. (no space)
      Value: ..\Apps\PcmHammer\bin\debug\

    /h
    -h
    --help
      Help Menu


    Example:
    build.cmd -aFF9090 -gC:\SysGCC\m68k-elf\bin\ -p..\Apps\PcmHammer\bin\debug\
Intelligence is in the details!

It is easier not to learn bad habits, then it is to break them!

If I was here to win a popularity contest, their would be no point, so I wouldn't be here!
User avatar
NSFW
Posts: 679
Joined: Fri Feb 02, 2018 3:13 pm

Re: PCM Hammer Suite development

Post by NSFW »

Good stuff. I merged this and some other pull requests just now.

There are some other pull requests still pending, and I will probably also merge, I just want to spend some more time pondering them first. I should have some time later this week or on the weekend.
Please don't PM me with technical questions - start a thread instead, and send me a link to it. That way I can answer in public, and help other people who have the same question. Thanks!
User avatar
Gampy
Posts: 2330
Joined: Sat Dec 15, 2018 7:38 am

Re: PCM Hammer Suite development

Post by Gampy »

Thank you!

I expected a couple of them to be well scrutinized ... Deserving so, they are kinda critical bits!

More to come as conflicts merge.

Your progress bar branch is awesome, the "background Debug log" for lack of a better term does in fact stop the slow down, after the first run, the times do remain consistent if the debug tab is hidden. :thumbup:

It also taught me how to integrate my status bar into your code without it looking like a sore thumb, so I added a progress bar to it, if interested I'll post an image, still working on getting 'the look' right.
Intelligence is in the details!

It is easier not to learn bad habits, then it is to break them!

If I was here to win a popularity contest, their would be no point, so I wouldn't be here!
User avatar
Gampy
Posts: 2330
Joined: Sat Dec 15, 2018 7:38 am

Re: PCM Hammer Suite development

Post by Gampy »

Whoops!

Just remembered I forgot to mention a very very important warning about PR#175 and Build.cmd.
(Or any DOS/Windows Batch file programming for that matter)

When editing,

Never use a Unix editor!
The correct Windows line endings ('\r\n') are extremely vital.

Never use invalid labels (': comment') as comments!

Both will cause internal line numbering issues and will break batch files.
Intelligence is in the details!

It is easier not to learn bad habits, then it is to break them!

If I was here to win a popularity contest, their would be no point, so I wouldn't be here!
Post Reply