From: "Slark Xiao" <slark_xiao@163.com>
To: "Gustavo A. R. Silva" <gustavo@embeddedor.com>
Cc: loic.poulain@oss.qualcomm.com, ryazanov.s.a@gmail.com,
johannes@sipsolutions.net, andrew+netdev@lunn.ch,
davem@davemloft.net, edumazet@google.com, kuba@kernel.org,
pabeni@redhat.com, gustavoars@kernel.org, netdev@vger.kernel.org,
linux-kernel@vger.kernel.org, linux-hardening@vger.kernel.org
Subject: Re:Re: [net] Revert "net: wwan: mhi_wwan_mbim: Avoid -Wflex-array-member-not-at-end warning"
Date: Thu, 22 Jan 2026 09:30:09 +0800 (CST) [thread overview]
Message-ID: <1eb019f4.1033.19be352bf0c.Coremail.slark_xiao@163.com> (raw)
In-Reply-To: <2fc0de60-6450-4ea3-957b-5de465a2313d@embeddedor.com>
At 2026-01-20 17:28:42, "Gustavo A. R. Silva" <gustavo@embeddedor.com> wrote:
>
>
>On 1/20/26 18:16, Slark Xiao wrote:
>>
>>
>> At 2026-01-20 15:51:57, "Gustavo A. R. Silva" <gustavo@embeddedor.com> wrote:
>>> Hi Slark,
>>>
>>> On 1/20/26 16:20, Slark Xiao wrote:
>>>> This reverts commit eeecf5d3a3a484cedfa3f2f87e6d51a7390ed960.
>>>>
>>>> This change lead to MHI WWAN device can't connect to internet.
>>>> I found a netwrok issue with kernel 6.19-rc4, but network works
>>>> well with kernel 6.18-rc1. After checking, this commit is the
>>>> root cause.
>>>
>>> Thanks for the report.
>>>
>>> Could you please apply the following patch on top of this revert,
>>> and let us know if the problem still manifests? Thank you!
>>>
>>> diff --git a/drivers/net/wwan/mhi_wwan_mbim.c b/drivers/net/wwan/mhi_wwan_mbim.c
>>> index 1d7e3ad900c1..a271a72fed63 100644
>>> --- a/drivers/net/wwan/mhi_wwan_mbim.c
>>> +++ b/drivers/net/wwan/mhi_wwan_mbim.c
>>> @@ -78,9 +78,12 @@ struct mhi_mbim_context {
>>>
>>> struct mbim_tx_hdr {
>>> struct usb_cdc_ncm_nth16 nth16;
>>> - struct usb_cdc_ncm_ndp16 ndp16;
>>> - struct usb_cdc_ncm_dpe16 dpe16[2];
>>> + __TRAILING_OVERLAP(struct usb_cdc_ncm_ndp16, ndp16, dpe16, __packed,
>>> + struct usb_cdc_ncm_dpe16 dpe16[2];
>>> + );
>>> } __packed;
>>> +static_assert(offsetof(struct mbim_tx_hdr, ndp16.dpe16) ==
>>> + offsetof(struct mbim_tx_hdr, dpe16));
>>>
>>> static struct mhi_mbim_link *mhi_mbim_get_link_rcu(struct mhi_mbim_context *mbim,
>>> unsigned int session)
>> This patch won't introduce previous problem.
>>
>
>Thanks for confirming this.
>
>I'll turn it into a proper patch.
>
>-Gustavo
Hi,
Are you planing to add this revert commit to your coming patch?
Actually it's urgent to revert this change since the release deadline of v6.19 is coming soon
and all my MHI WWAN device have been impacted.
I don't know why this commit has been tagged as "Not Applicable" and who did this.
I hope we can fix it regression before the official v6.19 release.
Thanks
next prev parent reply other threads:[~2026-01-22 1:31 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-01-20 7:20 [net] Revert "net: wwan: mhi_wwan_mbim: Avoid -Wflex-array-member-not-at-end warning" Slark Xiao
2026-01-20 7:51 ` Gustavo A. R. Silva
2026-01-20 9:16 ` Slark Xiao
2026-01-20 9:28 ` Gustavo A. R. Silva
2026-01-22 1:30 ` Slark Xiao [this message]
2026-01-22 2:14 ` Gustavo A. R. Silva
2026-01-22 2:18 ` Slark Xiao
2026-01-22 3:10 ` patchwork-bot+netdevbpf
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1eb019f4.1033.19be352bf0c.Coremail.slark_xiao@163.com \
--to=slark_xiao@163.com \
--cc=andrew+netdev@lunn.ch \
--cc=davem@davemloft.net \
--cc=edumazet@google.com \
--cc=gustavo@embeddedor.com \
--cc=gustavoars@kernel.org \
--cc=johannes@sipsolutions.net \
--cc=kuba@kernel.org \
--cc=linux-hardening@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=loic.poulain@oss.qualcomm.com \
--cc=netdev@vger.kernel.org \
--cc=pabeni@redhat.com \
--cc=ryazanov.s.a@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox