[BUG] Crash during Print, LA, IS, Bugfix.2.1x #25553

jesterhead82 · 2023-03-23T17:05:05Z

Did you test the latest `bugfix-2.1.x` code?

Yes, and the problem still exists.

Bug Description

Maybe related to
#25541
#25474

Printer crashes during print @2.88mm Layer, tried on 2 different sd cards
Prints fine if I disable LA, ~50/100mm Print speeds, IS @ ~43, LA 0.02, ESTEPS ~700

Configuration.zip
GCode.zip

Bug Timeline

No response

Expected behavior

No response

Actual behavior

No response

Steps to Reproduce

Version of Marlin Firmware

Bugfix 2.1.x as of this week

Printer model

Custom Corexy

Electronics

BTT 1.4T

Add-ons

No response

Bed Leveling

ABL Linear grid

Your Slicer

Prusa Slicer

Host Software

None

Don't forget to include

A ZIP file containing your Configuration.h and Configuration_adv.h.

Additional information & file uploads

No response

The text was updated successfully, but these errors were encountered:

ThomasToka · 2023-03-23T20:21:41Z

I can confirm that last commits are somehow faulty. I poke around today around my printer stopping at my usual cube at around 40%.

My last working build has this as last commit: 10983d0

Will disable LA now to check if it works without LA with my recent includes of the latest commits.

EDIT: Confirmed. Disabling LA fixes it. IS stayed enabled. My printer is a E3S1Pro with TMC2208_STANDALONE.

ThomasToka · 2023-03-23T21:15:06Z

Reverting this one fixes it ca77850

I have not fully understood it but maybe this missing else here is it:

e8b0f38

thisiskeithb · 2023-03-23T21:17:30Z

@tombrazier Can you take a look at this?

tombrazier · 2023-03-23T21:54:56Z

@jesterhead82 @ThomasToka Can you describe exactly what your printer does at the point where the print fails? My guess is movement freezes and then after a few seconds the firmware reboots. But if it is something other than that please describe the behaviour. Also: does the E stepper have any noticeable behaviour when the crash occurs?

tombrazier · 2023-03-23T21:56:40Z

I have not fully understood it but maybe this missing else here is it:

@ThomasToka What mainboard do you have? I think @jesterhead82 has a 32 bit MCU, in which case that code is not compiled.

ThomasToka · 2023-03-23T22:13:08Z

print stops and printer reboots. always at the same position. cube at around 40%.

i have creality f4 board on a ender 3 s1 pro. removing the commit fixes it.

ThomasToka · 2023-03-23T22:14:29Z

nothing else hearable.

jesterhead82 · 2023-03-23T22:32:12Z

I have not fully understood it but maybe this missing else here is it:

@ThomasToka What mainboard do you have? I think @jesterhead82 has a 32 bit MCU, in which case that code is not compiled.

Board has a LPC1769, so yes.

As for the stepper, there is some "eechy" noise shortly before the reboot, i guess some motor is doing something stupid. But it could be the board/display maybe, could try again tomorrow is that would be helpfull..

tombrazier · 2023-03-23T22:34:03Z

It could just possibly be a division by zero caused by the changes to all the calls to calc_timer_interval() in ca77850. Could one or both of you try changing them back?

ThomasToka · 2023-03-23T22:59:27Z

yes as i said if i revert this commit it works.

PendulumPit · 2023-03-24T03:55:55Z

I setup the latest bugfix the other day to do a speed test on a benchy and got through that no issue. After I turned on LA as usual and printer would stop and reboot. Tried again and same. I was going to mention this the other day but I got busy with work.
Was trying to beat Tom's 13 minute benchy lol I failed from a layer shift :)

thisiskeithb · 2023-03-24T05:18:42Z

As for the stepper, there is some "eechy" noise shortly before the reboot

My Prusa Bear with a BTT002 (STM32F4) has been printing all day with bugfix-2.1.x (724ba4b) without issue, but the Biqu Hurakan with Manta M4P (STM32G0) freezes & reboots in the same spot consistently with a similar "eechy" symptom after ~4-5 layers. Both are running Linear Advance & Input Shaping is disabled.

After reverting ca77850, the reboots stop.

tombrazier · 2023-03-24T08:04:29Z

Could someone test #25557?

ThomasToka · 2023-03-24T08:50:10Z

printing. result will follow in 20 min.

ThomasToka · 2023-03-24T08:58:53Z

Confirmed. Works again.

ThomasToka · 2023-03-24T09:21:23Z

Trying now a print with the follow up a3c4f25 .

result in some minutes..

ThomasToka · 2023-03-24T09:43:03Z

Nope crashed again on same position.

tombrazier · 2023-03-24T10:04:11Z

Thanks. So it appears that it's not a division by zero. I would really like to understand what is going on before proceeding.

