[Package] Change package dependency.

Previously, the single and pipeline API were separated.
Change the pipeline package contain the single package.

Signed-off-by: gichan <gichan2.jang@samsung.com>

[C-API/Service] Apply Singleton pattern

This patch applies the singleton pattern to MLServiceLevelDB class
so that a single instance exists in runtime.

Signed-off-by: Sangjung Woo <sangjung.woo@samsung.com>

[C-API/Service] Refactoring the service API for maintainability and readability

To support various database systems, this patch defines the interface
class IMLServiceDB and implements MLServiceLevelDB using LevelDB. If the
backend DB is changed, we just add a new class for the new DB without
changing the existing code. That increases the maintainability.
Moreover, this patch updates the signature of user-level APIs to
increase readability.

Signed-off-by: Sangjung Woo <sangjung.woo@samsung.com>

[C-API/TEST] Add various test cases for ML Service API

This patch newly adds the test cases for ML Service APIs. They include
the positive and negative cases for setting, getting, and deleting the
pipeline description.

Signed-off-by: Sangjung Woo <sangjung.woo@samsung.com>

[C-API] Add Error code for DB and filesystem

This patch newly adds the ML_ERROR_IO_ERROR code for the failure
of DB and filesystem operation.

Signed-off-by: Sangjung Woo <sangjung.woo@samsung.com>

[Package] Bugfix for package dependency

To maintain backward compatibility, capi-machine-learning-inference
package should require pipeline, single and common libraries. However,
common is omitted and it can cause break issues when making platform
:1image. This patch fixes this bug.

Signed-off-by: Sangjung Woo <sangjung.woo@samsung.com>

[C-API/Serivce] Update the doxygen tag

This patch updates the doxygen tag and comments for ML Service API.

Signed-off-by: Sangjung Woo <sangjung.woo@samsung.com>

[C-API/Serivce] Set SYS_DB_DIR as build configuration

This patch sets the SYS_DB_DIR as build configuration. In the case
of Tizen, a predefined DB path (i.e. TZ_SYS_GLOBALUSER_DB) is used.

Signed-off-by: Sangjung Woo <sangjung.woo@samsung.com>

[C-API/Service] Add testcase for ML Service API

This patch adds the basic testcase of the ML Service API for adding
and getting the pipeline description with the name.

Signed-off-by: Sangjung Woo <sangjung.woo@samsung.com>

[C-API/Service] implement ML Service API

This patch newly adds the ML Service APIs for adding and getting the
pipeline description with the name. To store the pipeline description,
leveldb is used.

Signed-off-by: Sangjung Woo <sangjung.woo@samsung.com>

[C-API/Service] Fix the compile-time error

Because of an omitted semicolon and header file, there is a compile-time error
when using ml-api-service.h header. This patch fixes this bug.

Signed-off-by: Sangjung Woo <sangjung.woo@samsung.com>

[debian] Rename invalid .install file names

Rename invalid .install file names.
...-dev-api.install -> ..-api-dev.install

Signed-off-by: gichan <gichan2.jang@samsung.com>

[debian] Separate debian packaging

 - Separate single/pipeline debian package
 - Change library name (capi-nnstreamer-* -> ml-inference-*)

Signed-off-by: gichan <gichan2.jang@samsung.com>

[C/inference] Apply Substitute Algorithm instead of Switch-Case statement

This patch does refactoring the code by applying the Substitute
Algorithm technique. Instead of a Switch-Case statement, a hash table is
used and it increases both readability and performance.

Signed-off-by: Sangjung Woo <sangjung.woo@samsung.com>

[C/inference] Refactoring the nested-if statement

A nested if statement makes it hard to understand and maintain the code.
This patch reorganizes the code structure and increases readability.

Signed-off-by: Sangjung Woo <sangjung.woo@samsung.com>

[C/inference] Apply extract method for type converting

Instead of a complicated switch-case statement in the function, this
patch extracts a new function for type converting. Since the enumeration
of both types is the same order, typecasting is good enough for
converting.

Signed-off-by: Sangjung Woo <sangjung.woo@samsung.com>

