The FPGA trace loop was the last OLD reply on the device outside the two the bootrom also serves. It stayed OLD because the DMA double-buffer was sized to the frame payload and an NG header did not obviously fit in front of it. DMA straight into chunk->data of a download_chunk_t instead, so a filled buffer is already a complete NG payload and needs no copy. Chunking now follows DOWNLOAD_CHUNK_MAX and scales with PM3_CMD_DATA_SIZE. The terminator carries download_done_t like the other bulk downloads. No client change needed, dl_it already had the NG branch. Two fixes fall out of it: FPGA_TRACE_SIZE is 3072, an exact multiple of 512 but not of DOWNLOAD_CHUNK_MAX. Each transfer is now armed for exactly the bytes still expected - arming a full chunk for the short last one would spin in FPGA_SSC_DMA_RX_Done() forever. This also drops the stray extra DMA the old loop left armed. get_tosend() moved after FpgaDownloadAndGo(). The loader calls BigBuf_free(), which nulls s_toSend.buf, then reuses that same region for its decompression ring buffer - the old code captured the pointer before the free and only worked because the loader was done with it in time. 3072 bytes goes from 6 OLD frames to 7 NG frames at PM3_CMD_DATA_SIZE 512, and would be 5 at 688.
23 KiB
New frame format documentation
This document is primarily intended for developers only.
A major change is the support of variable length frames between host and Proxmark3.
This is a step especially important for usage over FPC/USART/BT.
Table of Contents
- New frame format documentation
- Table of Contents
Old format
^Top
Previously, frames were, in both directions like this:
uint64_t cmd;
uint64_t arg[3];
union {
uint8_t asBytes[PM3_CMD_DATA_SIZE];
uint32_t asDwords[PM3_CMD_DATA_SIZE / 4];
} d;
with PM3_CMD_DATA_SIZE = 512 and there was no API abstraction, everybody was forging/parsing these frames.
These two structs now use PM3_CMD_DATA_SIZE_OLD, pinned at 512 independently of PM3_CMD_DATA_SIZE:
the bootloader only speaks OLD, so growing them would break flashing against every deployed bootrom.
So the frame size was fixed, 544 bytes, even for simple ACKs.
When snooping the USB transfers, we can observe the host is sending 544b Bulk USB frames while the Proxmark3 is limited by its internal buffers and is sending 128b, 128b, 128b, 128b, 32b, so in total 5 packets.
New format
^Top
Even if we make the payload part variable in the old format, we've still a minimum of 32 bytes per frame with fields arbitrarily large. So we designed a new format from scratch:
For commands being sent to the Proxmark3:
uint32_t magic;
uint16_t length : 15;
bool ng : 1;
uint16_t cmd;
uint8_t data[length];
uint16_t crc;
magic: arbitrary magic (PM3a) to help re-sync if neededlength: length of the variable payload, 0 if none, max 512 (PM3_CMD_DATA_SIZE) for now.ng: flag to tell if the data is following the new format (ng) or the old one, see transition notes belowcmd: as previously, on 16b as it's enoughdata: variable length payloadcrc: either an actual CRC (crc14a) or a Magic placeholder (a3)
For responses from the Proxmark3:
uint32_t magic;
uint16_t length : 15;
bool ng : 1;
int8_t status;
int8_t reason;
uint16_t cmd;
uint8_t data[length];
uint16_t crc;
magic: arbitrary magic (PM3b) to help re-sync if neededlength: length of the variable payload, 0 if none, max 512 (PM3_CMD_DATA_SIZE) for now.ng: flag to tell if the data is following the new format (ng) or the old one, see transition notes belowstatus: a field to send back the status of the command executionreason: details about what the status indicates for the specified commandcmd: as previously, on 16b as it's enoughdata: variable length payloadcrc: either an actual CRC (crc14a) or a Magic placeholder (b3)
We used to send an anonymous ACK, now we're replying with the corresponding command name and a status.
CRC is optional and on reception, the magic a3/b3 is accepted as placeholder. If it's different then it's checked as a CRC.
By default CRC is used over USART and is disabled over USB, on both directions.
Internal structures used to handle these packets are:
- PacketCommandNGPreamble
- PacketCommandNGPostamble
- PacketCommandNGRaw
- PacketResponseNGPreamble
- PacketResponseNGPostamble
- PacketResponseNGRaw
But they are abstracted from the developer view with a new API. See below.
Transition
^Top
Status: the C client and the ARM firmware are converted.
The client has no SendCommandOLD or SendCommandMIX call sites left. On the
device only three legacy replies remain, and all three are deliberate:
| Site | Why it stays |
|---|---|
armsrc/appmain.c — CMD_READ_MEM_DOWNLOADED and its CMD_ACK terminator |
also served by bootrom.c, which only speaks OLD |
armsrc/appmain.c — CMD_DEVICE_INFO |
same, see bootrom.c |
SendCommandBL marks the frames that must stay OLD because the bootloader
serves them. Do not "convert" those.
These sites chunk by PM3_CMD_DATA_SIZE_OLD, not PM3_CMD_DATA_SIZE: reply_old
clamps its payload to the OLD size, so a sender chunking by the NG size would build
oversized chunks, have them truncated on the wire, and still announce the full length
in oldarg[1].
armsrc/hfsnoop.c — CMD_FPGAMEM_DOWNLOADED used to be on this list because its
FPGA trace loop is a DMA double-buffer and an NG header did not obviously fit
alongside a full DMA buffer. It is converted: the DMA writes straight into
chunk->data of a download_chunk_t, so a filled buffer is already a complete NG
payload. Note the loop arms each transfer for exactly the bytes still expected -
FPGA_TRACE_SIZE is not a multiple of DOWNLOAD_CHUNK_MAX, and arming a full
chunk for the short last one waits forever on bytes the FPGA never sends.
PacketResponseNG.oldarg[] is likewise almost gone on the client. The only
readers left are client/src/flash.c and client/src/proxmark3.c, both talking
to the bootrom over SendCommandBL. The Lua binding no longer serialises the
oldargs into the response it hands scripts.
The old frames are still supported by the transport, so PacketCommandOLD and
PacketResponseOLD remain, abstracted from the developer view by the new API.
New format API
^Top
So the new API is a merge of the old and the new frame formats, to ensure a smooth transition.
The boolean ng indicates if the structure is storing data from the old or the new format.
Old format can come from either old 544b frames or mixed frames (variable length but still with oldargs).
After the full transition, we might remove the fields oldarg and ng.
PacketCommandNG and PacketResponseNG are the structs used by the API, as seen in a previous section, there are other variable-sized packed structs specifically for the data transmission.
typedef struct {
uint16_t cmd;
uint16_t length;
uint32_t magic; // NG
uint16_t crc; // NG
uint64_t oldarg[3]; // OLD
union {
uint8_t asBytes[PM3_CMD_DATA_SIZE];
uint32_t asDwords[PM3_CMD_DATA_SIZE / 4];
} data;
bool ng; // does it store NG data or OLD data?
} PacketCommandNG;
typedef struct {
uint16_t cmd;
uint16_t length;
uint32_t magic; // NG
int8_t status; // NG
int8_t reason; // NG
uint16_t crc; // NG
uint64_t oldarg[3]; // OLD
union {
uint8_t asBytes[PM3_CMD_DATA_SIZE];
uint32_t asDwords[PM3_CMD_DATA_SIZE / 4];
} data;
bool ng; // does it store NG data or OLD data?
} PacketResponseNG;
On the client, for sending frames
^Top
(client/comms.c)
void SendCommandNG(uint16_t cmd, uint8_t *data, size_t len);
void SendCommandBL(uint64_t cmd, uint64_t arg0, uint64_t arg1, uint64_t arg2, void *data, size_t len);
void SendCommandOLD(uint64_t cmd, uint64_t arg0, uint64_t arg1, uint64_t arg2, void *data, size_t len);
SendCommandNG is what every command uses. Put everything in an ad-hoc PACKED
struct in the data field.
SendCommandBL is for Bootloader-related activities, see the Bootrom section. It
is a thin alias over SendCommandOLD and exists so that the frames which must
stay OLD are visibly tagged as such. SendCommandOLD has no other caller.
SendCommandMIX and PM3_CMD_DATA_SIZE_MIX have been removed. MIX was the
halfway house: an NG-framed packet that still carried the three 64 bit oldargs in
the first 24 bytes of the payload. Nothing uses it any more, and the client can no
longer construct one.
Internally these functions prepare the frame and call uart_communication which
calls uart_send.
On the Proxmark3, for receiving frames
^Top
(armsrc/appmain.c)
PacketCommandNG
AppMain calls receive_ng(common/cmd.c) which calls usb_read_ng/usart_read_ng to get a PacketCommandNG, then passes it to PacketReceived.
(no matter if it's an old frame or a new frame, check PacketCommandNG.ng field to know if there are oldargs)
PacketReceive is the commands broker.
Old handlers will still find their stuff in PacketCommandNG.oldarg field.
On the Proxmark3, for sending frames
^Top
(common/cmd.c)
int reply_old(uint64_t cmd, uint64_t arg0, uint64_t arg1, uint64_t arg2, const void *data, size_t len);
int reply_ng(uint16_t cmd, int8_t status, const uint8_t *data, size_t len);
int reply_reason(uint16_t cmd, int8_t status, int8_t reason, const uint8_t *data, size_t len);
reply_mix has been removed. reply_old survives only for the handlers the
bootrom also serves; note it always transmits sizeof(PacketResponseOLD), 544
bytes, no matter what len says, whereas reply_ng sends exactly what you give
it.
A handler now simply validates the length and answers on its own opcode:
case CMD_FOOBAR: {
if (packet->length != sizeof(foobar_t)) {
reply_ng(CMD_FOOBAR, PM3_EINVARG, NULL, 0);
break;
}
foobar_t *payload = (foobar_t *)packet->data.asBytes;
...
reply_ng(CMD_FOOBAR, PM3_SUCCESS, resp, resplen);
break;
}
During the transition some handlers carried a if (packet->ng) { ... } else { ... }
dual-mode branch so converted and unconverted callers could coexist. None remain;
a frame arriving without the ng bit is now answered with PM3_EINVARG.
On the client, for receiving frames
^Top
(client/comms.c)
WaitForResponseTimeout ⇒ PacketResponseNG
uart_communication calls uart_receive and create a PacketResponseNG, then passes it to PacketResponseReceived.
PacketResponseReceived treats it immediately (prints) or stores it with storeReply.
Commands do WaitForResponseTimeoutW (or dl_it) which uses getReply to fetch responses.
API transition
^Top
In short, to move from one format to the other, we need for each command:
- (client TX)
SendCommandOLD⇒SendCommandNG(with all stuff in ad-hoc PACKED structs indatafield) - (pm3 RX)
PacketCommandNGparsing, fromoldargto only thedatafield - (pm3 TX)
reply_old⇒reply_ng(with all stuff in ad-hoc PACKED structs indatafield) - (client RX)
PacketResponseNGparsing, fromoldargto only thedatafield
A MIX frame was the historical halfway house (SendCommandMIX / reply_mix).
Do not add new ones: the client is free of them and every device handler that
mattered has been converted.
Practical notes from converting the tree
- Reply with the command's own opcode, never
CMD_ACK. An anonymous ACK is what forced hacks like iCLASS puttingCMD_HF_ICLASS_SIMULATEinsidearg0of its own reply so the client could tell replies apart. - A success flag belongs in the NG
statusfield, not in a data byte, and a second discriminator can usereply_reason(). - Validate
packet->lengthbefore casting, on both sides. Several converted commands previously read past the end of a short frame. PACKEDis not free. It sets the struct's alignment to 1, so taking the address of any member wider than a byte trips-Werror=address-of-packed-member, and on the ARM7TDMI an unaligned 32 bit load silently rotates rather than faulting. Pack a struct only to remove padding: ifsizeofis the same either way, leave it unpacked. Compareepa_replay_t(packed, itsuint16_ts sit at odd offsets) withepa_result_tandt55xx_setconfig_t(not packed, they already tile and their members get addressed).PM3_CMD_DATA_SIZE_MIXis a MIX-only constant. Any chunked transfer that moves to NG must size its chunks asPM3_CMD_DATA_SIZE - sizeof(payload_t)instead.reply_oldalways transmitssizeof(PacketResponseOLD), 544 bytes, whateverlensays.reply_ngsends only what you give it.- A command can answer more than once.
CMD_HF_ISO14443A_READERwithISO14A_CONNECT | ISO14A_RAWreplies twice, select then raw exchange; the client must consume both.
Bulk downloads
CMD_DOWNLOAD_BIGBUF, CMD_DOWNLOAD_EML_BIGBUF, CMD_SPIFFS_DOWNLOAD and
CMD_FLASHMEM_DOWNLOAD use download_req_t / download_chunk_t /
download_done_t (see include/pm3_cmd.h). The chunk header carries only the
offset, since the frame length gives the size, and the transfer ends with a
download_done_t answered on the original CMD_DOWNLOAD_* opcode rather than an
anonymous ACK. dl_it() in client/src/comms.c still understands the OLD chunk
format because CMD_READ_MEM_DOWNLOAD is served by the bootrom.
Bootrom
^Top
Bootrom code will still use the old frame format to remain compatible with other repos supporting the old format and because it would hardly gain anything from the new format:
- almost all frames convey 512b of payload, so difference in overhead is negligible
- bringing flash over usart sounds risky and would be terribly slow anyway (115200 bauds vs. 7M bauds).
SendCommandBL is the same as SendCommandOLD with a different name to be sure not to migrate it.
On the Proxmark3, for receiving frames
^Top
(bootrom/bootrom.c)
usb_read (common/usb_cdc.c) ⇒ UsbPacketReceived (bootrom.c)
⇒ CMD_DEVICE_INFO / CMD_START_FLASH / CMD_FINISH_WRITE / CMD_HARDWARE_RESET
also usb_enable, usb_disable (common/usb_cdc.c)
On the Proxmark3, for sending frames
^Top
(bootrom/bootrom.c)
reply_old (bootrom.c) ⇒ usb_write (common/usb_cdc.c)
also usb_enable, usb_disable (common/usb_cdc.c)
On the client, for sending frames
^Top
Therefore, the flasher client (client/flasher.c + client/flash.c) must still use these old frames.
It uses a few commands in common with current client code:
OpenProxmark
CloseProxmark
SendCommandOLD
⇒ CMD_DEVICE_INFO / CMD_START_FLASH / CMD_FINISH_WRITE / CMD_HARDWARE_RESET
On the client, for receiving frames
^Top
As usual, old frames are still supported
WaitForResponseTimeout ⇒ PacketResponseNG
New usart RX FIFO
^Top
USART code has been rewritten to cope with unknown size packets.
- using USART full duplex with double DMA buffer on RX & TX
- using internal FIFO for RX
usart_init:
- USART is activated all way long from usart_init(), no need to touch it in RX/TX routines:
pUS1->US_PTCR = AT91C_PDC_RXTEN | AT91C_PDC_TXTEN
usart_writebuffer_sync:
- still using DMA but accepts arbitrary packet sizes
- removed unneeded memcpy
- wait for DMA buffer to be treated before returning, therefore "sync"
- we could make an async version but caller must be sure the DMA buffer remains available!
- as it's sync, no need for next DMA buffer
usart_read_ng:
- user tells expected packet length
- relies on usart_rxdata_available to know if there is data in our FIFO buffer
- fetches data from our FIFO
- dynamic number of tries (depending on FPC speed) to wait for asked data
usart_rxdata_available:
- polls usart_fill_rxfifo
- returns number of bytes available in our FIFO
usart_fill_rxfifo:
- if next DMA buffer got moved to current buffer (
US_RNCR == 0), it means one DMA buffer is full- transfer current DMA buffer data to our FIFO
- swap to the other DMA buffer
- provide the emptied DMA buffer as next DMA buffer
- if current DMA buffer is partially filled
- transfer available data to our FIFO
- remember how many bytes we already copied to our FIFO
Timings
^Top
Reference (before new format):
linux usb: #db# USB Transfer Speed PM3 ⇒ Client = 545109 Bytes/s
On a Windows VM:
proxspace usb: #db# USB Transfer Speed PM3 ⇒ Client = 233998 Bytes/s
Over USART:
(common/usart.h)
USART_BAUD_RATE defined there
9600: #db# USB Transfer Speed PM3 ⇒ Client = 934 Bytes/s
115200: #db# USB Transfer Speed PM3 ⇒ Client = 11137 Bytes/s
460800: #db# USB Transfer Speed PM3 ⇒ Client = 43119 Bytes/s
linux usb: #db# USB Transfer Speed PM3 ⇒ Client = 666624 Bytes/s (equiv. to ~7Mbaud)
(pm3_cmd.h)
Receiving from USART need more than 30ms as we used on USB else we get errors about partial packet reception
FTDI 9600 hw status ⇒ we need 20ms
FTDI 115200 hw status ⇒ we need 50ms
FTDI 460800 hw status ⇒ we need 30ms
BT 115200 hf mf fchk --1k -f file.dic ⇒ we need 140ms
# define UART_FPC_CLIENT_RX_TIMEOUT_MS 200
# define UART_USB_CLIENT_RX_TIMEOUT_MS 20
# define UART_NET_CLIENT_RX_TIMEOUT_MS 500
# define UART_TCP_LOCAL_CLIENT_RX_TIMEOUT_MS 40
# define UART_UDP_LOCAL_CLIENT_RX_TIMEOUT_MS 20
This goes to uart_posix.c timeval struct
and uart_win32.c serial_port_windows struct
It starts at UART_FPC_CLIENT_RX_TIMEOUT_MS and once we detect we're working over USB it's reduced to UART_USB_CLIENT_RX_TIMEOUT_MS.
The timeout is configurable by the hw timeout command (since v4.17140).
Add automatically some communication delay in the WaitForResponseTimeout & dl_it timeouts.
Only when using FPC, timeout = 2* empirically measured delay (FTDI cable).
Empirically measured delay (FTDI cable) with "hw ping -l 512" :
usb ⇒ 6.. 32ms
460800 ⇒ 40.. 70ms
9600 ⇒ 1100..1150ms
(client/comms.c)
static size_t communication_delay(void) {
if (conn.send_via_fpc_usart) // needed also for Windows USB USART??
return 2 * (12000000 / uart_speed);
return 100;
}
Because some commands send a lot of frames before finishing (hw status, lf read,...),
WaitForResponseTimeout & dl_it timeouts are reset at each packet reception,
so timeout is actually counted after latest received packet,
it doesn't depend anymore on the number of received packets.
It was needed to tune pm3 RX usart maxtry :
(common/usart.c)
uint32_t usart_read_ng(uint8_t *data, size_t len) {
// Empirical max try observed: 3000000 / USART_BAUD_RATE
// Let's take 10x
uint32_t tryconstant = 0;
#ifdef USART_SLOW_LINK
// Experienced up to 13200 tries on BT link even at 460800
tryconstant = 50000;
#endif
uint32_t maxtry = 10 * (3000000 / USART_BAUD_RATE) + tryconstant;
DbpStringEx using reply_old:
time client/proxmark3 -p /dev/ttyACM0 -c "hw status"
2.52s
time client/proxmark3 -p /dev/ttyUSB0 -b 460800 -c "hw status"
3.03s
time client/proxmark3 -p /dev/ttyUSB0 -b 115200 -c "hw status"
4.88s
time client/proxmark3 -p /dev/ttyUSB0 -b 9600 -c "hw status"
26.5s
DbpStringEx using reply_mix:
time client/proxmark3 -p /dev/ttyUSB0 -b 9600 -c "hw status"
7.08s
DbpStringEx using reply_ng:
time client/proxmark3 -p /dev/ttyACM0 -c "hw status"
2.10s
time client/proxmark3 -p /dev/ttyUSB0 -b 460800 -c "hw status"
2.22s
time client/proxmark3 -p /dev/ttyUSB0 -b 115200 -c "hw status"
2.43s
time client/proxmark3 -p /dev/ttyUSB0 -b 9600 -c "hw status"
5.75s
time client/proxmark3 -p /dev/ttyUSB0 -b 9600 -c "lf read"
50.38s
time client/proxmark3 -p /dev/ttyUSB0 -b 115200 -c "lf read"
6.28s
time client/proxmark3 -p /dev/ttyACM0 -c "mem dump -f foo_usb"
1.48s
time client/proxmark3 -p /dev/ttyUSB0 -b 115200 -c "mem dump -f foo_fpc"
25.34s
Sending multiple commands can still be slow because it waits regularly for incoming RX frames and the timings are quite conservative because of BT (see struct timeval timeout in uart_posix.c, now at 200ms). When one knows there is no response to wait before the next command, he can use the same trick as in the flasher:
// fast push mode
conn.block_after_ACK = true;
some loop {
if (sending_last_command)
// Disable fast mode
conn.block_after_ACK = false;
SendCommandOLD / SendCommandMix
if (WaitForResponseTimeout(CMD_ACK, &resp, some_timeout) == false) {
....
conn.block_after_ACK = false;
return PM3_ETIMEOUT;
}
}
return PM3_SUCCESS;
Or if it's too complex to determine when we're sending the last command:
// fast push mode
conn.block_after_ACK = true;
some loop {
SendCommandNG
if (WaitForResponseTimeout(CMD_FOOBAR, &resp, some_timeout) == false) {
....
conn.block_after_ACK = false;
return PM3_ETIMEOUT;
}
}
// Disable fast mode and send a dummy command to make it effective
conn.block_after_ACK = false;
SendCommandNG(CMD_PING, NULL, 0);
WaitForResponseTimeout(CMD_ACK, NULL, 1000);
return PM3_SUCCESS;
Reference frames
^Top
For helping debug...
- OLD command and reply packets are 544 bytes.
- NG & MIX command packets are between 10 and 522 bytes.
- NG & MIX reply packets are between 12 and 524 bytes.
On linux USB
- sent packets can be 544
- received packets are max 128, so 544 = 128+128+128+128+32
On linux UART (FTDI)
- sent packets are max 256, so 544 = 256+256+32
- received packets are max 512, so 544 = 512+32
hw ping (old version, mix reply)
TestProxmark: SendCommandOLD(CMD_PING, 0, 0, 0, NULL, 0);
->544=0901000000000000000000000000000000000000000000000000000000000000 -> OLD
CMD_PING: reply_mix(CMD_ACK, reply_via_fpc, 0, 0, 0, 0);
<-36=504d336218000000ff0000000000000000000000000000000000000000000000 <- MIX
hw ping (intermediate version using MIX)
CmdPing SendCommandMIX(CMD_PING, 0, 0, 0, NULL, 0);
->34=504d336118000901000000000000000000000000000000000000000000000000 -> MIX
CMD_PING reply_mix(CMD_ACK, reply_via_fpc, 0, 0, 0, 0);
<-36=504d336218000000ff0000000000000000000000000000000000000000000000 <- MIX
hw ping (current NG version)
CmdPing SendCommandNG(CMD_PING, data, len);
->10=504d3361008009016133 -> NG
CMD_PING reply_ng(CMD_PING, PM3_SUCCESS, packet->data.asBytes, packet->length);
<-12=504d33620080000009016233 <- NG
hw ping -l 512 (NG)
CmdPing SendCommandNG(CMD_PING, data, len);
->522=504d336100820901000102030405060708090a0b0c0d0e0f1011121314151617 -> NG
CMD_PING reply_ng(CMD_PING, PM3_SUCCESS, packet->data.asBytes, packet->length);
<-128=504d3362008200000901000102030405060708090a0b0c0d0e0f101112131415 <- NG
<-128=767778797a7b7c7d7e7f808182838485868788898a8b8c8d8e8f909192939495
<-128=f6f7f8f9fafbfcfdfeff000102030405060708090a0b0c0d0e0f101112131415
<-128=767778797a7b7c7d7e7f808182838485868788898a8b8c8d8e8f909192939495
<-12=f6f7f8f9fafbfcfdfeff6233