I don't have time now to try and narrow it down but if anyone wants to play, it would be good to know exactly which of the four calls to calc_timer_interval() is the one that is causing the error. And if anyone is able to insert a SERIAL_ECHOLNPGMto report the offending value that is passed to calc_timer_interval() that would be very helpful.

ThomasToka · 2023-03-24T10:28:36Z

Can you give me a matching SERIAL_ECHOLNPGM so i could try?

tombrazier · 2023-03-24T11:44:44Z

For the call to calc_timer_interval() on around line 2260 it would be

SERIAL_ECHOLNPGM("step_rate ", (acc_step_rate + la_step_rate) >> current_block->la_scaling);

That will print out on the serial terminal, assuming you're using something like Octoprint or Prontrface.

tombrazier · 2023-03-24T11:46:29Z

And you can keep adding pairs of strings and values. e.g.

SERIAL_ECHOLNPGM("step_rate ", (acc_step_rate + la_step_rate) >> current_block->la_scaling, " asr ", acc_step_rate, " lsr ", la_step_rate, " las ", current_block->la_scaling);

ThomasToka · 2023-03-24T11:48:38Z

Can you give me a examples for all 4? I will then do the tests and post. I dont know what you exactly need on the four points.
But can invest the time to debug.

tombrazier · 2023-03-24T11:52:22Z

Just copy the expression that is passed to calc_timer_interval() and, if it makes sense, also the variables that make up that expression. Sorry to be difficult, I just really need to be doing something else today.

ThomasToka · 2023-03-24T12:03:25Z

Sorry i am not that good in serial operations.. but will help best i can.

SERIAL_ECHOLNPGM("step_rate ", (acc_step_rate + la_step_rate) >> current_block->la_scaling, " asr ", acc_step_rate, " lsr ", la_step_rate, " las ", current_block->la_scaling);

la_interval = calc_timer_interval((acc_step_rate + la_step_rate) >> current_block->la_scaling);

SERIAL_ECHOLNPGM("step_rate2 ", (reverse_e ? la_step_rate - step_rate : step_rate - la_step_rate) >> current_block->la_scaling, " sr ", step_rate, " lsr ", la_step_rate, " las ", current_block->la_scaling);
 
la_interval = calc_timer_interval((reverse_e ? la_step_rate - step_rate : step_rate - la_step_rate) >> current_block->la_scaling);

SERIAL_ECHOLNPGM("step_rate3 ", (current_block->initial_rate + la_step_rate) >> current_block->la_scaling, " lsr ", la_step_rate, " las ", current_block->la_scaling);

la_interval = calc_timer_interval((current_block->initial_rate + la_step_rate) >> current_block->la_scaling);

like this or after the la_interval ?

vovodroid · 2023-04-01T17:19:12Z

More than 1 hour print on BTT SKR 2 seems to be OK.

dwzg · 2023-04-01T17:30:55Z

Haven't had a crash again with the most recent bugfix branch. Even a print that would always trigger the crash at a specific point is now running fine on my SKR E3 Turbo.

PendulumPit · 2023-04-02T00:48:50Z

For anyone following this issue / experienced crashes, please download bugfix-2.1.x to test with the latest code and let us know if you're still having this issue.

Be sure to attach a ZIP file containing your Configuration.h, Configuration_adv.h, and G-code in your reply.

I will give it a go. I had the issue when testing awhile back. Thanks to the Team for your hard work. 👍

SpannMagoo · 2023-04-02T02:47:32Z

Crashed right when its about to do purge line. Creality 4.2.7 Ender 3
Configuration_4.2.7.zip
ringing_tower_56m_0.20mm_200C_PLA-2.0.zip

thisiskeithb · 2023-04-02T02:52:07Z

Crashed right when its about to do purge line. Creality 4.2.7 Ender 3 Configuration_4.2.7.zip

Can you attach the g-code you were printing with? Which PIO environment did you use?

SpannMagoo · 2023-04-02T14:50:45Z

Posted in same post. Used vscode, Auto Build Marlin.

tombrazier · 2023-04-02T17:00:48Z

@SpannMagoo Can you check whether it still happens in my new test branch?

https://github.com/tombrazier/Marlin/tree/lafreeze2

SpannMagoo · 2023-04-02T20:51:41Z

OK, ran the newest branch as requested. I ran 3 tests and failed on the 2nd test at the purge line. See below for configs and the .gcode
Configuration_LatestPR4.2.7.zip
ringing_tower_56m_0.20mm_200C_PLA-3.0.zip

SpannMagoo · 2023-04-04T01:33:22Z

Also just noticed it only does the first 3 points when doing the g29 and stops.

tombrazier · 2023-04-04T10:51:25Z

