From: Pkshih <pkshih@realtek.com>
To: Takashi Iwai <tiwai@suse.de>, Kalle Valo <kvalo@codeaurora.org>
Cc: "linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
"Larry.Finger@lwfinger.net" <Larry.Finger@lwfinger.net>
Subject: RE: [PATCH v3 0/3] rtw89: fix crash when loading firmware file on certain platforms
Date: Mon, 22 Nov 2021 00:22:47 +0000 [thread overview]
Message-ID: <3f7bced126cb496bb759d495d024d74d@realtek.com> (raw)
In-Reply-To: <s5h7dd4fdup.wl-tiwai@suse.de>
> -----Original Message-----
> From: Takashi Iwai <tiwai@suse.de>
> Sent: Friday, November 19, 2021 9:57 PM
> To: Kalle Valo <kvalo@codeaurora.org>
> Cc: Pkshih <pkshih@realtek.com>; linux-wireless@vger.kernel.org; tiwai@suse.de;
> Larry.Finger@lwfinger.net
> Subject: Re: [PATCH v3 0/3] rtw89: fix crash when loading firmware file on certain platforms
>
> On Fri, 19 Nov 2021 13:05:56 +0100,
> Kalle Valo wrote:
> >
> > Ping-Ke Shih <pkshih@realtek.com> writes:
> >
> > > First patch is to avoid writing "partition size" on read-only firmware
> > > buffer, so it has to fix the crash.
> > > The later two patches are used to make the semantics clear, but they don't
> > > change the logic at all.
> > >
> > > I hope at least first patch can be taken into 5.16-rc, so people can avoid
> > > this crash.
> >
> > My plan is to take patch 1 to wireless-drivers. And, after patch 1
> > trickles down to wireless-drivers-next, patch 2-3 to
> > wireless-drivers-next. Sounds good to everyone?
>
> Yes, sounds like a good plan.
>
I also agree with your plan.
--
Ping-Ke
prev parent reply other threads:[~2021-11-22 0:23 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-11-19 5:45 [PATCH v3 0/3] rtw89: fix crash when loading firmware file on certain platforms Ping-Ke Shih
2021-11-19 5:45 ` [PATCH v3 1/3] rtw89: update partition size of firmware header on skb->data Ping-Ke Shih
2021-11-22 18:42 ` Kalle Valo
2021-11-19 5:45 ` [PATCH v3 2/3] rtw89: add const in the cast of le32_get_bits() Ping-Ke Shih
2021-11-26 16:04 ` Kalle Valo
2021-12-08 18:26 ` Kalle Valo
2021-11-19 5:45 ` [PATCH v3 3/3] rtw89: use inline function instead macro to set H2C and CAM Ping-Ke Shih
2021-11-19 12:05 ` [PATCH v3 0/3] rtw89: fix crash when loading firmware file on certain platforms Kalle Valo
2021-11-19 13:56 ` Takashi Iwai
2021-11-22 0:22 ` Pkshih [this message]
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=3f7bced126cb496bb759d495d024d74d@realtek.com \
--to=pkshih@realtek.com \
--cc=Larry.Finger@lwfinger.net \
--cc=kvalo@codeaurora.org \
--cc=linux-wireless@vger.kernel.org \
--cc=tiwai@suse.de \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.