(Vulkan) Make we get UniformBlock's type_name for vulkan

Until now, due to some unknown reason, the name of uniform block didn't loaded well.

To fix this issue, let vulkan-reflection use type_name instead of it's name.

Change-Id: Iefc677abaa1832ed579797a3b2e69691151f6408
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

DALi Version 2.4.13

Change-Id: I3bd709990da9dd42903a4f5e1413f438e858f61e

Revert the previous SVACE fix for VMA null termination issue

The previous fix is try to fix the SVACE error to ensure null
termination in VmaStringBuilder::Add by resizing the buffer
and appending '\0' after copying. It's noticed that VMA
statistics string is broken after this fix.

We realised that such fix is wrong, because the data is only
used as an input to the function VmaCreateStringCopy, which
takes explicit length and adds null terminator to the created
copy.

We should ignore this SVACE error, as advised by the author
of VMA:
https://github.com/GPUOpen-LibrariesAndSDKs/VulkanMemoryAllocator/pull/463

Change-Id: I5027a4ebe07a102c05a07108cc0108bcf1d73fc1

Merge "DALi Version 2.4.12" into devel/master

DALi Version 2.4.12

Change-Id: If2031bb3a6e791aae30925bd97f0ea568dd970c2

Add appname and path to Dumptree

This reverts commit e47f563929f6b85fb81b96091a87a746073316d4.

Change-Id: Idfd37fb4ca3dee896693ae9111435e677ae2d5b3

Merge "Skip rendering if renderer never been added" into devel/master

Merge "Fix Svace issue about font variations." into devel/master

Fix Svace issue about font variations.

Change-Id: Ib022243ffd5d8d56e062ab0e1e58e65580f25f5d

Skip rendering if renderer never been added

Change-Id: Ia499b8b8f040503be1f47f913bb4910916f4a8fd
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "Consume exception during DBusServer destruct" into devel/master

Merge changes I549d2fd5,I5348dcc1,Ic3fb46fa,I05c8b768,I8e0b2555 into devel/master

* changes:
  Ensure that RenderScene actually renders something.
  Fixing swapchain buffer count and cmd buffer allocs
  Reworking pre-render/post-render
  Debug extra
  Ensure empty render instruction doesn't swap buffers

Merge "Add Font variations to font client" into devel/master

Consume exception during DBusServer destruct

Change-Id: Ifd513a2f38975268b33611ebcfe0923e943084ed
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Ensure that RenderScene actually renders something.

RenderScene should only get called if there is something to
present and nothing prevents a renderer being drawn.

The only early-out that's allowed now is null scene handle.

ClearScene will only get called if RenderScene did not present
when we expected it to have actually drawn something.

If there are no renderers, but we set the background color of
the window; currently, FullSwapNextFrame is set. If this is not
desired behaviour, it should be changed in
SceneHolder::SetBackgroundColor.

Change-Id: I549d2fd5a25f6a182d89c8c2cc5abfaa07f985c6
Signed-off-by: David Steele <david.steele@samsung.com>

Fixing swapchain buffer count and cmd buffer allocs

Number of swapchain buffers should be based on minImageCount
from surfaceCapabilities, rather than the current number of
images that getSwapchainImagesKHR returns.

Currently, it is possible for the swapchain to get invalidated,
and the new swapchain can be generated with more images than
min. We only need to sync the min number; so we double or triple
buffer based on this count.

(number of CommandBuffers / SwapchainBuffers is based on this min count,
not total image count)

Changed this to only reallocate command buffers in Reset(), rather than
breaking any outstanding sequence.

Change-Id: I5348dcc1762426e5370a35cd7f48d4612a3b5114

Reworking pre-render/post-render

Changed main render loop to check if there is a full-swap
without any rendering (usually caused by having no render-task
yet, but resizing or setting the background color of the window).

If so, we need to ensure that an empty scene can be correctly
rendered. Added a ClearScene to Core API.

This is more pertinent to Vulkan renderer, which has to ensure
that the window's render surface is properly acquired _before_
drawing to it, and presenting that surface needs the right
sync primitives.

Modified VulkanGraphicsController::PresentRenderTarget() to also call
Surface->PostRender(), in the same way that EglGraphicsController does.
(Prelude to other re-work for PartialRendering).

Change-Id: Ic3fb46faa17b3ec67053cfc4fbd48f6a31cc414c

Debug extra

