nvidia-tegra-modules.git/include/media/capture_common.h, branch rtss22-ae NVIDIA's kernel modules to support tegra chips (used in Jetson boards) capture-isp: use separate ringbuffer to pass iovas to rtcpu 2021-06-21T08:55:13+00:00 Kirill Artamonov kartamonov@nvidia.com 2021-06-15T14:13:46+00:00 b3060acfe7d3777892c02f69832f356f52d0dbbc Passing iova pointers using capture descriptor is not safe since usermode application can access them. Use additional ringbuffers shared between kernel and rtcpu to pass iova pointers securely. Related changes and bugfixes: Use permanent vmap mapping for ISP capture and program descriptors This should also improve performance and reduce fragmentation of kernel address space. Don't use relocs. Access descriptor like normal C structure. Remove unused reloc API. Fix map error reporting capture_common_pin_and_get_iova(). Allocate fixed size unpin buffer at setup time istead of relying on kzalloc/kfree during each request. This is to simplify critical runtime codepath and remove flexible array struct member which is prohibited by MISRA C conventions. Add unpin check to prevent submission of requests which are still in progress bug 2768992 bug 3231670 bug 200688288 bug 200690078 CAMASIL-7472 CAMASIL-7474 Signed-off-by: Kirill Artamonov <kartamonov@nvidia.com> Change-Id: Ie2c556cc61334ae478c7edc7e062e16e8b142031 Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2487425 (cherry picked from commit 23186dfdd7b090e6b93c3a6daed60860225b9663) Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2545380 Reviewed-by: Semi Malinen <smalinen@nvidia.com> Reviewed-by: Pekka Pessi <ppessi@nvidia.com> Reviewed-by: Mika Liljeberg <mliljeberg@nvidia.com> Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com> Tested-by: mobile promotions <svcmobile_promotions@nvidia.com> GVS: Gerrit_Virtual_Submit 
Passing iova pointers using capture descriptor is not safe since
usermode application can access them.

Use additional ringbuffers shared between kernel and rtcpu to
pass iova pointers securely.

Related changes and bugfixes:

Use permanent vmap mapping for ISP capture and program descriptors
This should also improve performance and reduce fragmentation
of kernel address space.

Don't use relocs. Access descriptor like normal C structure.
Remove unused reloc API.

Fix map error reporting capture_common_pin_and_get_iova().

Allocate fixed size unpin buffer at setup time istead of
relying on kzalloc/kfree during each request. This is to simplify
critical runtime codepath and remove flexible array struct
member which is prohibited by MISRA C conventions.

Add unpin check to prevent submission of requests which are
still in progress

bug 2768992
bug 3231670
bug 200688288
bug 200690078

CAMASIL-7472
CAMASIL-7474

Signed-off-by: Kirill Artamonov <kartamonov@nvidia.com>
Change-Id: Ie2c556cc61334ae478c7edc7e062e16e8b142031
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2487425
(cherry picked from commit 23186dfdd7b090e6b93c3a6daed60860225b9663)
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2545380
Reviewed-by: Semi Malinen <smalinen@nvidia.com>
Reviewed-by: Pekka Pessi <ppessi@nvidia.com>
Reviewed-by: Mika Liljeberg <mliljeberg@nvidia.com>
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
GVS: Gerrit_Virtual_Submit
capture-vi: use separate ringbuffer to pass iova to rtcpu 2021-06-21T08:55:03+00:00 Kirill Artamonov kartamonov@nvidia.com 2021-06-15T00:11:28+00:00 6e262c637e5116e3057b73788c02358bafe2b725 Passing iova pointers using capture descriptor is not safe since usermode application can access them. Use additional ringbuffer shared between kernel and rtcpu to pass iova pointers securely. Related changes and bugfixes: Use permanent vmap mapping for VI capture descriptors. This should also improve performance and reduce fragmentation of kernel address space. Don't use relocs. Access descriptor like normal C structure. Remove unused parameter capture_data from tegra_channel to avoid confusing null pointer investigation when accidentally using it. Check for zero pin count at submit to avoid potential memory mapping leak which will happen if usermode submits same capture descriptor without waiting for completion. Reboot rtcpu if fail to communicate with rtcpu during channel release IVC call. Allocate fixed size unpin buffer at setup time istead of relying on kzalloc/kfree during each request. This is to simplify critical runtime codepath and remove flexible array struct member which is prohibited by MISRA C conventions. CAMASIL-7466 bug 2768992 bug 3231670 bug 200688288 bug 200690078 CAMASIL-7460 CAMASIL-7477 CAMASIL-4619 CAMASIL-7465 Change-Id: Ia39de887093bbc158c72008ae387287ce30ebd82 Signed-off-by: Kirill Artamonov <kartamonov@nvidia.com> Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2475799 (cherry picked from commit 71ad1f9d54a45e54767d5fa9411a4aee83374398) Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2545006 Reviewed-by: Igor Mitsyanko <imitsyanko@nvidia.com> Reviewed-by: Semi Malinen <smalinen@nvidia.com> Reviewed-by: Pekka Pessi <ppessi@nvidia.com> Reviewed-by: Mika Liljeberg <mliljeberg@nvidia.com> Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com> Tested-by: Igor Mitsyanko <imitsyanko@nvidia.com> Tested-by: mobile promotions <svcmobile_promotions@nvidia.com> GVS: Gerrit_Virtual_Submit 
Passing iova pointers using capture descriptor is not safe since
usermode application can access them.

