2022-06-07 08:30:40

by Duoming Zhou

[permalink] [raw]
Subject: [PATCH v6 0/2] Remove useless param of devcoredump functions and fix bugs

The first patch removes the extra gfp_t param of dev_coredumpv()
and dev_coredumpm().

The second patch fix sleep in atomic context bugs of mwifiex
caused by dev_coredumpv().

Duoming Zhou (2):
devcoredump: remove the useless gfp_t parameter in dev_coredumpv and
dev_coredumpm
mwifiex: fix sleep in atomic context bugs caused by dev_coredumpv

drivers/base/devcoredump.c | 16 ++++++----------
drivers/bluetooth/btmrvl_sdio.c | 2 +-
drivers/bluetooth/hci_qca.c | 2 +-
drivers/gpu/drm/etnaviv/etnaviv_dump.c | 2 +-
drivers/gpu/drm/msm/disp/msm_disp_snapshot.c | 4 ++--
drivers/gpu/drm/msm/msm_gpu.c | 4 ++--
drivers/media/platform/qcom/venus/core.c | 2 +-
drivers/net/can/spi/mcp251xfd/mcp251xfd-dump.c | 2 +-
drivers/net/wireless/ath/ath10k/coredump.c | 2 +-
.../net/wireless/ath/wil6210/wil_crash_dump.c | 2 +-
.../wireless/broadcom/brcm80211/brcmfmac/debug.c | 2 +-
drivers/net/wireless/intel/iwlwifi/fw/dbg.c | 6 ++----
drivers/net/wireless/marvell/mwifiex/init.c | 9 +++++----
drivers/net/wireless/marvell/mwifiex/main.c | 3 +--
drivers/net/wireless/marvell/mwifiex/main.h | 3 ++-
drivers/net/wireless/marvell/mwifiex/sta_event.c | 6 +++---
drivers/net/wireless/mediatek/mt76/mt7615/mac.c | 3 +--
drivers/net/wireless/mediatek/mt76/mt7921/mac.c | 3 +--
drivers/net/wireless/realtek/rtw88/main.c | 2 +-
drivers/net/wireless/realtek/rtw89/ser.c | 2 +-
drivers/remoteproc/qcom_q6v5_mss.c | 2 +-
drivers/remoteproc/remoteproc_coredump.c | 8 ++++----
include/drm/drm_print.h | 2 +-
include/linux/devcoredump.h | 13 ++++++-------
sound/soc/intel/avs/apl.c | 2 +-
sound/soc/intel/avs/skl.c | 2 +-
sound/soc/intel/catpt/dsp.c | 2 +-
27 files changed, 50 insertions(+), 58 deletions(-)

--
2.17.1


2022-06-10 13:27:36

by Greg Kroah-Hartman

[permalink] [raw]
Subject: Re: [PATCH v6 0/2] Remove useless param of devcoredump functions and fix bugs

On Tue, Jun 07, 2022 at 11:26:24AM +0800, Duoming Zhou wrote:
> The first patch removes the extra gfp_t param of dev_coredumpv()
> and dev_coredumpm().
>
> The second patch fix sleep in atomic context bugs of mwifiex
> caused by dev_coredumpv().
>
> Duoming Zhou (2):
> devcoredump: remove the useless gfp_t parameter in dev_coredumpv and
> dev_coredumpm
> mwifiex: fix sleep in atomic context bugs caused by dev_coredumpv

Did you forget to cc: everyone on patch 2?

thanks,

greg k-h

2022-06-10 13:34:23

by Johannes Berg

[permalink] [raw]
Subject: Re: [PATCH v6 0/2] Remove useless param of devcoredump functions and fix bugs

On Fri, 2022-06-10 at 15:24 +0200, Greg KH wrote:
> On Tue, Jun 07, 2022 at 11:26:24AM +0800, Duoming Zhou wrote:
> > The first patch removes the extra gfp_t param of dev_coredumpv()
> > and dev_coredumpm().
> >
> > The second patch fix sleep in atomic context bugs of mwifiex
> > caused by dev_coredumpv().
> >
> > Duoming Zhou (2):
> > devcoredump: remove the useless gfp_t parameter in dev_coredumpv and
> > dev_coredumpm
> > mwifiex: fix sleep in atomic context bugs caused by dev_coredumpv
>
> Did you forget to cc: everyone on patch 2?
>

Don't think so? I got it, and you're listed there too, afaict.

That said, it's actually entirely independent from patch 1, and patch 2
should probably even be Cc stable.

johannes

2022-06-20 04:05:09

by Duoming Zhou

[permalink] [raw]
Subject: Re: [PATCH v6 0/2] Remove useless param of devcoredump functions and fix bugs

Hello maintainers,

This patch set has been reviewed by Johannes Berg and Brian Norris,
but it has not been accepted yet. Do I need to do any extra work?

Thank you for your time!

Best regards,
Duoming Zhou