Change-Id: I05c8b768acfdebfce8fa661ff69e6e13d6412a53

Ensure empty render instruction doesn't swap buffers

If we aren't going to draw anything to the surface, then
don't acquire next image or render anything.

SwapBuffers is handled through Swapchain::Present, which maintains
it's own framecount; The current buffer index is retrieved from the
main window's swapchain.

Updated VulkanGraphicsController::PresentRenderTarget to also call
WindowSurface::PostRender; this has a load of handling for partial
rendering that we may need in the future.

Will keep rendering framebuffers, regardless of whether they are drawn
or not.

Change-Id: I8e0b2555647e2ce74d425c9632246f19ea295470

Merge "DALi Version 2.4.11" into devel/master

DALi Version 2.4.11

Change-Id: Ic63a2f9489f166a4d2640f32a881da674223df6d

Fix UNSAFE DirectRendering flush EndRenderPass well

Since Context::ResetGLESState() reset current renderpass and render target info,
it might not worsk well for EndRenderPass stage.

Change-Id: If3223d784fa865565c457986acdfc1fdb40b92fb
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Add Font variations to font client

Change-Id: Ia48dc13f08135650f29fcb811b77e52ffddbd6c7

Merge "Make move operation as noexcept" into devel/master

Merge "Fix ISOLATE DirectRenderingGlView didn't show texture" into devel/master

Revert "Add appname and path to Dumptree"

This reverts commit 928399d10fbc0f1bc32248a6ab907f7b8a8ff9f8.

Change-Id: I3f395482ef9bdd09dfa45ad18473c31322c82713

Fix ISOLATE DirectRenderingGlView didn't show texture

We didn't use native renderer's context even we use ISOLATE mode.

It just breakdown normal dali pipeline codes and textures

Change-Id: I8d3c2f7e9f88a186a8f24c6beaccc4d1a42a21ed
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "Add appname and path to Dumptree" into devel/master

Make move operation as noexcept

It will sightly improve the performance when we use std::vector<>

Change-Id: I2b2e3f34165ff37e52afd7b58aeb299c75c0116d
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

(Automated Tests) Remove build error for gcc-14 : std::find_if return value nodiscard

Since std::find_if return value has no discard option for latest compiler,
automated-tests compile failed due to [-Werror=unused-result]

Change-Id: I35c0b59d0ebf2a53b6f816f0cac9d622fe1a70ae
Signed-off-by: Eunki Hong <eunkiki.hong@samsung.com>

DALi Version 2.4.10

Change-Id: If697ba8485e66d389a95f7c07074104f065a2caf

Error check when jpeg loader return negative size

it is possible that tjPlaneWidth return -1 if it might invalid.

Let we check for tjPlaneSizeYUV, tjPlaneWidth and tjPlaneHeight return
negative value case, so we can avoid unsigned integer overflow error.

Change-Id: I11f2d0b7eec3797a923c18db4d952578ad7767aa
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "Let we make multiple native render surface has seperated display" into devel/master

Let we make multiple native render surface has seperated display

Until now, we make native render surface use single unique dummy buffer.
But if we try to use multiple displays at the same time,
it might use same EGLDisplay, so might have invalid dummy display.

To guard this case, let we keep dummy display staci for it.

Change-Id: If3fdcc9fb4d64a7bb8067452b61fbc18138e3205
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

(Vulkan) Block mAllocator tag setter

Change-Id: I5ac8073a38fb38657709929a054b528bb04f6085
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Add appname and path to Dumptree

Add appname and path keys for making a11y objects.

Change-Id: I51fd77edfe06740f5c4759500d5ba0e6cceacafa

Fix rotation calculation logic for multi window

Change-Id: I7608c89416033968053b51324e2b0bfef20513bd

Merge "Fix issue of the converted geometry for DnD." into devel/master

Merge "DALi Version 2.4.9" into devel/master

DALi Version 2.4.9

Change-Id: I8fd70e99c820a104b9ab74c1ef28df1b8be9b882

Fix issue of the converted geometry for DnD.

Fix issue of the converted geometry for DnD when screen rotation works.

Change-Id: I408c977413db6768f364f4cc4f05b46a854cfaee

Fix mis-implements after change inputStride to inputStrideByte

Some API who call image-operator directly, didn't change the
stride value's as byte scale instead pixel scale.

Let we find all mis-implements points, and change the values name as
strideBytes, instead of width simply

