diff options
author | Linux Build Service Account <lnxbuild@localhost> | 2016-09-01 18:42:58 -0600 |
---|---|---|
committer | Linux Build Service Account <lnxbuild@localhost> | 2016-09-01 18:42:58 -0600 |
commit | 57aaa5e09827b47df67e08ec165173327a7e6ed8 (patch) | |
tree | 9322ba200099a7920bc2d97647489582b9877e10 | |
parent | 9cc01011e580a2a55ebabd39c62e042f3f9d5dde (diff) | |
parent | bee9c1acdd3a6910d5f4e57d39c4fe4d0ea9909d (diff) | |
download | hardware_qcom_display-57aaa5e09827b47df67e08ec165173327a7e6ed8.tar.gz hardware_qcom_display-57aaa5e09827b47df67e08ec165173327a7e6ed8.tar.bz2 hardware_qcom_display-57aaa5e09827b47df67e08ec165173327a7e6ed8.zip |
Promotion of display.lnx.2.0-00048.
CRs Change ID Subject
--------------------------------------------------------------------------------------------------------------
1040942 I28d757af4178f581e6a83dc06198106c85fc7262 hwc: Store real buffer resolution in private handle.
1040942 Ic4c03cc5779418959732332d26b8ecb59c2483b5 sdm: Add rect mapping and interface to store actual reso
Change-Id: I96c075e51d992f1db521b2f8649913c152439798
CRs-Fixed: 1040942
-rw-r--r-- | libcopybit/copybit_c2d.cpp | 2 | ||||
-rw-r--r-- | libgralloc/alloc_controller.cpp | 46 | ||||
-rw-r--r-- | libgralloc/framebuffer.cpp | 5 | ||||
-rw-r--r-- | libgralloc/gpu.cpp | 18 | ||||
-rw-r--r-- | libgralloc/gr.h | 8 | ||||
-rw-r--r-- | libgralloc/gralloc_priv.h | 13 | ||||
-rw-r--r-- | libgralloc/mapper.cpp | 9 | ||||
-rw-r--r-- | sdm/include/core/layer_buffer.h | 6 | ||||
-rw-r--r-- | sdm/include/utils/rect.h | 2 | ||||
-rw-r--r-- | sdm/libs/hwc/hwc_color_manager.cpp | 3 | ||||
-rw-r--r-- | sdm/libs/hwc/hwc_display.cpp | 25 | ||||
-rw-r--r-- | sdm/libs/hwc/hwc_display_virtual.cpp | 17 | ||||
-rw-r--r-- | sdm/libs/utils/rect.cpp | 20 |
13 files changed, 133 insertions, 41 deletions
diff --git a/libcopybit/copybit_c2d.cpp b/libcopybit/copybit_c2d.cpp index 240ba26c6..ffe5a0aca 100644 --- a/libcopybit/copybit_c2d.cpp +++ b/libcopybit/copybit_c2d.cpp @@ -1174,6 +1174,7 @@ static int stretch_copybit_internal( bufferInfo dst_info; populate_buffer_info(dst, dst_info); private_handle_t* dst_hnd = new private_handle_t(-1, 0, 0, 0, dst_info.format, + dst_info.width, dst_info.height, dst_info.width, dst_info.height); if (dst_hnd == NULL) { ALOGE("%s: dst_hnd is null", __FUNCTION__); @@ -1254,6 +1255,7 @@ static int stretch_copybit_internal( bufferInfo src_info; populate_buffer_info(src, src_info); private_handle_t* src_hnd = new private_handle_t(-1, 0, 0, 0, src_info.format, + src_info.width, src_info.height, src_info.width, src_info.height); if (NULL == src_hnd) { ALOGE("%s: src_hnd is null", __FUNCTION__); diff --git a/libgralloc/alloc_controller.cpp b/libgralloc/alloc_controller.cpp index 39acf6624..4ca4e2d55 100644 --- a/libgralloc/alloc_controller.cpp +++ b/libgralloc/alloc_controller.cpp @@ -173,6 +173,17 @@ void AdrenoMemInfo::getAlignedWidthAndHeight(const private_handle_t *hnd, int& a } +void AdrenoMemInfo::getRealWidthAndHeight(const private_handle_t *hnd, int& real_w, int& real_h) { + MetaData_t *metadata = (MetaData_t *)hnd->base_metadata; + if(metadata && metadata->operation & UPDATE_BUFFER_GEOMETRY) { + real_w = metadata->bufferDim.sliceWidth; + real_h = metadata->bufferDim.sliceHeight; + } else { + real_w = hnd->real_width; + real_h = hnd->real_height; + } +} + bool isUncompressedRgbFormat(int format) { bool is_rgb_format = false; @@ -693,6 +704,16 @@ unsigned int getBufferSizeAndDimensions(int width, int height, int format, return size; } +void getAlignedWidthAndHeight(int width, int height, int format, + int usage, int& alignedw, int &alignedh) +{ + AdrenoMemInfo::getInstance().getAlignedWidthAndHeight(width, + height, + format, + usage, + alignedw, + alignedh); +} void getBufferAttributes(int width, int height, int format, int usage, int& alignedw, int &alignedh, int& tiled, unsigned int& size) @@ -714,6 +735,8 @@ int getYUVPlaneInfo(private_handle_t* hnd, struct android_ycbcr* ycbcr) int width = hnd->width; int height = hnd->height; int format = hnd->format; + int real_width = hnd->real_width; + int real_height = hnd->real_height; unsigned int ystride, cstride; unsigned int alignment = 4096; @@ -734,8 +757,11 @@ int getYUVPlaneInfo(private_handle_t* hnd, struct android_ycbcr* ycbcr) usage = GRALLOC_USAGE_PRIVATE_ALLOC_UBWC; } - AdrenoMemInfo::getInstance().getAlignedWidthAndHeight(metadata->bufferDim.sliceWidth, - metadata->bufferDim.sliceHeight, format, usage, width, height); + real_width = metadata->bufferDim.sliceWidth; + real_height = metadata->bufferDim.sliceHeight; + + AdrenoMemInfo::getInstance().getAlignedWidthAndHeight(real_width, real_height, + format, usage, width, height); } // Get the chroma offsets from the handle width/height. We take advantage @@ -762,16 +788,16 @@ int getYUVPlaneInfo(private_handle_t* hnd, struct android_ycbcr* ycbcr) unsigned int y_stride, y_height, y_size; unsigned int c_meta_stride, c_meta_height, c_meta_size; - y_meta_stride = VENUS_Y_META_STRIDE(COLOR_FMT_NV12_UBWC, width); - y_meta_height = VENUS_Y_META_SCANLINES(COLOR_FMT_NV12_UBWC, height); + y_meta_stride = VENUS_Y_META_STRIDE(COLOR_FMT_NV12_UBWC, real_width); + y_meta_height = VENUS_Y_META_SCANLINES(COLOR_FMT_NV12_UBWC, real_height); y_meta_size = ALIGN((y_meta_stride * y_meta_height), alignment); - y_stride = VENUS_Y_STRIDE(COLOR_FMT_NV12_UBWC, width); - y_height = VENUS_Y_SCANLINES(COLOR_FMT_NV12_UBWC, height); + y_stride = VENUS_Y_STRIDE(COLOR_FMT_NV12_UBWC, real_width); + y_height = VENUS_Y_SCANLINES(COLOR_FMT_NV12_UBWC, real_height); y_size = ALIGN((y_stride * y_height), alignment); - c_meta_stride = VENUS_UV_META_STRIDE(COLOR_FMT_NV12_UBWC, width); - c_meta_height = VENUS_UV_META_SCANLINES(COLOR_FMT_NV12_UBWC, height); + c_meta_stride = VENUS_UV_META_STRIDE(COLOR_FMT_NV12_UBWC, real_width); + c_meta_height = VENUS_UV_META_SCANLINES(COLOR_FMT_NV12_UBWC, real_height); c_meta_size = ALIGN((c_meta_stride * c_meta_height), alignment); ycbcr->y = (void*)(hnd->base + y_meta_size); @@ -779,7 +805,7 @@ int getYUVPlaneInfo(private_handle_t* hnd, struct android_ycbcr* ycbcr) ycbcr->cr = (void*)(hnd->base + y_meta_size + y_size + c_meta_size + 1); ycbcr->ystride = y_stride; - ycbcr->cstride = VENUS_UV_STRIDE(COLOR_FMT_NV12_UBWC, width); + ycbcr->cstride = VENUS_UV_STRIDE(COLOR_FMT_NV12_UBWC, real_width); ycbcr->chroma_step = 2; break; @@ -856,7 +882,7 @@ int alloc_buffer(private_handle_t **pHnd, int w, int h, int format, int usage) private_handle_t* hnd = new private_handle_t(data.fd, data.size, data.allocType, 0, format, - alignedw, alignedh); + alignedw, alignedh, w, h); hnd->base = (uint64_t) data.base; hnd->offset = data.offset; hnd->gpuaddr = 0; diff --git a/libgralloc/framebuffer.cpp b/libgralloc/framebuffer.cpp index 5c297c1f8..94a72a757 100644 --- a/libgralloc/framebuffer.cpp +++ b/libgralloc/framebuffer.cpp @@ -352,8 +352,9 @@ int mapFrameBufferLocked(framebuffer_device_t *dev) // Create framebuffer handle using the ION fd module->framebuffer = new private_handle_t(fd, fbSize, private_handle_t::PRIV_FLAGS_USES_ION, - BUFFER_TYPE_UI, - module->fbFormat, info.xres, info.yres); + BUFFER_TYPE_UI, module->fbFormat, + info.xres, info.yres, + info.xres, info.yres); module->framebuffer->base = uint64_t(vaddr); memset(vaddr, 0, fbSize); //Enable vsync diff --git a/libgralloc/gpu.cpp b/libgralloc/gpu.cpp index 0eee65e53..4f29a7003 100644 --- a/libgralloc/gpu.cpp +++ b/libgralloc/gpu.cpp @@ -53,6 +53,16 @@ int gpu_context_t::gralloc_alloc_buffer(unsigned int size, int usage, { int err = 0; int flags = 0; + int alignedw = 0; + int alignedh = 0; + + AdrenoMemInfo::getInstance().getAlignedWidthAndHeight(width, + height, + format, + usage, + alignedw, + alignedh); + size = roundUpToPageSize(size); alloc_data data; data.offset = 0; @@ -159,8 +169,8 @@ int gpu_context_t::gralloc_alloc_buffer(unsigned int size, int usage, flags |= data.allocType; uint64_t eBaseAddr = (uint64_t)(eData.base) + eData.offset; private_handle_t *hnd = new private_handle_t(data.fd, size, flags, - bufferType, format, width, height, eData.fd, eData.offset, - eBaseAddr); + bufferType, format, alignedw, alignedh, width, height, eData.fd, + eData.offset, eBaseAddr); hnd->offset = data.offset; hnd->base = (uint64_t)(data.base) + data.offset; @@ -228,7 +238,7 @@ int gpu_context_t::gralloc_alloc_framebuffer_locked(int usage, private_handle_t::PRIV_FLAGS_USES_ION | private_handle_t::PRIV_FLAGS_FRAMEBUFFER, BUFFER_TYPE_UI, m->fbFormat, m->info.xres, - m->info.yres); + m->info.yres, m->info.xres, m->info.yres); // find a free slot for (uint32_t i=0 ; i<numBuffers ; i++) { @@ -328,7 +338,7 @@ int gpu_context_t::alloc_impl(int w, int h, int format, int usage, err = gralloc_alloc_framebuffer(usage, pHandle); } else { err = gralloc_alloc_buffer(size, usage, pHandle, bufferType, - grallocFormat, alignedw, alignedh); + grallocFormat, w, h); } if (err < 0) { diff --git a/libgralloc/gr.h b/libgralloc/gr.h index 1b8d9b4a5..d05055bfa 100644 --- a/libgralloc/gr.h +++ b/libgralloc/gr.h @@ -140,6 +140,14 @@ class AdrenoMemInfo : public android::Singleton <AdrenoMemInfo> int tileEnabled, int& alignedw, int &alignedh); /* + * Function to compute real width and real height based on + * private handle + * + * @return real width, real height + */ + void getRealWidthAndHeight(const private_handle_t *hnd, int& real_w, int& real_h); + + /* * Function to return whether GPU support MacroTile feature * * @return >0 : supported diff --git a/libgralloc/gralloc_priv.h b/libgralloc/gralloc_priv.h index 7a5fc845a..4aa47541b 100644 --- a/libgralloc/gralloc_priv.h +++ b/libgralloc/gralloc_priv.h @@ -222,8 +222,10 @@ struct private_handle_t : public native_handle { // The gpu address mapped into the mmu. uint64_t gpuaddr __attribute__((aligned(8))); int format; - int width; - int height; + int width; // specifies aligned width + int height; // specifies aligned height + int real_width; + int real_height; uint64_t base_metadata __attribute__((aligned(8))); #ifdef __cplusplus @@ -235,13 +237,14 @@ struct private_handle_t : public native_handle { static const int sMagic = 'gmsm'; private_handle_t(int fd, unsigned int size, int flags, int bufferType, - int format, int width, int height, int eFd = -1, + int format, int aligned_width, int aligned_height, + int width, int height, int eFd = -1, unsigned int eOffset = 0, uint64_t eBase = 0) : fd(fd), fd_metadata(eFd), magic(sMagic), flags(flags), size(size), offset(0), bufferType(bufferType), base(0), offset_metadata(eOffset), gpuaddr(0), - format(format), width(width), height(height), - base_metadata(eBase) + format(format), width(aligned_width), height(aligned_height), + real_width(width), real_height(height), base_metadata(eBase) { version = (int) sizeof(native_handle); numInts = sNumInts(); diff --git a/libgralloc/mapper.cpp b/libgralloc/mapper.cpp index 142586b83..549951bc3 100644 --- a/libgralloc/mapper.cpp +++ b/libgralloc/mapper.cpp @@ -309,6 +309,7 @@ int gralloc_perform(struct gralloc_module_t const* module, int width = va_arg(args, int); int height = va_arg(args, int); int format = va_arg(args, int); + int alignedw = 0, alignedh = 0; native_handle_t** handle = va_arg(args, native_handle_t**); private_handle_t* hnd = (private_handle_t*)native_handle_create( @@ -321,8 +322,12 @@ int gralloc_perform(struct gralloc_module_t const* module, hnd->offset = offset; hnd->base = uint64_t(base) + offset; hnd->gpuaddr = 0; - hnd->width = width; - hnd->height = height; + AdrenoMemInfo::getInstance().getAlignedWidthAndHeight(width, + height, format, 0, alignedw, alignedh); + hnd->width = alignedw; + hnd->height = alignedh; + hnd->real_width = width; + hnd->real_height = height; hnd->format = format; *handle = (native_handle_t *)hnd; res = 0; diff --git a/sdm/include/core/layer_buffer.h b/sdm/include/core/layer_buffer.h index c437dab29..5ee441e4d 100644 --- a/sdm/include/core/layer_buffer.h +++ b/sdm/include/core/layer_buffer.h @@ -183,8 +183,10 @@ struct LayerBufferFlags { @sa LayerStack */ struct LayerBuffer { - uint32_t width = 0; //!< Actual width of the Layer that this buffer is for. - uint32_t height = 0; //!< Actual height of the Layer that this buffer is for. + uint32_t width = 0; //!< Aligned width of the Layer that this buffer is for. + uint32_t height = 0; //!< Aligned height of the Layer that this buffer is for. + uint32_t real_width = 0; //!< Real width of the Layer that this buffer is for. + uint32_t real_height = 0; //!< Real height of the Layer that this buffer is for. LayerBufferFormat format = kFormatRGBA8888; //!< Format of the buffer content. LayerBufferPlane planes[4]; //!< Array of planes that this buffer contains. RGB buffer formats //!< have 1 plane whereas YUV buffer formats may have upto 4 planes. diff --git a/sdm/include/utils/rect.h b/sdm/include/utils/rect.h index c19864560..7153f2b47 100644 --- a/sdm/include/utils/rect.h +++ b/sdm/include/utils/rect.h @@ -49,6 +49,8 @@ namespace sdm { bool flip_horizontal, LayerRect *out_rects); void SplitTopBottom(const LayerRect &in_rect, uint32_t split_count, uint32_t align_y, bool flip_horizontal, LayerRect *out_rects); + void MapRect(const LayerRect &src_domain, const LayerRect &dst_domain, const LayerRect &in_rect, + LayerRect *out_rect); } // namespace sdm #endif // __RECT_H__ diff --git a/sdm/libs/hwc/hwc_color_manager.cpp b/sdm/libs/hwc/hwc_color_manager.cpp index 5abe94a42..844c2b873 100644 --- a/sdm/libs/hwc/hwc_color_manager.cpp +++ b/sdm/libs/hwc/hwc_color_manager.cpp @@ -259,7 +259,8 @@ int HWCColorManager::CreateSolidFillLayers(HWCDisplay *hwc_display) { // handle for solid fill layer with fd = -1. private_handle_t *handle = new private_handle_t(-1, 0, private_handle_t::PRIV_FLAGS_FRAMEBUFFER, BUFFER_TYPE_UI, - HAL_PIXEL_FORMAT_RGBA_8888, primary_width, primary_height); + HAL_PIXEL_FORMAT_RGBA_8888, primary_width, primary_height, + primary_width, primary_height); if (!buf || !handle) { DLOGE("Failed to allocate memory."); diff --git a/sdm/libs/hwc/hwc_display.cpp b/sdm/libs/hwc/hwc_display.cpp index c6151e640..270e8f393 100644 --- a/sdm/libs/hwc/hwc_display.cpp +++ b/sdm/libs/hwc/hwc_display.cpp @@ -414,8 +414,17 @@ int HWCDisplay::PrepareLayerParams(hwc_layer_1_t *hwc_layer, Layer *layer) { if (pvt_handle) { layer_buffer->format = GetSDMFormat(pvt_handle->format, pvt_handle->flags); - layer_buffer->width = pvt_handle->width; - layer_buffer->height = pvt_handle->height; + int aligned_width, aligned_height; + int real_width, real_height; + + AdrenoMemInfo::getInstance().getAlignedWidthAndHeight(pvt_handle, aligned_width, + aligned_height); + AdrenoMemInfo::getInstance().getRealWidthAndHeight(pvt_handle, real_width, real_height); + + layer_buffer->width = aligned_width; + layer_buffer->height = aligned_height; + layer_buffer->real_width = real_width; + layer_buffer->real_height = real_height; if (SetMetaData(pvt_handle, layer) != kErrorNone) { return -EINVAL; @@ -466,6 +475,8 @@ int HWCDisplay::PrepareLayerParams(hwc_layer_1_t *hwc_layer, Layer *layer) { usage, aligned_width, aligned_height); layer_buffer->width = aligned_width; layer_buffer->height = aligned_height; + layer_buffer->real_width = x_pixels; + layer_buffer->real_height = y_pixels; layer_buffer->format = GetSDMFormat(format, flags); } } @@ -569,6 +580,8 @@ int HWCDisplay::PrePrepareLayerStack(hwc_display_contents_1_t *content_list) { LayerBuffer *input_buffer = layer.input_buffer; input_buffer->width = layer.dst_rect.right - layer.dst_rect.left; input_buffer->height = layer.dst_rect.bottom - layer.dst_rect.top; + input_buffer->real_width = input_buffer->width; + input_buffer->real_height = input_buffer->height; layer.src_rect.left = 0; layer.src_rect.top = 0; layer.src_rect.right = input_buffer->width; @@ -1362,14 +1375,6 @@ DisplayError HWCDisplay::SetMetaData(const private_handle_t *pvt_handle, Layer * layer_buffer->format = GetSDMFormat(meta_data->linearFormat, 0); } - if (meta_data->operation & UPDATE_BUFFER_GEOMETRY) { - int actual_width = pvt_handle->width; - int actual_height = pvt_handle->height; - AdrenoMemInfo::getInstance().getAlignedWidthAndHeight(pvt_handle, actual_width, actual_height); - layer_buffer->width = actual_width; - layer_buffer->height = actual_height; - } - if (meta_data->operation & SET_SINGLE_BUFFER_MODE) { layer->flags.single_buffer = meta_data->isSingleBufferMode; // Graphics can set this operation on all types of layers including FB and set the actual value diff --git a/sdm/libs/hwc/hwc_display_virtual.cpp b/sdm/libs/hwc/hwc_display_virtual.cpp index 47d78f257..712f02421 100644 --- a/sdm/libs/hwc/hwc_display_virtual.cpp +++ b/sdm/libs/hwc/hwc_display_virtual.cpp @@ -269,12 +269,19 @@ int HWCDisplayVirtual::SetOutputBuffer(hwc_display_contents_1_t *content_list) { return -EINVAL; } - int output_buffer_width, output_buffer_height; - AdrenoMemInfo::getInstance().getAlignedWidthAndHeight(output_handle, output_buffer_width, - output_buffer_height); + int aligned_width, aligned_height; + int real_width, real_height; + + AdrenoMemInfo::getInstance().getAlignedWidthAndHeight(output_handle, aligned_width, + aligned_height); + AdrenoMemInfo::getInstance().getRealWidthAndHeight(output_handle, real_width, real_height); + + output_buffer_->width = aligned_width; + output_buffer_->height = aligned_height; + + output_buffer_->real_width = real_width; + output_buffer_->real_height = real_height; - output_buffer_->width = output_buffer_width; - output_buffer_->height = output_buffer_height; output_buffer_->flags.secure = 0; output_buffer_->flags.video = 0; diff --git a/sdm/libs/utils/rect.cpp b/sdm/libs/utils/rect.cpp index e756464e6..5392aaa4b 100644 --- a/sdm/libs/utils/rect.cpp +++ b/sdm/libs/utils/rect.cpp @@ -198,5 +198,25 @@ void SplitTopBottom(const LayerRect &in_rect, uint32_t split_count, uint32_t ali } } +void MapRect(const LayerRect &src_domain, const LayerRect &dst_domain, const LayerRect &in_rect, + LayerRect *out_rect) { + if (!IsValid(src_domain) || !IsValid(dst_domain) || !IsValid(in_rect)) { + return; + } + + float src_domain_width = src_domain.right - src_domain.left; + float src_domain_height = src_domain.bottom - src_domain.top; + float dst_domain_width = dst_domain.right - dst_domain.left; + float dst_domain_height = dst_domain.bottom - dst_domain.top; + + float width_ratio = dst_domain_width / src_domain_width; + float height_ratio = dst_domain_height / src_domain_height; + + out_rect->left = dst_domain.left + (width_ratio * in_rect.left); + out_rect->top = dst_domain.top + (height_ratio * in_rect.top); + out_rect->right = dst_domain.left + (width_ratio * in_rect.right); + out_rect->bottom = dst_domain.top + (height_ratio * in_rect.bottom); +} + } // namespace sdm |