[meson] change global link -> project link

As recommended from a review in #137, add_global_link_argument has
changed to add_project_link_argument

**Self evaluation:**
1. Build test: [X]Passed [ ]Failed [ ]Skipped
2. Run test: [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Jihoon Lee <jhoon.it.lee@samsung.com>

[Debian] update ml-inference-api-dev package

After #131, ml-inference-api-dev is relying on capi-ml-inference-single
and capi-ml-inference-pipeline pc.

It might have been better that ml-inference-api-dev relying on
ml-inference-single/pipeline-dev. As the package is not introduced,
this patch add pc files into the ml-inference-api-dev package.

See also: https://github.com/nnstreamer/nntrainer/issues/1853#issuecomment-1064220362

**Self evaluation:**
1. Build test: [X]Passed [ ]Failed [ ]Skipped
2. Run test: [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Jihoon Lee <jhoon.it.lee@samsung.com>

Fix libcapi-nnstremer is empty due to linker optimization

From #131, libcapi-nnstreamer.so became empty because
libcapi-nnstreamer.so does not contain any symbol from
libcapi-nnstreamer-single, libcapi-nnstreamer-pipeline.so

This should be semantically resolvable by using 'link-whole' argument in
meson but it is not implemented for shared_library.

This patch address the issue by disabling `-bas-needed=false` in meson
and add -Wl,--as-needed to global linker argument.

https://github.com/mesonbuild/meson/issues/3046.

**Self evaluation:**
1. Build test: [X]Passed [ ]Failed [ ]Skipped
2. Run test: [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Jihoon Lee <jhoon.it.lee@samsung.com>

[Log] log utils from nnstreamer

The log util was separated from nnstreamer repo.
Remove unnecessary header and update log util.

Signed-off-by: Jaeyun <jy1210.jung@samsung.com>

[Package] Separate single-shot and pipeline API.

 - `machine-learning-inference` is separated to `machine-learning-inference-single` and `machine-learning-inference-pipeline`.
 - Separate internal header files.

Signed-off-by: gichan <gichan2.jang@samsung.com>

[build] Build android api with gstreamer v1.20.0

- Assume that android builds are with gstreamer v1.20.0
- Remove the patch file which is no needed for the new version
- Modify the patch file for tracing

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[Tizen/TEST] Add gcov package

Add gcov package for tizen coverage auto measurement tool.

Signed-off-by: Gichan Jang <gichan2.jang@samsung.com>

[C-Api/Test] condition to check allowed element

1. init gstreamer before starting the unittest.
2. check element restriction flag from config file.

Signed-off-by: Jaeyun <jy1210.jung@samsung.com>

[C-Api/CodeClean] internal struct for pipeline

Code clean, cleanup definition for internal data struct.

TODO: We now have a plan to separate single and pipeline source.
Then it is necessary to hide internal data struct for pipeline functions.

Signed-off-by: Jaeyun <jy1210.jung@samsung.com>

[C-Api/Util] util function to get gst element

Add util function to get gst-element instance.

Signed-off-by: Jaeyun <jy1210.jung@samsung.com>

[TEST] Check restricted elements.

Check whether the restricted elements are properly handled or not.

Signed-off-by: Gichan Jang <gichan2.jang@samsung.com>

C/Service: first prototypes of service API Phase 1

This is the start of designing https://github.com/nnstreamer/nnstreamer/issues/3473
Phase 1 and 2 APIs are declared.

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

C/common: refactor data structure locations for service API

A few definitions in pipeline/single APIs are used by service API.
Move such definitions to the common header.

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

[C/inference] Use allowed_elements instead of restricted-elements

The name 'restricted-elements' of nnstreamer configuration is not
matched as its real operation. Because of this reason, its name is
renamed to 'allowed_elements'. This patch updates the related code.

Signed-off-by: Sangjung Woo <sangjung.woo@samsung.com>

[TEST] Add replacing string test

Add replacing string test.
(Copied from nnstreamer/nnstreamer/tests/unitest_util.c)

Signed-off-by: gichan <gichan2.jang@samsung.com>

[android/pytorch] Add pytorch v1.10.1 android support

- Support pytorch v1.10.1 for android (only arm64)

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[android/tflite] Add tflite v2.7.0 support

- Add tflite v2.7.0 support
- Set v2.7.0 as default version of tflite

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[CODEOWNERS] Fix codeowners syntex.

The last matching pattern takes the most precedence.
Change orders.
Refer: https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners

Signed-off-by: Gichan Jang <gichan2.jang@samsung.com>

[Single] Fix coverity issues

 - Remove unrechable code.
 - Fix use after free.

Signed-off-by: Gichan Jang <gichan2.jang@samsung.com>

C/single: elaborate invoke stream-pipe error.

When the framework version does not match the given
model's requirement, STREAM_PIPE error may happen.
Because it appears that such cases are common,
add more explanation.

Related with https://github.com/nnstreamer/nnstreamer/issues/3603

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

[Single] Allow NULL value for some properties.

Allow to set NULL value for some properties such as custom, accelator
and the rest.

Signed-off-by: Gichan Jang <gichan2.jang@samsung.com>

[TEST] Update the test case

Both plugin_availability_fail_invalid_01_n and
plugin_availability_fail_invalid_02_n have the same test code. This
patch updates the parameter to test other case.

Signed-off-by: Sangjung Woo <sangjung.woo@samsung.com>

[CODEOWNER] Fix sourcecode path.

Currently, the source code path of the CODEOWNER is set differently than we were intended.
For example, /c/* mathces files like `c/README.md`
but not further nested files like `c/src/ml-api-common.c`
To include nested files, fix source code path syntex.

Signed-off-by: Gichan Jang <gichan2.jang@samsung.com>

C/common: override dlog infra with stderr for unit testing

With unittesting, especially in GBS, dlog output is
not shown because dlog server is not running.

Replace it with stderr print in unittesting
for better readability in GBS.

This is similar with
https://github.com/nnstreamer/nnstreamer/pull/3571

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

C/Single: elaborate error messages.

For the sake of app developers, let's elaborate error messages.

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

[build/android] Add comments for nnfw and snpe makefiles about external libs

- Add comments for how external libs should be configured.

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[build/android] Fix Pytorch makefile to compatibile with tflite (xnnpack)

- Exclude xnnpack and other prebuilt libs from `LOCAL_WHOLE_STATIC_LIBRARIES`
  to avoid link crash issue with tflite which also contains xnnpack and other common prebuilt libs

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[build/android] Add a comment in file `Android-nnstreamer-prebuilt.mk`

- Add a comment for native developers.

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

C/inference: more errors with elaborated messages.

Elaborate error and exception messages of pipeline API
and internal functions.
Use the new log/report infrastructure.

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

C/common: update error reporting macros

1. Error continue-report only if the op has returned a non-zero value.
2. Commonize formatting macro.

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

[C-Api] Fix the omitted Requires section in capi-ml-inference.pc

Because of the bug in the meson file, Requires section in
capi-ml-inference.pc is empty. It causes the compile-time error when
using ML APIs. This patch fixes this bug.

Signed-off-by: Sangjung Woo <sangjung.woo@samsung.com>

[fix/trivial] Fix unnecessary null check

- Remove unnecessary null check

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[fix/trivial] Fix unreacable code

- Remove unreachable code

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

C/Service: introduction of the new API set.

Declaration of what the "ML Service APIs" will provide.

This is the start of designing https://github.com/nnstreamer/nnstreamer/issues/3473

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

C-API/Common: more elaborated error reporting.

As an example for other API implementations,
error messages of ml-api-common.c are elaborated
with the new error reporting infra.

Unit test cases will follow later.

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

C-API/Common: macro for formatting error messages.

New macros for error reporting infrastructure with some more info.

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

[android/test] Add pipeline api test for mqtt elements

- Add a test for mqtt elements. Because it needs running mqtt broker,
  it should be ignored at this moment.
- At least it is passed in configured environment (mqtt broker is
  installed in the android device and running locally)
- Add permisiion for internet in AndroidManifest.xml

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[android/mqtt] Add MQTT support

- Add MQTT support
- It requires paho.mqtt.c prebuilt lib

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[Build/CodeClean] std headers

Code clean, it is recommended to not use stddef for NDK build, use stdlib instead.

Signed-off-by: Jaeyun <jy1210.jung@samsung.com>

[Spec] Fix the typo in Requires section

This patch fixes the typo in Requires section.
* capi-machine-larning-common -> capi-machine-learning-common

Signed-off-by: Sangjung Woo <sangjung.woo@samsung.com>

[Singleshot] Enhance dim inference for nntrainer

There was a voc that `ml_single_open(&single, 'model.ini', NULL, NULL,
ML_NNFW_TYPE_NNTR_INF, ML_NNFW_HW_ANY);` should work, which is giving
NULL for the tensor_input info and tensor output info.

This was not feasible because the nntrainer inference filter is not
defining `getInputDim()`, `getOutputDim()` on purpose to make it able to
infer output dimension from input dimension in pipeline scenario.

This patch introduce the dimension inferencing mechanism for nntrainer
to make it coherent with the pipeline api.

Below rules apply when input_info or output_info is nullptr for
nntrainer.

1. After opening the framework, allow calling set_input_dim with input_info with
count==0 in case of nntrainer.
2. nntrainer checks if input_info->count==0, if true, nntrainer assums
it should be default dimension and set input/output dimension accordingly.

**Self evaluation:**
1. Build test: [X]Passed [ ]Failed [ ]Skipped
2. Run test: [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Jihoon Lee <jhoon.it.lee@samsung.com>

C-API/Common: add _ prefix for internal functions.

Add _ prefix for all internal functions
so that developers can distinguish external (API) functions
and internal functions easily.

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

C-API/Common: add _ prefix to internal interface - log

Rename mlapi_log* --> _ml_log*

This is the first of adding _ prefix to internal interfaces series.

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

C-API/Common: error reporting API implementation.

API implementation:
ml_error(), ml_strerror()

Internal interface for API implementation:
_ml_error_report()

TODO: apply _ prefix for all internal interfaces.

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

C-API/Common: error reporting function skeleton.

dlerror()-like function that can provide error descriptions and
strerror()-like function that provide general descriptions for error codes.

This is to review its format and behavior.
The contents and internal APIs for API implementation will
follow after this API is approved.

This addresses (starts addressing) https://github.com/nnstreamer/nnstreamer/issues/3528

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

[C/Inference] Refactoring C-API module relations.

ML C-API interfaces and implementations are being refactored for better relations.

ml-common will be no longer depending on nnstreamer.

Then, in the future, nntrainer will be no longer depending on nnstreamer except for its nnstreamer-nntrainer subplugin.

1. Renamed headers and sources (except for Tizen Studio's files, which we need to tend later)
2. Refactored inter-file dependency and relations.

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

[Headers] Fix missing include in ml-api-common

As tensor data structure now being moved to ml-api-common.
`size_t` is now used in `int ml_tensors_info_get_tensor_size`.

Does including <stddef.h> here.

Please  refer to below error in http://nnstreamer.mooo.com/nntrainer/ci/repo-workers/pr-checker/1648-202110201315200.81315588951111-c03945380718edd44c6a19b56056dcf80a64636d/report/build_log_1648_android_error.txt

```
 error: unknown type name 'size_t'
int ml_tensors_info_get_tensor_size (ml_tensors_info_h info, int index, size_t *data_size);
```

Signed-off-by: Jihoon Lee <jhoon.it.lee@samsung.com>

[Common] refactor headers to clean up dependency

Data handling APIs are used by ML.training as well as ML.inference.
Move such APIs to common header.

TODO: Move implementation to another .c file so that
we can refactor subpackage dependencies, too.

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

Release of ML-API 1.8.0

 - Release of ML-API 1.8.0.

Signed-off-by: Gichan Jang <gichan2.jang@samsung.com>

[Single] Don't close single handle during invoke process

Don't close single handle during invoke process.
If single handle is released during invoke prcoess, an interpreter may
access invalid memory.

Signed-off-by: Gichan Jang <gichan2.jang@samsung.com>

[Coverage] Set coverage badge

This patch enables coverage badge. Behind the scene, daily coverage
check is enabled :)

**Self evaluation:**
1. Build test: [ ]Passed [ ]Failed [X]Skipped
2. Run test: [ ]Passed [ ]Failed [X]Skipped

Signed-off-by: Jihoon Lee <jhoon.it.lee@samsung.com>

[TEST] Add checking nnfw availability util

Add checking nnfw availability util.

Signed-off-by: gichan <gichan2.jang@samsung.com>

Unittest: data type consistency with nnstreamer/gstreamer

There are type re-definitions in C ML-API, which
should inherit values from nnstreamer and gstreamer.
Add unittests that ensures the consistency.

Fixes https://github.com/nnstreamer/nnstreamer/issues/2587

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

nnfw_ver to 1.17.0

nnfw(ONE) v1.17.0 is released. This will bump up nnfw version to 1.17.0

Signed-off-by: Chunseok Lee <chunseok.lee@samsung.com>

[C-Api] function to check hw availability

Add new function to check hw availability with custom option string.

Signed-off-by: Jaeyun <jy1210.jung@samsung.com>

[Android/Build] remove release flag

Remove release flag and related source.
Uploading library to jcenter is not available anymore.

Signed-off-by: Jaeyun <jy1210.jung@samsung.com>

[C-API] Change src-callback name

Change src-callback function name.
 - Names of callback function types should have the '_cb' suffix.
 - A modile providing callbacks called on events should support exactly one of two registration schemes.
   > single callback registration: {module}_set_{details}_cb

Signed-off-by: gichan <gichan2.jang@samsung.com>

[Android/Build] dependency to glib

clearly denote dependency to glib in android build.

Signed-off-by: Jaeyun <jy1210.jung@samsung.com>

[C-API] Use ML_NNFW_TYPE_TRIX_ENGINE instead of ML_NNFW_TYPE_TRIx_ENGINE

According to the Tizen naming convention, the names of the entities must
be composed of upper case letters. This patch fixes this issue.

Signed-off-by: Sangjung Woo <sangjung.woo@samsung.com>

[Test/C-Api] src-callback test

Add new testcases for src callback.

Signed-off-by: Jaeyun <jy1210.jung@samsung.com>

[C-API] Rename srnpu to trix-engine

The patch updates the name 'srnpu' to 'trix-engine'.

Signed-off-by: Sangjung Woo <sangjung.woo@samsung.com>

[Test] update flex-tensor caps

Update caps for flexible tensor, old mime type for flex-tensor is removed.

Signed-off-by: Jaeyun <jy1210.jung@samsung.com>

[C-Api] implement src callback

Implement new API - appsrc callbacks.

Signed-off-by: Jaeyun <jy1210.jung@samsung.com>

[Android] Test SNAP with tensorflow lite model

- SNAP v3.0 supports tensorflow lite model.
- Add pipeline and singleshot test using it.

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[C-Api] framework priority from conf

Load framework priority from nnstreamer conf file.

Signed-off-by: Jaeyun <jy1210.jung@samsung.com>

[C-Api] remove warning while checking framework

When checking fw with model files is failed, it returns null and this will print warning message even if this works properly.
Update macro to not print this message. (requested from VD)

Signed-off-by: Jaeyun <jy1210.jung@samsung.com>

[TEST] Add tensor clone test

 - Add tensor clone test

Signed-off-by: Gichan Jang <gichan2.jang@samsung.com>

[debian] Add jni debian package

Add jni wrapper debian package.

Signed-off-by: Gichan Jang <gichan2.jang@samsung.com>

[JNI] remove unnecessary feature in native

We should call subplugin registration only on Android.
Remove unnecessary feature in native code.

Signed-off-by: Jaeyun <jy1210.jung@samsung.com>

[C-Api/Util] Add ML_NNFW_HW_NPU_SLSI case to converting function

To support VD product, this patch adds the ML_NNFW_HW_NPU_SLSI case
to ml_nnfw_to_accl_hw() function.

Signed-off-by: Sangjung Woo <sangjung.woo@samsung.com>

[Build/Java] add script to build jar

Add new script to build java library.
To prevent build error, mark prefix in java files and change it while building the library.

Signed-off-by: Jaeyun <jy1210.jung@samsung.com>

[Java] prepare ubuntu build

Prepare Java API on ubuntu, add feature for android and remove android annotation.

Signed-off-by: Jaeyun <jy1210.jung@samsung.com>

[C-Api] handle flexible tensor

In flexible tensor stream, sink and src handle cannot set exact tensor info from pad caps.
To handle flex-tensor, ignore tensor info while configuring the element handle.

Signed-off-by: Jaeyun <jy1210.jung@samsung.com>

[Android/Build] check target abi

Check target abi when build android library, and set dir name to get gstreamer prebuilt libs.

Signed-off-by: Jaeyun <jy1210.jung@samsung.com>

[Java] build JNI wrapper

Build JNI wrapper using meson.
The source for JNI wrapper is same with Android native code.

Signed-off-by: Jaeyun <jy1210.jung@samsung.com>

[Api/TensorsData] hold tensors info in data handle

Clone tensors-info handle and hold it in tensors-data handle.
Also, pass info handle to parse data object in Android native build.

Signed-off-by: Jaeyun <jy1210.jung@samsung.com>

[C-Api/Util] remove duplicated source

Add private util function and remove duplicated source.

Signed-off-by: Jaeyun <jy1210.jung@samsung.com>

[Android] min sdk version to 24

For android release, set minimum sdk version to 24.

Signed-off-by: Jaeyun <jy1210.jung@samsung.com>

[Data] Unlock data and remove unreachable code

 - Unlock data when policy neither ML_PIPELINE_BUF_POLICY_AUTO_FREE nor
ML_PIPELINE_BUF_POLICY_DO_NOT_FREE.
 - Remove unreachable code

Signed-off-by: Gichan Jang <gichan2.jang@samsung.com>

[Android] nnfw feature

Change nnfw feature definition in android build, same to c-api define.

Signed-off-by: Jaeyun <jy1210.jung@samsung.com>

[android/build] Fix android build script for wget

- Fix wget urls to support git lfs files

Signed-off-by: Yongjoo Ahn <yongjoo1.ahn@samsung.com>

[capi] Update on thread safety for ml_tensors_info

This patch fixes the case of Time of Check Time of Use bug in
ml_pipeline_custom_easy_filter_register().
The provided arguments `in` and `out` are first checked for validity
under their locks and then used for clone again under their locks.
However, between the two operations, both the variable are left lock
free, and the two variables can become invalid.
This patch adds the corresponding fix by adding support for
checking validity of ml_tensors_info_s lockfree which is done
under the lock acquired inside ml_tensors_info_clone().

Further, as a minor fix, redundant locking and unlocking
has been removed from ml_pipeline_custom_easy_filter_register().

See also #42 https://github.com/nnstreamer/api/pull/42#issuecomment-840265534
Resolves https://github.com/nnstreamer/nnstreamer/issues/2600

**Self evaluation:**
1. Build test: [x]Passed [ ]Failed [ ]Skipped
2. Run test: [x]Passed [ ]Failed [ ]Skipped

Signed-off-by: Parichay Kapoor <pk.kapoor@samsung.com>

[C-API] Fix the coverity issue

This patch fixes the coverity issue: "waiting while holding a lock".
Before entering the loop, mutex lock is release so that other thread can
use the CPU.

Signed-off-by: Sangjung Woo <sangjung.woo@samsung.com>

Thread safety for ml_tensors_data APIs

Fixes #nnstreamer/2600
Note that locally created-and-freed ml-tensors-data
are not required to be protected.

Another fix: do not destroy user data even if it is AUTO_FREE
if push_buffer is NOT called.

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>