Change-Id: Ib6cccef79bf6f1d3366ee91ef3c812bc0f272880
Signed-off-by: Eunki Hong <eunkiki.hong@samsung.com>

Add ICU class to text abstaction

Dali ICU class dynamically loads icu.so via ICUPLugin of dali-extension.
* ICUPlugin is an abstract interface, used by dali-adaptor to access icu plugin.

Change-Id: I1f2b9c702ee000a13e733cc5c232b7c66d256149
Signed-off-by: Bowon Ryu <bowon.ryu@samsung.com>

Merge "Set correct VMA allocation flag based on the memory usage of the image" into devel/master

Fix coverity issue : Vulkan::SurfaceImpl leak

Change-Id: I18a4ffb39da57715eee4dd6d5ef238a79a8b2801
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Set correct VMA allocation flag based on the memory usage of the image

Change-Id: I1fa3ce0dce4f300167a945ed68422b17e6fdbeca

(GLES::Buffer) Skip recycled buffer initialize multiple times

It is possible that single buffer be recycled used multiple times.
If then, AddBuffer() called multiple time.

Until now, buffer only use single flags, so if buffer recycled 2 times, logic
will be broken.

To avoid this case, let we use recycled counter, and allow to recycle multiple times in a loop

Change-Id: Iac11b2fd2eaf658dabf3ee6f72dcf8ac2759842d
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Catch std::length_error exception at oss.str()

Since we could call PrintSystemError API at destructor,
we should not throw exception whenever we could.

It might be false-positive usually.
But also, we need to ignore the future of false-positive coverity alarm.

Change-Id: I9e316d26e713cf65774d10d99e70de6f0bc7de09
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Fix null termination issue for VMA

Change-Id: Ib31c79fc068168b8a48e2cd6ec1b21f508eb80a3

Merge "Fix SVACE errors in VMA" into devel/master

Fix SVACE errors in VMA

Change-Id: Ie6498d1c5278ac2aa055c7cd9aeafd77b13e8464

DALi Version 2.4.8

Change-Id: Id6919fddfba7f7c8fe30a9b9bd0ad6a25f728ffd

Vulkan Memory Allocator integration

Change-Id: I93d318adf3c67dc32c808d553a322122fa549063

Merge "Make PixelBuffer Stride return byte size, instead of pixel" into devel/master

Re-factoring vk discard queues.

Changed Resource, so now we have ResourceWithDeleter
and ResourceWithoutDeleter, but ResourceBase can be used
as common type with calls to GetAllocationCallbacks and
InvokeDeleter.

(Note, only Program/Pipeline objects currently inherit
from ResourceWithoutDeleter, as these are proxy objects
into a cache)

Means that the DiscardQueue is simplified so it only
deals with ResourceBase objects. (Don't have to do
specialization magic depending on resource type).

Change-Id: If8538fa9bbdae9fb6f5b3aeb183c25b9523c673b

Merge "Fix rarely crash if NativeImage discarded after draw occured" into devel/master

Fix rarely crash if NativeImage discarded after draw occured

It is possible s.t. DiscardNativeImage called before EndRenderPass called.

Until now, we remove the container directly. If this case,
the count of native texture dependency might be zero, so EndRenderPass logic
might be skipped!

To avoid this case, let we don't touch the contianer itself, and just mark
it as synced.

Change-Id: I3171493c0d72ccd58b7bdcacac47876ea32ed1d9
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Reset vertex buffer cache at the end of flush

We only check the internal value of VertexBufferBindingDescriptor.

But if the size of mImpl->mCurrentVertexBufferBindings changed, or cleared,
the vertex buffer's pointer might be same as previos old items.
In this case, we might miss vertex attribute, or VAO relative actions.

To avoid this case, let we clear current binded vertex
whenever if the size of mImpl->mCurrentVertexBufferBindings changed.

Change-Id: I8a4081d9ead39ea9927572876e1a6fe50bef376e
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Do not call glInvalidateFramebuffer only TV profile

Since TV graphic driver might have some issue, it print
some useless gles_error logs when we call glInvalidateFramebuffer.

This log show only for TV case. So let we just ignore them, so reduce GPU
overhead.

Change-Id: If3f10acdc345dfc5c20b7a156dc92d4a2d341e2a
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge changes I724e5d3a,Ie01f9465 into devel/master

* changes:
  N buffering vulkan resource discard queues
  Ensured that swapchain Present is only called when needed

Make PixelBuffer Stride return byte size, instead of pixel

