mbox series

[v6,0/2] Remove useless param of devcoredump functions and fix bugs

Message ID cover.1654569290.git.duoming@zju.edu.cn (mailing list archive)
Headers show
Series Remove useless param of devcoredump functions and fix bugs | expand

Message

Duoming Zhou June 7, 2022, 3:26 a.m. UTC
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(-)

Comments

Greg Kroah-Hartman June 10, 2022, 1:24 p.m. UTC | #1
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
Johannes Berg June 10, 2022, 1:27 p.m. UTC | #2
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
Duoming Zhou June 20, 2022, 3:57 a.m. UTC | #3
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