[GIT,PULL,for,v6.4-rc7] media fixes

Message ID 20230615121721.1f901ebb@sal.lan (mailing list archive)
State Not Applicable
Headers
Series [GIT,PULL,for,v6.4-rc7] media fixes |

Pull-request

git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media tags/media/v6.4-6

Message

Mauro Carvalho Chehab June 15, 2023, 11:17 a.m. UTC
  Hi Linus,

Please pull from:
  git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media tags/media/v6.4-6

For a fix at dvb-core to avoid a race condition during DVB board register
time.

Regards,
Mauro


The following changes since commit 9561de3a55bed6bdd44a12820ba81ec416e705a7:

  Linux 6.4-rc5 (2023-06-04 14:04:27 -0400)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media tags/media/v6.4-6

for you to fetch changes up to ec21a38df77a5aefbd2f70c48127003b6f259cf3:

  Revert "media: dvb-core: Fix use-after-free on race condition at dvb_frontend" (2023-06-14 23:16:29 +0100)

----------------------------------------------------------------
media fixes for v6.4-rc7

----------------------------------------------------------------
Mauro Carvalho Chehab (3):
      Merge tag 'v6.4-rc4' into v4l_for_linus
      Merge tag 'v6.4-rc5' into v4l_for_linus
      Revert "media: dvb-core: Fix use-after-free on race condition at dvb_frontend"

 drivers/media/dvb-core/dvb_frontend.c | 53 ++++++-----------------------------
 include/media/dvb_frontend.h          |  6 +---
 2 files changed, 10 insertions(+), 49 deletions(-)
  

Comments

Jenkins June 15, 2023, 11:43 a.m. UTC | #1
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/20230615121721.1f901ebb@sal.lan/
Build log: https://builder.linuxtv.org/job/patchwork/314812/
Build time: 00:21:03
Link: https://lore.kernel.org/linux-media/20230615121721.1f901ebb@sal.lan

gpg: Signature made Thu 15 Jun 2023 11:15:44 AM UTC
gpg:                using RSA key F909AE68FC11DF09C1755C00085F3EBD8EE4E115
gpg: Good signature from "Mauro Carvalho Chehab <mchehab+huawei@kernel.org>" [ultimate]
gpg:                 aka "Mauro Carvalho Chehab <mchehab@kernel.org>" [ultimate]
gpg:                 aka "Mauro Carvalho Chehab <m.chehab@samsung.com>" [ultimate]
gpg:                 aka "Mauro Carvalho Chehab <mchehab@osg.samsung.com>" [ultimate]
gpg:                 aka "Mauro Carvalho Chehab <mchehab@s-opensource.com>" [ultimate]
gpg:                 aka "Mauro Carvalho Chehab <mchehab+samsung@kernel.org>" [ultimate]
gpg:                 aka "[jpeg image of size 3594]" [ultimate]

Summary: got 1/1 patches with issues, being 1 at build time, plus one error when buinding PDF document

Error/warnings:

patches/0001-Revert-media-dvb-core-Fix-use-after-free-on-race-con.patch:

    allyesconfig: return code #0:
	../scripts/genksyms/parse.y: warning: 9 shift/reduce conflicts [-Wconflicts-sr]
	../scripts/genksyms/parse.y: warning: 5 reduce/reduce conflicts [-Wconflicts-rr]
	../scripts/genksyms/parse.y: note: rerun with option '-Wcounterexamples' to generate conflict counterexamples
	SPARSE:../drivers/staging/media/tegra-video/vip.c ../drivers/staging/media/tegra-video/vip.c:280:24: warning: symbol 'tegra_vip_driver' was not declared. Should it be static?
	../drivers/staging/media/atomisp/i2c/atomisp-gc0310.c:447 gc0310_s_stream() warn: missing error code 'ret'
	../drivers/staging/media/atomisp/i2c/atomisp-ov2680.c: ../drivers/staging/media/atomisp/i2c/atomisp-ov2680.c:476 ov2680_init_cfg() error: we previously assumed 'sd_state' could be null (see line 468)
	../drivers/staging/media/atomisp/i2c/atomisp-ov2680.c:524 ov2680_s_stream() warn: missing error code 'ret'
	../drivers/staging/media/atomisp/pci/atomisp_fops.c: ../drivers/staging/media/atomisp/pci/atomisp_fops.c:517 atomisp_open() warn: missing unwind goto?
	../drivers/staging/media/atomisp/pci/atomisp_cmd.c: ../drivers/staging/media/atomisp/pci/atomisp_cmd.c:2801 atomisp_cp_dvs_6axis_config() warn: missing unwind goto?
	../drivers/staging/media/atomisp/pci/atomisp_cmd.c: ../drivers/staging/media/atomisp/pci/atomisp_cmd.c:2900 atomisp_cp_morph_table() warn: missing unwind goto?
	../drivers/staging/media/atomisp/pci/atomisp_cmd.c: ../drivers/staging/media/atomisp/pci/atomisp_cmd.c:4283 atomisp_set_fmt_to_snr() warn: variable dereferenced before check 'asd' (see line 4268)

    allyesconfig: return code #0:
	../drivers/media/i2c/adp1653.c: ../drivers/media/i2c/adp1653.c:444 adp1653_of_init() warn: missing unwind goto?
	../drivers/media/platform/mediatek/vcodec/vdec_msg_queue.c: ../drivers/media/platform/mediatek/vcodec/vdec_msg_queue.c:347 vdec_msg_queue_init() warn: missing unwind goto?
	../drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c: ../drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c:2775 mxc_jpeg_probe() warn: missing unwind goto?
	../drivers/media/i2c/ov5645.c: ../drivers/media/i2c/ov5645.c:687 ov5645_set_power_on() warn: 'ov5645->xclk' from clk_prepare_enable() not released on lines: 687.
	SMATCH:../drivers/media/usb/siano/smsusb.c ../drivers/media/usb/siano/smsusb.c:53:38: :warning: array of flexible structures
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2570 dvb_register() parse error: OOM: 3000032Kb sm_state_count = 1973014
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2570 dvb_register() warn: Function too hairy.  No more merges.
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2570 dvb_register() parse error: __split_smt: function too hairy.  Giving up after 54 seconds
	../drivers/media/usb/uvc/uvc_v4l2.c: note: in included file (through ../arch/x86/include/asm/uaccess.h, ../include/linux/uaccess.h, ../include/linux/sched/task.h, ../include/linux/sched/signal.h, ../include/linux/rcuwait.h, ...):
	SPARSE:../drivers/media/usb/uvc/uvc_v4l2.c ../arch/x86/include/asm/uaccess_64.h:88:24: warning: cast removes address space '__user' of expression
	../drivers/media/pci/ivtv/ivtvfb.c: note: in included file (through ../arch/x86/include/asm/uaccess.h, ../include/linux/uaccess.h, ../include/linux/sched/task.h, ../include/linux/sched/signal.h, ../drivers/media/pci/ivtv/ivtv-driver.h):
	SPARSE:../drivers/media/pci/ivtv/ivtvfb.c ../arch/x86/include/asm/uaccess_64.h:88:24: warning: cast removes address space '__user' of expression
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2884 em28xx_v4l2_init() parse error: turning off implications after 60 seconds
	../drivers/media/usb/pvrusb2/pvrusb2-hdw.c: ../drivers/media/usb/pvrusb2/pvrusb2-hdw.c:3293 pvr2_hdw_get_tuner_status() warn: inconsistent indenting


Error #512 when building PDF docs
  
Linus Torvalds June 16, 2023, 2:17 a.m. UTC | #2
On Thu, 15 Jun 2023 at 04:17, Mauro Carvalho Chehab <mchehab@kernel.org> wrote:
>
> Mauro Carvalho Chehab (3):
>       Merge tag 'v6.4-rc4' into v4l_for_linus
>       Merge tag 'v6.4-rc5' into v4l_for_linus
>       Revert "media: dvb-core: Fix use-after-free on race condition at dvb_frontend"

Please don't do those back-merges. Particularly the *pointless* ones
with bad commit messages.

I've pulled this, bnut your tree literally had three commits, two of
which were WRONG. Their commit messages were pure garbage.

Don't do this. Seriously. It's garbage.

If you can't do proper merges with proper commit messages, just don't
merge things. It's that easy.

              Linus
  
pr-tracker-bot@kernel.org June 16, 2023, 3:10 a.m. UTC | #3
The pull request you sent on Thu, 15 Jun 2023 12:17:21 +0100:

> git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media tags/media/v6.4-6

has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/c926a55f650ce58eae33cda504d50ac1837d53fd

Thank you!