New v0.5 simtrace firmware release

[Myonium]

Hi Harald

Thank you for the new release.

So far everything works fine for me except for the lost byte issue for fast communication such as for the “Gemalto .NET v2.0” card. (Issue: bytes get lost between UART chunks.)

However this is easily fixed just by reducing the rctx buffer size (same as in the last release). I set the buffer size “rctx->size” to 16 Byte.

A quick and dirty fix is:
diff --git a/firmware/src/simtrace/iso7816_uart.c b/firmware/src/simtrace/iso7816_uart.c
index 4169ab9..f1db406 100644
--- a/firmware/src/simtrace/iso7816_uart.c
+++ b/firmware/src/simtrace/iso7816_uart.c
@@ -536,7 +536,8 @@ static void process_byte(struct iso7816_3_handle *ih, u_int8_t byte)
       rctx->data[rctx->tot_len] = byte;
       rctx->tot_len++;

-       if (rctx->tot_len >= rctx->size || ih->rctx_must_be_sent) {
+       /* if (rctx->tot_len >= rctx->size || ih->rctx_must_be_sent) { */
+       if (rctx->tot_len >= 16 || ih->rctx_must_be_sent) {
               ih->rctx_must_be_sent = 0;
               send_rctx(ih);
       }

Regards,

Ben
On Mar 23, 2012, at 12:12 AM, Harald Welte wrote:

> Hi all!
> 
> Just in time before OsmoDevCon, I managed to releaes a new version of
> the SIMtrace firmware.
> 
> The list of changes is quite extensive and should address a number of
> the reliability problems that people have experienced.
> 
> The changes include:
> * don't activate pass-through and LDO supply for the sim card at
>  the same time, leading in power leaking from simtrace into phones
> * fix a watchdog timer misconfiguration leading to occasional watchdog
>  resets
> * fix detection of sim insert/removal
> * better VCC_PHONE detection
> * reduce the amount of debug logging on the serial console and replace
>  it by statistics (count of bytes, overruns, parity errors, etc)
> * statistics can be read from the UART by pressing 't'.
> * no longer crash the simtrace firmware if no simtrace host program is
>  running
> 
> Using this v0.5 firmware and a pretty aggressiv program causing lots of
> traffic (hunz' simdump) on a cm3121 reader at card Fi(9) Di(4) ratio 64
> didn't cause any lost bytes or SIMtrace resets for me anymore.
> 
> Any feedback is appreciated.
> 
> Regards,
> 	Harald
> -- 
> - Harald Welte <laforge at gnumonks.org>           http://laforge.gnumonks.org/
> ============================================================================
> "Privacy in residential applications is a desirable marketing option."
>                                                  (ETSI EN 300 175-7 Ch. A6)
> <main_simtrace_v05.bin><main_simtrace_v05.samba>