mbox series

[GIT,PULL,FOR,v6.7] Venus updates

Message ID 20231022111222.484567-1-stanimir.k.varbanov@gmail.com (mailing list archive)
State New, archived
Headers show
Series [GIT,PULL,FOR,v6.7] Venus updates | expand

Pull-request

git://linuxtv.org/svarbanov/media_tree.git tags/tag-venus-for-v6.7

Message

Stanimir Varbanov Oct. 22, 2023, 11:12 a.m. UTC
Hello Mauro,

The pull request includes:
 - A flex-array fix
 - Few fixes to avoid OOB accesses
 - Correct firmware path for more SOC versions

Please, pull.

regards,
Stan

The following changes since commit 94e27fbeca27d8c772fc2bc807730aaee5886055:

  media: cec: meson: always include meson sub-directory in Makefile (2023-10-13 11:33:22 +0200)

are available in the Git repository at:

  git://linuxtv.org/svarbanov/media_tree.git tags/tag-venus-for-v6.7

for you to fetch changes up to 9060ee6e0c0e865922ffd51517a7693ae3b231b8:

  media: venus: Fix firmware path for resources (2023-10-22 03:41:57 +0300)

----------------------------------------------------------------
Venus updates for v6.7

----------------------------------------------------------------
Gustavo A. R. Silva (1):
      media: venus: hfi_cmds: Replace one-element array with flex-array member and use __counted_by

Patrick Whewell (1):
      media: venus: Fix firmware path for resources

Vikash Garodia (4):
      venus: hfi: add checks to perform sanity on queue pointers
      venus: hfi: fix the check to handle session buffer requirement
      venus: hfi: add checks to handle capabilities from firmware
      venus: hfi_parser: Add check to keep the number of codecs within range

 drivers/media/platform/qcom/venus/core.c       | 10 +++++-----
 drivers/media/platform/qcom/venus/hfi_cmds.h   |  2 +-
 drivers/media/platform/qcom/venus/hfi_msgs.c   |  2 +-
 drivers/media/platform/qcom/venus/hfi_parser.c | 15 +++++++++++++++
 drivers/media/platform/qcom/venus/hfi_venus.c  | 10 ++++++++++
 5 files changed, 32 insertions(+), 7 deletions(-)

Comments

Jenkins Oct. 22, 2023, 11:42 a.m. UTC | #1
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/20231022111222.484567-1-stanimir.k.varbanov@gmail.com/
Build log: https://builder.linuxtv.org/job/patchwork/350425/
Build time: 00:23:30
Link: https://lore.kernel.org/linux-media/20231022111222.484567-1-stanimir.k.varbanov@gmail.com

gpg: Signature made Sun 22 Oct 2023 10:54:22 AM UTC
gpg:                using RSA key E1558C2497CE3CCC2B5AA30F25B55FC81B7035F2
gpg: Good signature from "Stanimir Varbanov <stanimir.varbanov@linaro.org>" [expired]
gpg: Note: This key has expired!
Primary key fingerprint: 34CF E039 8A16 AD93 18FD  D5E8 A6D0 26D8 E358 14D4
     Subkey fingerprint: E155 8C24 97CE 3CCC 2B5A  A30F 25B5 5FC8 1B70 35F2

Summary: got 2/6 patches with issues, being 1 at build time

Error/warnings:

patches/0001-venus-hfi-add-checks-to-perform-sanity-on-queue-poin.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

    allyesconfig: return code #0:
	SMATCH:../drivers/media/usb/siano/smsusb.c ../drivers/media/usb/siano/smsusb.c:53:38: :warning: array of flexible structures
	../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.
	../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/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2517 dvb_register() parse error: OOM: 3014560Kb sm_state_count = 1757213
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2517 dvb_register() parse error: __split_smt: function too hairy.  Giving up after 52 seconds
	../drivers/media/pci/cx25821/cx25821-medusa-video.c: ../drivers/media/pci/cx25821/cx25821-medusa-video.c:399 medusa_set_videostandard() parse error: OOM: 3003988Kb sm_state_count = 2158
	../drivers/media/pci/cx25821/cx25821-medusa-video.c: ../drivers/media/pci/cx25821/cx25821-medusa-video.c:399 medusa_set_videostandard() parse error: __split_smt: function too hairy.  Giving up after 4 seconds
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2831 em28xx_v4l2_init() parse error: turning off implications after 60 seconds
	../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

patches/0005-media-venus-hfi_cmds-Replace-one-element-array-with-.patch:

   checkpatch.pl:
	$ cat patches/0005-media-venus-hfi_cmds-Replace-one-element-array-with-.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:12: WARNING: Prefer a maximum 75 chars per line (possible unwrapped commit description?)