Additional info: @SpannMagoo tells me that when the hang occurs, it is an M112 halt, complete with a screen saying "M112 Shutdown". This is not the same bug.

[Edit: looking at the code, it appears that the only way you can get there is if M112 is sent to the printer.]

SpannMagoo · 2023-04-04T13:29:20Z

Additional info: @SpannMagoo tells me that when the hang occurs, it is an M112 halt, complete with a screen saying "M112 Shutdown". This is not the same bug.

[Edit: looking at the code, it appears that the only way you can get there is if M112 is sent to the printer.]

This is interesting because I did not have the pi plugged in the last time.

tombrazier · 2023-04-05T16:23:47Z

@SpannMagoo reports (offline conversation) that the M112 bug began after the 18 March edition of bugfix. There were a lot of merges on 18 March and depending on quite where the snapshot occurred which SpannMagoo downloaded, this may or may not mean that there are a lot of other potential changes that could be related to this bug.

I think this probably warrants a new bug report.

As for this one, i.e. #25553, no one else is reporting anything here.

thisiskeithb · 2023-04-05T16:43:11Z

I think this probably warrants a new bug report.

@SpannMagoo: Please open a new bug report for your issue.

ThomasToka · 2023-04-05T18:52:36Z

I had not the time and capacity to test very much.

But: I did a fresh compile of bugfix-2.1.x downloaded 2 days ago. Enabled LA as always and was not suffering from this problem.

Where i am affected is if a activate this commit with my fork of a ender 3 s1 port: ca77850

Also the follow up fixes for this do not work. If i revert this commit all is good.

I had maybe 25 testers in the last 4 weeks for my firmware fork. All of them Ender 3 s1 Pro or Plus. Only 2 of them suffered also from this commit. Reverting this commit fixed the halts for both affected.

But it may be really something other that this here mentioned. I had on my printer no etchy sound or something. It just halted.

cybercatnet · 2023-04-11T20:23:11Z

I am using the latest build and on the fourth layer it starts printing slow until the print stops completely and the extruder motor starts making noises. The program stops working and does not respond.

tombrazier · 2023-04-11T20:44:04Z

@cybercatnet Could you elaborate a bit please? What do you mean by "the latest build"? And could you post your gcode? And is this repeatable?

cybercatnet · 2023-04-12T20:48:24Z

@cybercatnet Could you elaborate a bit please? What do you mean by "the latest build"? And could you post your gcode? And is this repeatable?

@tombrazier

The build commit 7369a6a

Example G-Code cap_large.zip

tombrazier · 2023-04-13T08:37:15Z

Next question: can I have your config please. And what printer / mainboard do you have?

And can you check whether the freeze still happens with my branch here.

tombrazier · 2023-04-13T08:49:07Z

@ThomasToka What actually is the difference between your fork and upstream Marlin? Looking at your branch MARLIN-E3S1PRO-FORK-BYTT it appears to be identical.

