spec: buffers alignment

buf_in and buf_out need to be aligned, in order not to have bug in msg_pack and msg_unpack functions. This issue should be better fixed than what done with the present patch.

spec: buffers alignment
buf_in and buf_out need to be aligned, in order not to have bug in msg_pack and msg_unpack functions. This issue should be better fixed than what done with the present patch.
495a4e85 · Aurelio Colosimo · fd4dbce4 · 495a4e85 · 495a4e85
Commit 495a4e85 authored Jul 06, 2012 by Aurelio Colosimo
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 3 deletions

main-loop.c arch-spec/main-loop.c +3 -1

spec-socket.c arch-spec/spec-socket.c +3 -2

No files found.
--- a/arch-spec/main-loop.c
+++ b/arch-spec/main-loop.c
@@ -28,7 +28,9 @@ void spec_main_loop(struct pp_instance *ppi)
 	 */
 	delay_ms = pp_state_machine(ppi, NULL, 0);
 	while (1) {
-		unsigned char packet[1500];
+		unsigned char _packet[1500];
+		/* FIXME Alignment */
+		unsigned char *packet = _packet + 2;
 		/* Wait for a packet or for the timeout */
 		while (delay_ms && !minic_poll_rx()) {
 			spec_udelay(1000);

--- a/arch-spec/spec-socket.c
+++ b/arch-spec/spec-socket.c
@@ -77,8 +77,9 @@ int spec_net_init(struct pp_instance *ppi)
 	ppi->buf_out = buffer_out;
 	ppi->buf_out = PROTO_PAYLOAD(ppi->buf_out);

-	//UDP only for now
-	pp_printf("spec_net_init UDP\n");
+	ppi->buf_out+= 4 - (((int)ppi->buf_out) % 4);
+	/* FIXME Alignment */
+	pp_printf("spec_net_init ETH %x\n",ppi->buf_out);
 	spec_open_ch(ppi);

 	return 0;