Reduce the memory allocation per draw call during Vulkan resource binding

Change-Id: I604d9ccdb204dc9a6ed7bc9b773988e28553c682

Merge "Adds to get window rotation's angle and screen rotation." into devel/master

Merge "Add OverScrolled in WebView" into devel/master

DALi Version 2.4.16

Change-Id: I22ac7002af4ab1a89d902cb38846ca364dcb5e22

Adds to get window rotation's angle and screen rotation.

Adds to get window rotation's angle and screen rotation for Dnd.

Change-Id: I8d95c4ab6e981875ebbf93ac9cd7ff2d134397e6

Add OverScrolled in WebView

Change-Id: Ie4b3b4d33c42d19507a620f48876279e3a534157

Merge "(GLES) Ensure gles framebuffer bound at shared context" into devel/master

Merge "Ignore UniformBlocks for GLES3" into devel/master

Merge "(GLES) Match the binded target and matched target" into devel/master

Merge "Add flag  for precompile shader" into devel/master

(GLES) Ensure gles framebuffer bound at shared context

gles-context have framebuffer dependency cache objects.
But now, it looks that we only allow to Create/Bind the framebuffers
at shared context.

Since we cant destroy the framebuffer at another context,
we should ensure to remove the cache objects at shared conetext.

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

(GLES) Match the binded target and matched target

Until now, we use GL_COPY_WRITE_BUFFER for mapping the buffers.
This target usually be used for glCopyBufferSubData.

Since we have specific destination target per each GLES::Buffer, let we
ensure the matched buffer target and generate them.

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

Ignore UniformBlocks for GLES3

Since ResolveUniformBuffers() time doubled after we use UniformBlock at shader,
let we don't use it and use standalone block instead.

It will be changed after we can keep ther performance.

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

Add flag  for precompile shader

Change-Id: If48a07ac3943fd82f8c316bb68ff4d49701ff405

(Gles) Minor optimize UBOBinding

1.
std::vector<bool> is specialize for bitfields, which has less memory
and might have overhead at get/set. Let we avoid this panalty if possible.

2.
Remove unused member values for UniformBufferBindingDescriptor.

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

DALi Version 2.4.15

Change-Id: Icdb0a705fff5d39fb100868bb841b91cd0246324

Use ResizeUnitialize for UBO and texture bindings

Let we reserve the memory for some bindings - texture and UBO.

At ClearState() phase, we always clear all container, and re-allocate next draw.

It will less allocate, and less touch the count area memory. frequency.

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

DALi Version 2.4.14

Change-Id: I46ebe5569848b07c79abdcdee2899a0c407009bb

Fix coverity issue : Do not use FILE* after fclose()

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

Merge "Reserving descriptor write vector to known size" into devel/master

Merge "Reduce memory consumption by tweaking VMA preferred heap block size" into devel/master

Merge "(MacOS) Fix for startup error on recent mac builds" into devel/master

Merge "Use swap for some cases instead move assign. + UpdateOnce for TextureManager" into devel/master

(MacOS) Fix for startup error on recent mac builds

Change-Id: I8f78ff9c8ec81a1169f46b89af236e0ff7603dcb

Use swap for some cases instead move assign. + UpdateOnce for TextureManager

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

Add more SystemError print logs + Do not use DALI_LOG_WARNING

Print some more system error logs where using fread / fseek / ftell.

+

DALI_LOG_WARNINGS print some logs only for debug modes.
Let we print some critical logs as error level, and verbose logs as debug level.

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

Merge "(Vulkan) Make we get UniformBlock's type_name for vulkan" into devel/master

Reserving descriptor write vector to known size

Change-Id: I3c10c5a5415eb0ac6a78f8901fb43479e13c777c

Merge "(Vulkan) Vulkan setup documents for Ubuntu at README.md" into devel/master

(WebEngine) Fix doxygen 1.9.3 error

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

Merge "(WebView) Update and sync all the latest patches related to WebView that have been modified so far" into devel/master

Merge "Check key is null for UngrabKeyList" into devel/master

Merge "Inverted samplers/bound textures in descriptor binding" into devel/master

(WebView) Update and sync all the latest patches related to WebView that have been modified so far

- Fix for UserMediaPermissionRequest object to be maintained by user
- Add WebView FeedMouseWheel as hidden api
- Add SetVideoHole() for EFL window
- Add WebView DeviceListGet,SetDefaultAudio
- Add APIs for setting IME position, cursor theme name.

Change-Id: I2670d74b9f04a620119bd4fd38ee7fcca0be650c

ClearScene if previous frame presented + Present if dirty rect not empty

Their was some error if rendererAdded, but render instruction is empty.

If then, we don't swap the presented buffer. So latest rendered scene
don't cleared.

To fix this issue, let we acquire next frame if previous scene had some render
instructions to scene, but not now.

Also (for gles case) if we call eglSetDamageRegionKHR, we should call eglSwapBuffer.
If not, 0x3002 error occured.

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

Merge "Let we allow to throw exception at LRUCacheContainer::InternalReserve" into devel/master

Check key is null for UngrabKeyList

We check key is null or not at GrabKeyList API.
Let's follow the logic pairwise.

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

(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>

Reduce memory consumption by tweaking VMA preferred heap block size

Change-Id: I53d21b0e758de494b9eb9f1c97951bda945e59d0

DALi Version 2.4.13

Change-Id: I3bd709990da9dd42903a4f5e1413f438e858f61e

Let we allow to throw exception at LRUCacheContainer::InternalReserve

Coverity said std::vector constructor can throw exception.
We don't need to control those exception cases detail but,
let we just avoid useless exception check for it.

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

(Vulkan) Vulkan setup documents for Ubuntu at README.md

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

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

Inverted samplers/bound textures in descriptor binding

Change-Id: I1c3d57b0a79d11d6118271ba38b1d5948b1dafe3

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>