Re: [PATCH v8 0/2] Add new APIs of devcoredump and fix bugs

From: Brian Norris
Date: Wed Aug 24 2022 - 16:50:44 EST


On Tue, Aug 23, 2022 at 4:21 AM Duoming Zhou <duoming@xxxxxxxxxx> 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.

Brian