Use additional ringbuffer shared between kernel and rtcpu to
pass iova pointers securely.

Related changes and bugfixes:

Use permanent vmap mapping for VI capture descriptors.
This should also improve performance and reduce fragmentation
of kernel address space.

Don't use relocs. Access descriptor like normal C structure.

Remove unused parameter capture_data from tegra_channel to
avoid confusing null pointer investigation when accidentally
using it.

Check for zero pin count at submit to avoid potential memory
mapping leak which will happen if usermode submits same capture
descriptor without waiting for completion.

Reboot rtcpu if fail to communicate with rtcpu during
channel release IVC call.

Allocate fixed size unpin buffer at setup time istead of
relying on kzalloc/kfree during each request. This is to simplify
critical runtime codepath and remove flexible array struct
member which is prohibited by MISRA C conventions.

CAMASIL-7466

bug 2768992
bug 3231670
bug 200688288
bug 200690078
CAMASIL-7460
CAMASIL-7477
CAMASIL-4619
CAMASIL-7465

Change-Id: Ia39de887093bbc158c72008ae387287ce30ebd82
Signed-off-by: Kirill Artamonov <kartamonov@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2475799
(cherry picked from commit 71ad1f9d54a45e54767d5fa9411a4aee83374398)
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2545006
Reviewed-by: Igor Mitsyanko <imitsyanko@nvidia.com>
Reviewed-by: Semi Malinen <smalinen@nvidia.com>
Reviewed-by: Pekka Pessi <ppessi@nvidia.com>
Reviewed-by: Mika Liljeberg <mliljeberg@nvidia.com>
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: Igor Mitsyanko <imitsyanko@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
GVS: Gerrit_Virtual_Submit
nvcapture: capture buffer management 2021-06-21T08:54:53+00:00 Ziqi Qing zqing@nvidia.com 2019-03-14T23:46:21+00:00 582806934788414445291cb103f8560bfd1618d6 Buffer management in KMD allow client to pin and setup a set of surface before any capture request. Also keep the compatibility to pin the surface when it is not registered before. - Add ioctl to handle buffer pin request. - Add capture buffer table. Bug 2462034 Change-Id: I71021a241aab4e61d27bd504fb5657b169e56643 Signed-off-by: Ziqi Qing <zqing@nvidia.com> Reviewed-on: https://git-master.nvidia.com/r/1972655 (cherry picked from commit 05c14274b4070aaa0fd02f07166554d07ede85ec) Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2545282 Reviewed-by: svc-mobile-coverity <svc-mobile-coverity@nvidia.com> Reviewed-by: Kirill Artamonov <kartamonov@nvidia.com> Reviewed-by: Igor Mitsyanko <imitsyanko@nvidia.com> Reviewed-by: Semi Malinen <smalinen@nvidia.com> Reviewed-by: Pekka Pessi <ppessi@nvidia.com> Reviewed-by: Mika Liljeberg <mliljeberg@nvidia.com> Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com> Tested-by: Kirill Artamonov <kartamonov@nvidia.com> Tested-by: Igor Mitsyanko <imitsyanko@nvidia.com> Tested-by: mobile promotions <svcmobile_promotions@nvidia.com> GVS: Gerrit_Virtual_Submit 
Buffer management in KMD allow client to pin and setup a set of surface
before any capture request. Also keep the compatibility to pin the
surface when it is not registered before.

