Messages in this thread | | | Date | Thu, 25 Aug 2022 08:44:55 +0800 (GMT+08:00) | From | duoming@zju ... | Subject | Re: [PATCH v8 0/2] Add new APIs of devcoredump and fix bugs |
| |
Hello,
On Wed, 24 Aug 2022 13:42:09 -0700 Brian Norris wrote:
> On Tue, Aug 23, 2022 at 4:21 AM Duoming Zhou <duoming@zju.edu.cn> wrote: > > > > The first patch adds new APIs to support migration of users > > from old device coredump related APIs. > > > > The second patch fix sleep in atomic context bugs of mwifiex > > caused by dev_coredumpv(). > > > > Duoming Zhou (2): > > devcoredump: add new APIs to support migration of users from old > > device coredump related APIs > > mwifiex: fix sleep in atomic context bugs caused by dev_coredumpv > > I would have expected a third patch in here, that actually converts > existing users. Then in the following release cycle, clean up any new > users of the old API that pop up in the meantime and drop the old API. > > But I'll defer to the people who would actually be merging your code. > Technically it could also work to simply provide the API this cycle, > and convert everyone in the next.
Thank your for your time and reply.
If this patch set is merged into the linux-next tree, I will send the third patch which targets at linux-next tree and converts existing users at later timer of this release cycle. Because there are new users that may use the old APIs comes into linux-next tree during the remaining time of this release cycle.
Best regards, Duoming Zhou | |