2023-07-07 10:27:04 |
You-Sheng Yang |
bug |
|
|
added bug |
2023-07-11 16:11:41 |
You-Sheng Yang |
summary |
failed to build against linux next-20230706 |
failed to build against linux v6.5-rc1 |
|
2023-07-11 16:12:34 |
You-Sheng Yang |
nominated for series |
|
Ubuntu Mantic |
|
2023-07-11 16:12:34 |
You-Sheng Yang |
bug task added |
|
ipu6-drivers (Ubuntu Mantic) |
|
2023-07-11 16:28:09 |
You-Sheng Yang |
ipu6-drivers (Ubuntu Mantic): status |
New |
In Progress |
|
2023-07-11 16:28:12 |
You-Sheng Yang |
ipu6-drivers (Ubuntu Mantic): importance |
Undecided |
High |
|
2023-07-11 16:28:14 |
You-Sheng Yang |
ipu6-drivers (Ubuntu Mantic): assignee |
|
You-Sheng Yang (vicamo) |
|
2023-07-25 10:49:18 |
You-Sheng Yang |
description |
```
<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.c: In function 'ipu_psys_get_userpages':
<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.c:214:22: error: too many arguments to function 'get_user_pages'
214 | nr = get_user_pages(start & PAGE_MASK, npages,
| ^~~~~~~~~~~~~~
In file included from ./include/linux/scatterlist.h:8,
from ./include/linux/dma-buf.h:19,
from <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.c:7:
./include/linux/mm.h:2476:6: note: declared here
2476 | long get_user_pages(unsigned long start, unsigned long nr_pages,
| ^~~~~~~~~~~~~~
<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-isys-csi2.c:473:1: warning: 'csi2_try_fmt' defined but not used [-Wunused-function]
473 | csi2_try_fmt(struct ipu_isys_video *av,
| ^~~~~~~~~~~~
make[6]: *** [scripts/Makefile.build:251: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.o] Error 1
make[6]: *** Waiting for unfinished jobs....
make[5]: *** [scripts/Makefile.build:485: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6] Error 2
make[4]: *** [scripts/Makefile.build:485: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel] Error 2
make[3]: *** [<<DKMSDIR>>/headers/linux-headers-6.4.0-20230706-next/Makefile:2025: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build] Error 2
make[2]: *** [Makefile:234: __sub-make] Error 2
make[2]: Leaving directory '<<DKMSDIR>>/headers/linux-headers-6.4.0-20230706-next'
make[1]: *** [Makefile:37: all] Error 2
make[1]: Leaving directory '<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build'
make: *** [debian/rules.d/2-binary-arch.mk:240: /build/mantic/debian/stamps/stamp-install-next] Error 10
``` |
[SRU Justifications]
[Impact]
Bug 2012407: kernel build fails without a newly learned directive in /sbin/dkms in bug 2012612 that helps identify whether a dkms modules is supposed to be built at all.
Bug 2021740: To allow dkms driver installation initiated by ubuntu-drivers, packages must carry XB-Modaliases in debian/control.
Bug 2026402: ipu6 dkms driver now fails to build against v6.5-rc1 and on.
[Fix]
* A BUILD_EXCLUSIVE_CONFIG patch for bug 2012407 was added to ivsc-driver/{jammy,lunar} because ivsc-driver/mantic has that already from bug 2012410. Also applied to ipu6-drivers/{mantic,lunar,jammy}.
* Modaliases fixes for bug 2021740 was applied to ivsc-drivers/{mantic,lunar,jammy}, and ipu6-drivers/{mantic,lunar,jammy}
* Additional fixes from upstream ipu6-drivers repo were added to ipu6-drivers/mantic for new hardware platforms.
[Test Case]
Bug 2012407: make sure intel-ipu6-dkms build will be skipped on platforms without CONFIG_VIDEO_V4L2_I2C, and intel-vsc-dkms on platforms without CONFIG_GPIOLIB_IRQCHIP, such as linux-headers-kvm/jammy.
Bug 2021740: check built binary deb has XB-Modaliases tag in `dpkg-deb --info` output.
For rest of the changes, they should be verified on next IPU6 equipped platform.
[Where problems could occur]
For changes for bug 2012407 and bug 2021740, they do no side effect to runtime system but to reinforce the interaction between other components.
For rest of the changes, only in ipu6-drivers/mantic, they're to sync with upstream development for new hardware platforms, and expects bug fixes, updates in the coming development cycle.
[Other Info]
========== original bug report ==========
```
<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.c: In function 'ipu_psys_get_userpages':
<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.c:214:22: error: too many arguments to function 'get_user_pages'
214 | nr = get_user_pages(start & PAGE_MASK, npages,
| ^~~~~~~~~~~~~~
In file included from ./include/linux/scatterlist.h:8,
from ./include/linux/dma-buf.h:19,
from <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.c:7:
./include/linux/mm.h:2476:6: note: declared here
2476 | long get_user_pages(unsigned long start, unsigned long nr_pages,
| ^~~~~~~~~~~~~~
<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-isys-csi2.c:473:1: warning: 'csi2_try_fmt' defined but not used [-Wunused-function]
473 | csi2_try_fmt(struct ipu_isys_video *av,
| ^~~~~~~~~~~~
make[6]: *** [scripts/Makefile.build:251: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.o] Error 1
make[6]: *** Waiting for unfinished jobs....
make[5]: *** [scripts/Makefile.build:485: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6] Error 2
make[4]: *** [scripts/Makefile.build:485: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel] Error 2
make[3]: *** [<<DKMSDIR>>/headers/linux-headers-6.4.0-20230706-next/Makefile:2025: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build] Error 2
make[2]: *** [Makefile:234: __sub-make] Error 2
make[2]: Leaving directory '<<DKMSDIR>>/headers/linux-headers-6.4.0-20230706-next'
make[1]: *** [Makefile:37: all] Error 2
make[1]: Leaving directory '<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build'
make: *** [debian/rules.d/2-binary-arch.mk:240: /build/mantic/debian/stamps/stamp-install-next] Error 10
``` |
|
2023-10-31 03:12:16 |
You-Sheng Yang |
ipu6-drivers (Ubuntu Mantic): status |
In Progress |
Triaged |
|
2023-11-08 22:38:27 |
Launchpad Janitor |
ipu6-drivers (Ubuntu): status |
In Progress |
Fix Released |
|
2023-11-15 08:40:02 |
You-Sheng Yang |
ipu6-drivers (Ubuntu Mantic): status |
Triaged |
In Progress |
|
2023-11-15 09:42:57 |
Ubuntu Archive Robot |
bug |
|
|
added subscriber Timo Aaltonen |
2023-11-23 18:28:32 |
Andreas Hasenack |
ipu6-drivers (Ubuntu Mantic): status |
In Progress |
Incomplete |
|
2023-11-24 13:45:35 |
Andreas Hasenack |
bug |
|
|
added subscriber Andreas Hasenack |
2023-11-27 10:14:49 |
You-Sheng Yang |
description |
[SRU Justifications]
[Impact]
Bug 2012407: kernel build fails without a newly learned directive in /sbin/dkms in bug 2012612 that helps identify whether a dkms modules is supposed to be built at all.
Bug 2021740: To allow dkms driver installation initiated by ubuntu-drivers, packages must carry XB-Modaliases in debian/control.
Bug 2026402: ipu6 dkms driver now fails to build against v6.5-rc1 and on.
[Fix]
* A BUILD_EXCLUSIVE_CONFIG patch for bug 2012407 was added to ivsc-driver/{jammy,lunar} because ivsc-driver/mantic has that already from bug 2012410. Also applied to ipu6-drivers/{mantic,lunar,jammy}.
* Modaliases fixes for bug 2021740 was applied to ivsc-drivers/{mantic,lunar,jammy}, and ipu6-drivers/{mantic,lunar,jammy}
* Additional fixes from upstream ipu6-drivers repo were added to ipu6-drivers/mantic for new hardware platforms.
[Test Case]
Bug 2012407: make sure intel-ipu6-dkms build will be skipped on platforms without CONFIG_VIDEO_V4L2_I2C, and intel-vsc-dkms on platforms without CONFIG_GPIOLIB_IRQCHIP, such as linux-headers-kvm/jammy.
Bug 2021740: check built binary deb has XB-Modaliases tag in `dpkg-deb --info` output.
For rest of the changes, they should be verified on next IPU6 equipped platform.
[Where problems could occur]
For changes for bug 2012407 and bug 2021740, they do no side effect to runtime system but to reinforce the interaction between other components.
For rest of the changes, only in ipu6-drivers/mantic, they're to sync with upstream development for new hardware platforms, and expects bug fixes, updates in the coming development cycle.
[Other Info]
========== original bug report ==========
```
<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.c: In function 'ipu_psys_get_userpages':
<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.c:214:22: error: too many arguments to function 'get_user_pages'
214 | nr = get_user_pages(start & PAGE_MASK, npages,
| ^~~~~~~~~~~~~~
In file included from ./include/linux/scatterlist.h:8,
from ./include/linux/dma-buf.h:19,
from <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.c:7:
./include/linux/mm.h:2476:6: note: declared here
2476 | long get_user_pages(unsigned long start, unsigned long nr_pages,
| ^~~~~~~~~~~~~~
<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-isys-csi2.c:473:1: warning: 'csi2_try_fmt' defined but not used [-Wunused-function]
473 | csi2_try_fmt(struct ipu_isys_video *av,
| ^~~~~~~~~~~~
make[6]: *** [scripts/Makefile.build:251: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.o] Error 1
make[6]: *** Waiting for unfinished jobs....
make[5]: *** [scripts/Makefile.build:485: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6] Error 2
make[4]: *** [scripts/Makefile.build:485: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel] Error 2
make[3]: *** [<<DKMSDIR>>/headers/linux-headers-6.4.0-20230706-next/Makefile:2025: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build] Error 2
make[2]: *** [Makefile:234: __sub-make] Error 2
make[2]: Leaving directory '<<DKMSDIR>>/headers/linux-headers-6.4.0-20230706-next'
make[1]: *** [Makefile:37: all] Error 2
make[1]: Leaving directory '<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build'
make: *** [debian/rules.d/2-binary-arch.mk:240: /build/mantic/debian/stamps/stamp-install-next] Error 10
``` |
[SRU Justifications]
[Impact]
ipu6 dkms driver now fails to build against v6.5-rc1 and on.
[Fix]
Additional fixes from upstream ipu6-drivers repo were added to ipu6-drivers/mantic for new hardware platforms.
[Test Case]
This is verified on MTL VSC platforms, as well as previous generations e.g. TGL and ADL.
To actually verify the camera framework:
```
# enable mantic-proposed, update the kernel, make sure you get 6.5.0-12.12
# reboot into the 6.5.0-12.12 kernel
$ sudo apt install intel-ipu6-dkms intel-vsc-dkms
$ sudo add-apt-repository ppa:oem-solutions-group/intel-ipu6 -y -u
$ sudo apt install gstreamer1.0-icamera v4l2-relayd
```
In the above steps, make sure that only gstreamer1.0-icamera, v4l2-relayd, and their dependencies, are fetched from the oem-solutions-group PPA. Everything else must come from the mantic archive.
And then browse https://webcamtests.com/ for testing.
[Where problems could occur]
The fixing patch itself adds a few conditional compiling sections that utilize V4L2/GPIO APIs from the new kernel version, no functional change and no runtime behavior change is expected.
[Other Info]
========== original bug report ==========
```
<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.c: In function 'ipu_psys_get_userpages':
<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.c:214:22: error: too many arguments to function 'get_user_pages'
214 | nr = get_user_pages(start & PAGE_MASK, npages,
| ^~~~~~~~~~~~~~
In file included from ./include/linux/scatterlist.h:8,
from ./include/linux/dma-buf.h:19,
from <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.c:7:
./include/linux/mm.h:2476:6: note: declared here
2476 | long get_user_pages(unsigned long start, unsigned long nr_pages,
| ^~~~~~~~~~~~~~
<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-isys-csi2.c:473:1: warning: 'csi2_try_fmt' defined but not used [-Wunused-function]
473 | csi2_try_fmt(struct ipu_isys_video *av,
| ^~~~~~~~~~~~
make[6]: *** [scripts/Makefile.build:251: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.o] Error 1
make[6]: *** Waiting for unfinished jobs....
make[5]: *** [scripts/Makefile.build:485: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6] Error 2
make[4]: *** [scripts/Makefile.build:485: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel] Error 2
make[3]: *** [<<DKMSDIR>>/headers/linux-headers-6.4.0-20230706-next/Makefile:2025: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build] Error 2
make[2]: *** [Makefile:234: __sub-make] Error 2
make[2]: Leaving directory '<<DKMSDIR>>/headers/linux-headers-6.4.0-20230706-next'
make[1]: *** [Makefile:37: all] Error 2
make[1]: Leaving directory '<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build'
make: *** [debian/rules.d/2-binary-arch.mk:240: /build/mantic/debian/stamps/stamp-install-next] Error 10
``` |
|
2023-11-27 10:16:29 |
You-Sheng Yang |
description |
[SRU Justifications]
[Impact]
ipu6 dkms driver now fails to build against v6.5-rc1 and on.
[Fix]
Additional fixes from upstream ipu6-drivers repo were added to ipu6-drivers/mantic for new hardware platforms.
[Test Case]
This is verified on MTL VSC platforms, as well as previous generations e.g. TGL and ADL.
To actually verify the camera framework:
```
# enable mantic-proposed, update the kernel, make sure you get 6.5.0-12.12
# reboot into the 6.5.0-12.12 kernel
$ sudo apt install intel-ipu6-dkms intel-vsc-dkms
$ sudo add-apt-repository ppa:oem-solutions-group/intel-ipu6 -y -u
$ sudo apt install gstreamer1.0-icamera v4l2-relayd
```
In the above steps, make sure that only gstreamer1.0-icamera, v4l2-relayd, and their dependencies, are fetched from the oem-solutions-group PPA. Everything else must come from the mantic archive.
And then browse https://webcamtests.com/ for testing.
[Where problems could occur]
The fixing patch itself adds a few conditional compiling sections that utilize V4L2/GPIO APIs from the new kernel version, no functional change and no runtime behavior change is expected.
[Other Info]
========== original bug report ==========
```
<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.c: In function 'ipu_psys_get_userpages':
<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.c:214:22: error: too many arguments to function 'get_user_pages'
214 | nr = get_user_pages(start & PAGE_MASK, npages,
| ^~~~~~~~~~~~~~
In file included from ./include/linux/scatterlist.h:8,
from ./include/linux/dma-buf.h:19,
from <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.c:7:
./include/linux/mm.h:2476:6: note: declared here
2476 | long get_user_pages(unsigned long start, unsigned long nr_pages,
| ^~~~~~~~~~~~~~
<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-isys-csi2.c:473:1: warning: 'csi2_try_fmt' defined but not used [-Wunused-function]
473 | csi2_try_fmt(struct ipu_isys_video *av,
| ^~~~~~~~~~~~
make[6]: *** [scripts/Makefile.build:251: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.o] Error 1
make[6]: *** Waiting for unfinished jobs....
make[5]: *** [scripts/Makefile.build:485: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6] Error 2
make[4]: *** [scripts/Makefile.build:485: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel] Error 2
make[3]: *** [<<DKMSDIR>>/headers/linux-headers-6.4.0-20230706-next/Makefile:2025: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build] Error 2
make[2]: *** [Makefile:234: __sub-make] Error 2
make[2]: Leaving directory '<<DKMSDIR>>/headers/linux-headers-6.4.0-20230706-next'
make[1]: *** [Makefile:37: all] Error 2
make[1]: Leaving directory '<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build'
make: *** [debian/rules.d/2-binary-arch.mk:240: /build/mantic/debian/stamps/stamp-install-next] Error 10
``` |
[SRU Justifications]
[Impact]
ipu6 dkms driver now fails to build against v6.5-rc1 and on.
[Fix]
https://github.com/intel/ipu6-drivers/pull/174
[Test Case]
This is verified on MTL VSC platforms, as well as previous generations e.g. TGL and ADL.
To actually verify the camera framework:
```
# enable mantic-proposed, update the kernel, make sure you get 6.5.0-12.12
# reboot into the 6.5.0-12.12 kernel
$ sudo apt install intel-ipu6-dkms intel-vsc-dkms
$ sudo add-apt-repository ppa:oem-solutions-group/intel-ipu6 -y -u
$ sudo apt install gstreamer1.0-icamera v4l2-relayd
```
In the above steps, make sure that only gstreamer1.0-icamera, v4l2-relayd, and their dependencies, are fetched from the oem-solutions-group PPA. Everything else must come from the mantic archive.
And then browse https://webcamtests.com/ for testing.
[Where problems could occur]
The fixing patch itself adds a few conditional compiling sections that utilize V4L2/GPIO APIs from the new kernel version, no functional change and no runtime behavior change is expected.
[Other Info]
========== original bug report ==========
```
<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.c: In function 'ipu_psys_get_userpages':
<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.c:214:22: error: too many arguments to function 'get_user_pages'
214 | nr = get_user_pages(start & PAGE_MASK, npages,
| ^~~~~~~~~~~~~~
In file included from ./include/linux/scatterlist.h:8,
from ./include/linux/dma-buf.h:19,
from <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.c:7:
./include/linux/mm.h:2476:6: note: declared here
2476 | long get_user_pages(unsigned long start, unsigned long nr_pages,
| ^~~~~~~~~~~~~~
<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-isys-csi2.c:473:1: warning: 'csi2_try_fmt' defined but not used [-Wunused-function]
473 | csi2_try_fmt(struct ipu_isys_video *av,
| ^~~~~~~~~~~~
make[6]: *** [scripts/Makefile.build:251: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.o] Error 1
make[6]: *** Waiting for unfinished jobs....
make[5]: *** [scripts/Makefile.build:485: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6] Error 2
make[4]: *** [scripts/Makefile.build:485: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel] Error 2
make[3]: *** [<<DKMSDIR>>/headers/linux-headers-6.4.0-20230706-next/Makefile:2025: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build] Error 2
make[2]: *** [Makefile:234: __sub-make] Error 2
make[2]: Leaving directory '<<DKMSDIR>>/headers/linux-headers-6.4.0-20230706-next'
make[1]: *** [Makefile:37: all] Error 2
make[1]: Leaving directory '<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build'
make: *** [debian/rules.d/2-binary-arch.mk:240: /build/mantic/debian/stamps/stamp-install-next] Error 10
``` |
|
2023-11-27 10:19:53 |
You-Sheng Yang |
ipu6-drivers (Ubuntu Mantic): status |
Incomplete |
In Progress |
|
2023-11-29 17:57:28 |
You-Sheng Yang |
summary |
failed to build against linux v6.5-rc1 |
failed to build against linux v6.6 |
|
2023-11-29 18:00:21 |
You-Sheng Yang |
description |
[SRU Justifications]
[Impact]
ipu6 dkms driver now fails to build against v6.5-rc1 and on.
[Fix]
https://github.com/intel/ipu6-drivers/pull/174
[Test Case]
This is verified on MTL VSC platforms, as well as previous generations e.g. TGL and ADL.
To actually verify the camera framework:
```
# enable mantic-proposed, update the kernel, make sure you get 6.5.0-12.12
# reboot into the 6.5.0-12.12 kernel
$ sudo apt install intel-ipu6-dkms intel-vsc-dkms
$ sudo add-apt-repository ppa:oem-solutions-group/intel-ipu6 -y -u
$ sudo apt install gstreamer1.0-icamera v4l2-relayd
```
In the above steps, make sure that only gstreamer1.0-icamera, v4l2-relayd, and their dependencies, are fetched from the oem-solutions-group PPA. Everything else must come from the mantic archive.
And then browse https://webcamtests.com/ for testing.
[Where problems could occur]
The fixing patch itself adds a few conditional compiling sections that utilize V4L2/GPIO APIs from the new kernel version, no functional change and no runtime behavior change is expected.
[Other Info]
========== original bug report ==========
```
<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.c: In function 'ipu_psys_get_userpages':
<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.c:214:22: error: too many arguments to function 'get_user_pages'
214 | nr = get_user_pages(start & PAGE_MASK, npages,
| ^~~~~~~~~~~~~~
In file included from ./include/linux/scatterlist.h:8,
from ./include/linux/dma-buf.h:19,
from <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.c:7:
./include/linux/mm.h:2476:6: note: declared here
2476 | long get_user_pages(unsigned long start, unsigned long nr_pages,
| ^~~~~~~~~~~~~~
<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-isys-csi2.c:473:1: warning: 'csi2_try_fmt' defined but not used [-Wunused-function]
473 | csi2_try_fmt(struct ipu_isys_video *av,
| ^~~~~~~~~~~~
make[6]: *** [scripts/Makefile.build:251: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.o] Error 1
make[6]: *** Waiting for unfinished jobs....
make[5]: *** [scripts/Makefile.build:485: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6] Error 2
make[4]: *** [scripts/Makefile.build:485: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel] Error 2
make[3]: *** [<<DKMSDIR>>/headers/linux-headers-6.4.0-20230706-next/Makefile:2025: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build] Error 2
make[2]: *** [Makefile:234: __sub-make] Error 2
make[2]: Leaving directory '<<DKMSDIR>>/headers/linux-headers-6.4.0-20230706-next'
make[1]: *** [Makefile:37: all] Error 2
make[1]: Leaving directory '<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build'
make: *** [debian/rules.d/2-binary-arch.mk:240: /build/mantic/debian/stamps/stamp-install-next] Error 10
``` |
[SRU Justifications]
[Impact]
ipu6 dkms driver now fails to build against v6.6 and on.
[Fix]
https://github.com/intel/ipu6-drivers/pull/174
[Test Case]
This is verified on MTL VSC platforms, as well as previous generations e.g. TGL and ADL.
To actually verify the camera framework:
```
# enable mantic-proposed, install linux-unstable-6.6 kernel from kernel team ppa:
$ sudo apt-add-repository ppa:canonical-kernel-team/unstable
$ sudo apt install linux-generic-wip
$ sudo apt install intel-ipu6-dkms intel-vsc-dkms
$ sudo add-apt-repository ppa:oem-solutions-group/intel-ipu6 -y -u
$ sudo apt install gstreamer1.0-icamera v4l2-relayd
```
In the above steps, make sure that only gstreamer1.0-icamera, v4l2-relayd, and their dependencies, are fetched from the oem-solutions-group PPA. Everything else must come from the mantic archive.
And then browse https://webcamtests.com/ for testing.
[Where problems could occur]
The fixing patch itself adds a few conditional compiling sections that utilize V4L2/GPIO APIs from the new kernel version, no functional change and no runtime behavior change is expected.
[Other Info]
========== original bug report ==========
```
<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.c: In function 'ipu_psys_get_userpages':
<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.c:214:22: error: too many arguments to function 'get_user_pages'
214 | nr = get_user_pages(start & PAGE_MASK, npages,
| ^~~~~~~~~~~~~~
In file included from ./include/linux/scatterlist.h:8,
from ./include/linux/dma-buf.h:19,
from <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.c:7:
./include/linux/mm.h:2476:6: note: declared here
2476 | long get_user_pages(unsigned long start, unsigned long nr_pages,
| ^~~~~~~~~~~~~~
<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-isys-csi2.c:473:1: warning: 'csi2_try_fmt' defined but not used [-Wunused-function]
473 | csi2_try_fmt(struct ipu_isys_video *av,
| ^~~~~~~~~~~~
make[6]: *** [scripts/Makefile.build:251: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6/../ipu-psys.o] Error 1
make[6]: *** Waiting for unfinished jobs....
make[5]: *** [scripts/Makefile.build:485: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel/ipu6] Error 2
make[4]: *** [scripts/Makefile.build:485: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build/drivers/media/pci/intel] Error 2
make[3]: *** [<<DKMSDIR>>/headers/linux-headers-6.4.0-20230706-next/Makefile:2025: <<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build] Error 2
make[2]: *** [Makefile:234: __sub-make] Error 2
make[2]: Leaving directory '<<DKMSDIR>>/headers/linux-headers-6.4.0-20230706-next'
make[1]: *** [Makefile:37: all] Error 2
make[1]: Leaving directory '<<DKMSDIR>>/build/ipu6-drivers/0~git202302081010.7fdfb5eb-0ubuntu0.23.10.1/build'
make: *** [debian/rules.d/2-binary-arch.mk:240: /build/mantic/debian/stamps/stamp-install-next] Error 10
``` |
|
2023-12-01 10:21:24 |
Timo Aaltonen |
ipu6-drivers (Ubuntu Mantic): status |
In Progress |
Won't Fix |
|