Antonio,
On 08/10/2022 09:33, Antonio Borneo wrote:
I would like to go ahead with merge for v0.12.0-rc2, but I have not
clear if you agree merging this series:
- https://review.openocd.org/7230
"target: re-examine before arp_waitstate in ocd_process_reset_inner"
- https://review.openocd.org/7229
"target/hla_target: try to re-examine under reset in hl_assert_reset()"
- https://review.openocd.org/7228
"target/cortex_m: try to re-examine under reset in cortex_m_assert_reset()"
- https://review.openocd.org/6745
"target/cortex_m: make reset robust again"
Yes, let's merge them if you don't have objections.
nor if you plan to revert "target: reset target examined flag if
target::examine() fails" with
https://review.openocd.org/6753
can you please let me know, or go directly to merge them?
Currently I know about two regressions (one in cortex_m and independent
one in hla/waitstate) caused by
6548: target: reset target examined flag if target::examine() fails |
https://review.openocd.org/c/openocd/+/6548
They should be mostly fixed by the series #6745, 7228-7230.
But how can we be sure that there is no other regressions - not yet
discovered - in other parts of code, in other targets etc?
I see both pro: #6753 gives some safety from possible regressions from
the change target_examined behaviour
and con: OpenOCD git master has been with #6548 for long time, we can
also break something with #6753 in rc2.
But honestly it's not my fault that nobody cared since November 2021.
I'm slightly for merging #6753 to rc2, this is IMO the better from two
bad options.
Please decide.
Tom