ptka commented on pull request #5249:
URL: https://github.com/apache/incubator-nuttx/pull/5249#issuecomment-1014958870
I did some further investigations and the issue is now at the next place in
arp_table.c:
void arp_hdr_update(FAR struct net_driver_s *dev, FAR uint16_t *pipaddr,
FAR uint8_t *ethaddr)
{
in_addr_t ipaddr = net_ip4addr_conv32(pipaddr);
/* Update the ARP table */
Here it’s not an element of a struct, but a pointer to an uint16_t, which
gets now the problem.
So the solution with begin_packed_struct/end_packed_struct but it’s not
catching all issues with the unaligned d_buf.
Disassembling is not needed.
> Am 17.01.2022 um 23:39 schrieb Petro Karashchenko ***@***.***>:
>
>
> @pkarashchenko commented on this pull request.
>
> In drivers/usbdev/cdcecm.c
<https://github.com/apache/incubator-nuttx/pull/5249#discussion_r786318309>:
>
> > @@ -120,7 +120,7 @@ struct cdcecm_driver_s
> FAR struct usbdev_ep_s *epbulkout; /* Bulk OUT endpoint */
> uint8_t config; /* Selected configuration
number */
>
> - uint8_t pktbuf[CONFIG_NET_ETH_PKTSIZE +
> + aligned_data(2) uint8_t pktbuf[CONFIG_NET_ETH_PKTSIZE +
> @ptka <https://github.com/ptka> I believe you and unfortunately I do not
have tiny2040-composite board to try it. Is it possible for you to generate
disassembly for cdcecm_receive()? I would like to examine what instructions are
generated for if (BUF->type == HTONS(ETHTYPE_IP)) so it leads for hard fault.
>
> —
> Reply to this email directly, view it on GitHub
<https://github.com/apache/incubator-nuttx/pull/5249#discussion_r786318309>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAGUYCXWHHQO67C4MCTTPLLUWSLBFANCNFSM5MEWZEKA>.
> Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
> You are receiving this because you were mentioned.
>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