- Add ioctl to handle buffer pin request.
- Add capture buffer table.

Bug 2462034

Change-Id: I71021a241aab4e61d27bd504fb5657b169e56643
Signed-off-by: Ziqi Qing <zqing@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/1972655
(cherry picked from commit 05c14274b4070aaa0fd02f07166554d07ede85ec)
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2545282
Reviewed-by: svc-mobile-coverity <svc-mobile-coverity@nvidia.com>
Reviewed-by: Kirill Artamonov <kartamonov@nvidia.com>
Reviewed-by: Igor Mitsyanko <imitsyanko@nvidia.com>
Reviewed-by: Semi Malinen <smalinen@nvidia.com>
Reviewed-by: Pekka Pessi <ppessi@nvidia.com>
Reviewed-by: Mika Liljeberg <mliljeberg@nvidia.com>
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: Kirill Artamonov <kartamonov@nvidia.com>
Tested-by: Igor Mitsyanko <imitsyanko@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
GVS: Gerrit_Virtual_Submit
tegra: camera: Progress status buffer support 2018-08-23T17:53:21+00:00 Andrew Jiang andrewj@nvidia.com 2018-07-11T07:03:38+00:00 19fcb5e39cfba50310ed8324a42793ff3ac0738c Add necessary structs and ioctl calls to allow KMD to use a new progress status buffer allocated by UMD as a convenient way to communicate the status of each request. This is part of supporting the new detached mode mechanism. Bug 2173282 Change-Id: Id279af4e7541d3b9f30e0b68bc838d924e5ab234 Signed-off-by: Andrew Jiang <andrewj@nvidia.com> Reviewed-on: https://git-master.nvidia.com/r/1750545 Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com> Tested-by: mobile promotions <svcmobile_promotions@nvidia.com> 
Add necessary structs and ioctl calls to allow KMD to use a new
progress status buffer allocated by UMD as a convenient way to
communicate the status of each request. This is part of
supporting the new detached mode mechanism.

Bug 2173282

Change-Id: Id279af4e7541d3b9f30e0b68bc838d924e5ab234
Signed-off-by: Andrew Jiang <andrewj@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/1750545
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
media: tegra: camera: refactor pin reloc APIs 2017-11-14T17:53:43+00:00 Sudhir Vyas svyas@nvidia.com 2017-11-09T13:55:43+00:00 cc210201488de431c0602603f0b9a5a24925be81 Refactor pin and reloc API functions. Also rebase vi capture driver to capture common. Jira CRTC-1472 Change-Id: I4f6eeba8e0637ea864a8b8c3ad1220b2770e9ad1 Signed-off-by: Sudhir Vyas <svyas@nvidia.com> Reviewed-on: https://git-master.nvidia.com/r/1595206 Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com> Tested-by: mobile promotions <svcmobile_promotions@nvidia.com> 
Refactor pin and reloc API functions.
Also rebase vi capture driver to capture common.

Jira CRTC-1472

Change-Id: I4f6eeba8e0637ea864a8b8c3ad1220b2770e9ad1
Signed-off-by: Sudhir Vyas <svyas@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/1595206
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
media: platform: tegra: Add capture isp driver 2017-09-25T11:35:49+00:00 Sudhir Vyas svyas@nvidia.com 2017-06-15T13:26:23+00:00 1d43c1d5d8c14765045eca22fc7eef47c1862270 Jira CRTC-1184 Jira CRTC-1185 Jira CRTC-1186 Change-Id: I87ac1a9ff83c3c68aa5a5bdeadc79188b074c675 Signed-off-by: Sudhir Vyas <svyas@nvidia.com> Reviewed-on: https://git-master.nvidia.com/r/1503123 Reviewed-by: svccoveritychecker <svccoveritychecker@nvidia.com> Reviewed-by: Saleh Dindar <sdindar@nvidia.com> GVS: Gerrit_Virtual_Submit Reviewed-by: Sachin Nikam <snikam@nvidia.com> 
Jira CRTC-1184
Jira CRTC-1185
Jira CRTC-1186

Change-Id: I87ac1a9ff83c3c68aa5a5bdeadc79188b074c675
Signed-off-by: Sudhir Vyas <svyas@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/1503123
Reviewed-by: svccoveritychecker <svccoveritychecker@nvidia.com>
Reviewed-by: Saleh Dindar <sdindar@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: Sachin Nikam <snikam@nvidia.com>