Some cases doesn't have stride byte as multiply of BytesPerPixel().
But until now, PixelBuffer and PixelData only support Stride as pixels count
(It looks since GL_UNPACK_ROW_LENGTH is pixel scale, not a bytes.)

For example, If we have 2 width image with RGB888 format, stride byte
should be 8 byte, which is not a multiply of 3 byte.

To support those cases, let we set stride value as bytes scale.

TODO : For now, we convert from RGB to RGBA at graphics-texture side.
Couldn't we support RGB format at GL standard?

Change-Id: Ide30895239b64a9e1fdfae6c607dec512994c8b3
Signed-off-by: Eunki Hong <eunkiki.hong@samsung.com>

Merge "Add early return condition when there is no fontconfig handle" into devel/master

Merge "Changed command buffer static default initializer" into devel/master

DALi Version 2.4.7

Change-Id: Iac39adf3e326a38aefdf0ba522cb3f7b721e6cfd

N buffering vulkan resource discard queues

As we now have multiple command buffers per frame, it is important
to only discard resources for old frames; we run the GC every time
Submit is called.

So, we need a discard q per buffer index; which may be 0,1 or 0,1,2
depending on platform.

Added a vector of queues that gets automatically allocated as needed.

Change-Id: I724e5d3ae3c5b15f99ab2821ef1c75944b9e24cb
Signed-off-by: David Steele <david.steele@samsung.com>

Ensured that swapchain Present is only called when needed

Changed assert's to DALI_ASSERT_ALWAYS;
Added extra logging.

Change-Id: Ie01f9465c7ee9b1898d2e1a8d7741472f4d192ef
Signed-off-by: David Steele <david.steele@samsung.com>

Merge "Ensured swapchain/surface cleanup on device destruction" into devel/master

Merge "(NativeImageSource) Return nullptr if NativeImageSource::New() failed" into devel/master

Merge "Load the shader using glProgramBinary" into devel/master

Merge "Fix fontclient coverity issue" into devel/master

Add early return condition when there is no fontconfig handle

Change-Id: I4270e9d08185dc5e4c0016bb6b9d5d46f43cbcb5
Signed-off-by: Bowon Ryu <bowon.ryu@samsung.com>

Merge "Separate FontConfig handle" into devel/master

Fix fontclient coverity issue

Change-Id: Ib2c0bfb1f863c49f978a0836e305e9cedddc3824
Signed-off-by: Bowon Ryu <bowon.ryu@samsung.com>

(NativeImageSource) Return nullptr if NativeImageSource::New() failed

For vulkan backend case, NativeImageSource's mImpl return nullptr.

But NativeImageSource object itself is not empty, we can call
NativeImageSource API freely, and got crash.

To avoid these kind of issue, let we return nullptr if NativeImageSource
creation failed. (Similar with NativeImageSourceQueue)

Change-Id: I5113a01bf8183a35a3bcfebcca5b4673db753e49
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Separate FontConfig handle

This patch changes FontClient to create and use a handle to FontConfig.
Previously we always used FontConfig globally.
There was no problem because DALi had already recognized and developed this internally.

However, since FontConfig is a library open to the system,
FontConfig may be used by other libs that dali does not know about in the same process,
which may cause Thread safe problems.

Change-Id: Ie603016f08376f9c5ca389069b6c8da7a345d7a2
Signed-off-by: Bowon Ryu <bowon.ryu@samsung.com>

Merge "(Vulkan::Reflection) Append uniform blocks only 1 times per each programs + Clean up some logic" into devel/master

Merge "(Vulkan) Fix minor coverity issues" into devel/master

Ensured swapchain/surface cleanup on device destruction

VulkanDevice should take ownership of vulkan surface / swapchain pair.
For the moment; have ensured that these are properly disposed of
in the VulkanDevice destructor.

[Could change SurfaceSwapchainPair to use UniquePtr instead of raw ptr,
using Destroy() as deallocator fn, then less work!]

Change-Id: Ibd996cd46d8b06bda201fe87f932153f4096e203
Signed-off-by: David Steele <david.steele@samsung.com>

Merge "Fixing swapchain for use with triple buffering" into devel/master

Merge "Add locale api to text abstraction" into devel/master

(Vulkan) Fix minor coverity issues

1. Non-initalized structs (spriv.h, command-buffer.h)
2. Make mMemory nullptr if allocation failed. (image-impl, buffer-impl)
3. Avoid nullptr copy, or negative length string for some case (texture, shader-impl)