And could you post your config and platformio.ini please (I realise you might already have done so above but I want the latest and don't have the capacity to search the conversation history).

And do you know whether the reported bugs are on F1, F4 or both?

ThomasToka · 2023-04-13T10:44:00Z

@tombrazier i have rebuild my sources last week on the actual sources and ported the creality code. i will release the commits in the next 30 days cause i have to clean up a bit before public release. with this newer sources the error did not occure once.

it did only occur with two users. both ender3s1pro or plus with creality mainboards. the build it occured was the build before this i used. this was based on the synman release of the creality port. https://github.com/synman/Marlin/tree/bugfix-2.1.x i is reproducable if you add the mentioned commit to this. ca77850

ThomasToka · 2023-04-13T10:47:50Z

Configs:
configuartions.zip
Platformio:
platformio.zip

cybercatnet · 2023-04-19T23:53:43Z

Next question: can I have your config please. And what printer / mainboard do you have?

And can you check whether the freeze still happens with my branch here.

@tombrazier, sorry for the delay, I tried the version that is in your lafreeze2 branch, the crash problem remains the same, it fails at the same moment, but also specifically the motors brake at different times during printing (this did not happen in the Master branch). I hope it helps, anything let me know.

Printer P3Steel, Arduino Mega2560
Config lafreeze2.zip
Config commit 7369a6a.zip

thinkyhead · 2023-05-11T23:13:12Z

@cybercatnet — Are you still seeing such issues with the current bugfix-2.1.x? Is there some earlier point in time between the release of 2.1.2 and today where the problem does not exist?

cybercatnet · 2023-05-17T00:22:29Z

@cybercatnet — Are you still seeing such issues with the current bugfix-2.1.x? Is there some earlier point in time between the release of 2.1.2 and today where the problem does not exist?

@thinkyhead
Testing the build commit 3136435
Is not crashing but the motors stop some miliseconds, like 250ms, in random moments (short travel times?).
I dont know how to see the commit number in the files for the build where the problem does not exist, the download date was march 7 21:23.
Config commit 3136435.zip

github-actions · 2023-07-16T01:56:06Z

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

thisiskeithb added Bug: Potential ? C: Motion F: Linear Advance labels Mar 23, 2023

thisiskeithb added Bug: Confirmed ! and removed Bug: Potential ? labels Mar 24, 2023

tombrazier mentioned this issue Mar 24, 2023

Fix print hang with LA #25557

Merged

thisiskeithb closed this as completed Apr 5, 2023

SpannMagoo mentioned this issue Apr 5, 2023

M112 Shutdown: Printer Halted right before purge line start #25640

Closed

1 task

tombrazier mentioned this issue Apr 17, 2023

Further improvement to prevent potential LA related hang #25696

Merged

paulorlyk mentioned this issue May 9, 2023

G1 with E hangs after disabling CLASSIC_JERK if LIN_ADVANCE enabled #25797

Closed

1 task

github-actions bot locked and limited conversation to collaborators Jul 16, 2023

[BUG] Crash during Print, LA, IS, Bugfix.2.1x #25553

[BUG] Crash during Print, LA, IS, Bugfix.2.1x #25553

Comments

jesterhead82 commented Mar 23, 2023 • edited Loading

Did you test the latest bugfix-2.1.x code?

Bug Description

Bug Timeline

Expected behavior

Actual behavior

Steps to Reproduce

Version of Marlin Firmware

Printer model

Electronics

Add-ons

Bed Leveling

Your Slicer

Host Software

Don't forget to include

Additional information & file uploads

ThomasToka commented Mar 23, 2023 • edited Loading

ThomasToka commented Mar 23, 2023

thisiskeithb commented Mar 23, 2023

tombrazier commented Mar 23, 2023

tombrazier commented Mar 23, 2023

ThomasToka commented Mar 23, 2023

ThomasToka commented Mar 23, 2023

jesterhead82 commented Mar 23, 2023

tombrazier commented Mar 23, 2023

ThomasToka commented Mar 23, 2023

PendulumPit commented Mar 24, 2023

thisiskeithb commented Mar 24, 2023

tombrazier commented Mar 24, 2023

ThomasToka commented Mar 24, 2023

ThomasToka commented Mar 24, 2023

ThomasToka commented Mar 24, 2023

ThomasToka commented Mar 24, 2023

tombrazier commented Mar 24, 2023

ThomasToka commented Mar 24, 2023

tombrazier commented Mar 24, 2023

tombrazier commented Mar 24, 2023

ThomasToka commented Mar 24, 2023

tombrazier commented Mar 24, 2023

ThomasToka commented Mar 24, 2023 • edited Loading

vovodroid commented Apr 1, 2023

dwzg commented Apr 1, 2023

PendulumPit commented Apr 2, 2023 • edited Loading

SpannMagoo commented Apr 2, 2023 • edited Loading

thisiskeithb commented Apr 2, 2023 • edited Loading

SpannMagoo commented Apr 2, 2023

tombrazier commented Apr 2, 2023

SpannMagoo commented Apr 2, 2023 • edited Loading

SpannMagoo commented Apr 4, 2023

tombrazier commented Apr 4, 2023 • edited Loading

SpannMagoo commented Apr 4, 2023

tombrazier commented Apr 5, 2023

thisiskeithb commented Apr 5, 2023

ThomasToka commented Apr 5, 2023

cybercatnet commented Apr 11, 2023

tombrazier commented Apr 11, 2023

cybercatnet commented Apr 12, 2023

tombrazier commented Apr 13, 2023

tombrazier commented Apr 13, 2023

ThomasToka commented Apr 13, 2023

ThomasToka commented Apr 13, 2023 • edited Loading

cybercatnet commented Apr 19, 2023 • edited Loading

thinkyhead commented May 11, 2023

cybercatnet commented May 17, 2023 • edited Loading

github-actions bot commented Jul 16, 2023

jesterhead82 commented Mar 23, 2023 •

edited

Loading

Did you test the latest `bugfix-2.1.x` code?

ThomasToka commented Mar 23, 2023 •

edited

Loading

ThomasToka commented Mar 24, 2023 •

edited

Loading

PendulumPit commented Apr 2, 2023 •

edited

Loading

SpannMagoo commented Apr 2, 2023 •

edited

Loading

thisiskeithb commented Apr 2, 2023 •

edited

Loading

SpannMagoo commented Apr 2, 2023 •

edited

Loading

tombrazier commented Apr 4, 2023 •

edited

Loading

ThomasToka commented Apr 13, 2023 •

edited

Loading

cybercatnet commented Apr 19, 2023 •

edited

Loading

cybercatnet commented May 17, 2023 •

edited

Loading