Change-Id: Ib2b99af36d430e4159ef7eb7619b040a08f3c688
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

(Vulkan::Reflection) Append uniform blocks only 1 times per each programs + Clean up some logic

Since we create relfections per each shader at vulkan side,
there was some problem if we try to use same uniform block at both
vertex and fragment shader.

Since current spriv don't know the uniform block's name,
let we just seperate it by binding of layout.

+

Their might be issue if VertexShader code have sampler2D (e.g. Scene3D blendshape)
It is code side bug, and we don't need to append sampler informations
whenever we call BuildReflection().

To make ensure code clean, let we clear mUniformBlocks and mUniformOpaques
at the begin of BuildReflection(), and sort + unique each of them.

TODO : Shouldn't we need to consider multi-set case?

Change-Id: Icece552a1dff9f144be580e93e7f3d65d401cf6a
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Add locale api to text abstraction

Prevents the setlocale system call from being called repeatedly.

Change-Id: Ibd00d7e0f8145a4a6aa305a3ba4997c351acbd18
Signed-off-by: Bowon Ryu <bowon.ryu@samsung.com>

Load the shader using glProgramBinary

If there is a saved shader, load the shader using glProbramBinary

Change-Id: I5db0118d5bcc2f45537172d8b5039afe1c5fc309

Fixing swapchain for use with triple buffering

Some platforms are using triple buffering, but the
SwapchainBuffers only allocate 2. Changed to ensure that
the buffer count matches at least the minimum number of
images returned on swapchain creation.

Fixed coverity error in CreateSwapchainForSurface /
ReplaceSwapchainForSurface.

Change-Id: I6292daec0bf6e568f6818751ef1994c5ce87b24b
Signed-off-by: David Steele <david.steele@samsung.com>

Merge "(Webp) Do not load file again if we know size and frame count" into devel/master

Merge "Collect strerror_r relative error log printing logs" into devel/master

Merge "DALi Version 2.4.6" into devel/master

DALi Version 2.4.6

Change-Id: Idafa6b6b3ec9f545e8351df04313664f2c3a8bd5

Collect strerror_r relative error log printing logs

Let we collect all cases to file description error cases.

Until now, we use strerror_r return type value to print error logs.
But it will not works at XSI base kernel.
And also, strerror_r need a lots of parameters so the line of codes increased
wherever we want to print errno value.

So we can control both GNU and XSI case,
and easly check the error occured point.

Change-Id: Ice950acacf7cefc1e3dc3c1427342b91cfa66b5b
Signed-off-by: Eunki Hong <eunkiki.hong@samsung.com>

(Webp) Do not load file again if we know size and frame count

For single webp case, we release the buffer data after load finished.
But user can request the GetImageSize() and GetImageCount().
In this case, we re-load the webp buffer again, and
fail to release the buffers until loader desturcted.

Change-Id: I04817b9e8c589937b36f821e7653b0257c54e067
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "Fontconfig: Replace FcInitReinitialize with FcInitLoadConfigAndFonts" into devel/master

Fix build error when we use tizen_9.0 repo

Let we add new macro OVER_TIZEN_VERSION_10, and use it for latest APIs

Change-Id: I9d4cbaa523ca5b50eba99441fa45f57580a0dab8
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Fontconfig: Replace FcInitReinitialize with FcInitLoadConfigAndFonts

FcInitReinitialize() causes global font cache and config reload.
which may lead to race conditions and crashes.
Replaced it with FcInitLoadConfigAndFonts() to safely reload config within the current process without affecting others.

Change-Id: Ic5c911f399de141310e99711ae72db64c9dad3cd

Changed command buffer static default initializer

Change-Id: I21e31265e1f6d9604762057c0f7385cd02f0fa60

Merge "Changing swapchain acquisition at frame start" into devel/master

Merge "Fixing coverity errors" into devel/master

Merge "[Vulkan] Reduced queue submissions" into devel/master

Merge "Removing DeviceWaitIdle from Vk::SwapBuffers" into devel/master

Changing swapchain acquisition at frame start

Need to ensure that we don't create end of frame fence if we're not
going to render anything.

But, we do want to have same pipelines for fbos and scene; so we need
to sync swapchain _before_ drawing to fbos.

Ensured that Swapchain owns it's external images.

Change-Id: I4cc64607ad964362cb793f435324110027b45740