f53fb5e2e0
This allows apps to catch out-of-memory errors when allocating textures. Textures can be pretty huge at times and so it's quite possible for an application to try and allocate more memory than is available. It's also very possible that the application can take some action in response to reduce memory pressure (such as freeing up texture caches perhaps) so we shouldn't just automatically abort like we do for trivial heap allocations. These public functions now take a CoglError argument so applications can catch out of memory errors: cogl_buffer_map cogl_buffer_map_range cogl_buffer_set_data cogl_framebuffer_read_pixels_into_bitmap cogl_pixel_buffer_new cogl_texture_new_from_data cogl_texture_new_from_bitmap Note: we've been quite conservative with how many apis we let throw OOM CoglErrors since we don't really want to put a burdon on developers to be checking for errors with every cogl api call. So long as there is some lower level api for apps to use that let them catch OOM errors for everything necessary that's enough and we don't have to make more convenient apis more awkward to use. The main focus is on bitmaps and texture allocations since they can be particularly large and prone to failing. A new cogl_attribute_buffer_new_with_size() function has been added in case developers need to catch OOM errors when allocating attribute buffers whereby they can first use _buffer_new_with_size() (which doesn't take a CoglError) followed by cogl_buffer_set_data() which will lazily allocate the buffer storage and report OOM errors. Reviewed-by: Neil Roberts <neil@linux.intel.com> (cherry picked from commit f7735e141ad537a253b02afa2a8238f96340b978) Note: since we can't break the API for Cogl 1.x then actually the main purpose of cherry picking this patch is to keep in-line with changes on the master branch so that we can easily cherry-pick patches. All the api changes relating stable apis released on the 1.12 branch have been reverted as part of cherry-picking this patch so this most just applies all the internal plumbing changes that enable us to correctly propagate OOM errors.
124 lines
4.2 KiB
C
124 lines
4.2 KiB
C
#include <cogl/cogl.h>
|
|
|
|
#include <string.h>
|
|
|
|
#include "test-utils.h"
|
|
|
|
static uint8_t
|
|
tex_data[2 * 2 * 4] =
|
|
{
|
|
0xff, 0x00, 0x00, 0xff, 0x00, 0xff, 0x00, 0xff,
|
|
0x00, 0x00, 0xff, 0xff, 0xff, 0x00, 0xff, 0xff
|
|
};
|
|
|
|
/* Vertex data for a quad with all of the texture coordinates set to
|
|
* the top left (red) pixel */
|
|
static CoglVertexP2T2
|
|
vertex_data[4] =
|
|
{
|
|
{ -1, -1, 0, 0 },
|
|
{ 1, -1, 0, 0 },
|
|
{ -1, 1, 0, 0 },
|
|
{ 1, 1, 0, 0 }
|
|
};
|
|
|
|
void
|
|
test_map_buffer_range (void)
|
|
{
|
|
CoglTexture2D *tex;
|
|
CoglPipeline *pipeline;
|
|
int fb_width, fb_height;
|
|
CoglAttributeBuffer *buffer;
|
|
CoglVertexP2T2 *data;
|
|
CoglAttribute *pos_attribute;
|
|
CoglAttribute *tex_coord_attribute;
|
|
|
|
tex = cogl_texture_2d_new_from_data (test_ctx,
|
|
2, 2, /* width/height */
|
|
COGL_PIXEL_FORMAT_RGBA_8888_PRE,
|
|
COGL_PIXEL_FORMAT_ANY,
|
|
2 * 4, /* rowstride */
|
|
tex_data,
|
|
NULL /* error */);
|
|
|
|
pipeline = cogl_pipeline_new (test_ctx);
|
|
|
|
cogl_pipeline_set_layer_texture (pipeline, 0, COGL_TEXTURE (tex));
|
|
cogl_pipeline_set_layer_filters (pipeline,
|
|
0, /* layer */
|
|
COGL_PIPELINE_FILTER_NEAREST,
|
|
COGL_PIPELINE_FILTER_NEAREST);
|
|
cogl_pipeline_set_layer_wrap_mode (pipeline,
|
|
0, /* layer */
|
|
COGL_PIPELINE_WRAP_MODE_CLAMP_TO_EDGE);
|
|
|
|
fb_width = cogl_framebuffer_get_width (test_fb);
|
|
fb_height = cogl_framebuffer_get_height (test_fb);
|
|
|
|
buffer = cogl_attribute_buffer_new (test_ctx,
|
|
sizeof (vertex_data),
|
|
vertex_data);
|
|
|
|
/* Replace the texture coordinates of the third vertex with the
|
|
* coordinates for a green texel */
|
|
data = cogl_buffer_map_range (COGL_BUFFER (buffer),
|
|
sizeof (vertex_data[0]) * 2,
|
|
sizeof (vertex_data[0]),
|
|
COGL_BUFFER_ACCESS_WRITE,
|
|
COGL_BUFFER_MAP_HINT_DISCARD_RANGE,
|
|
NULL); /* don't catch errors */
|
|
g_assert (data != NULL);
|
|
|
|
data->x = vertex_data[2].x;
|
|
data->y = vertex_data[2].y;
|
|
data->s = 1.0f;
|
|
data->t = 0.0f;
|
|
|
|
cogl_buffer_unmap (COGL_BUFFER (buffer));
|
|
|
|
pos_attribute =
|
|
cogl_attribute_new (buffer,
|
|
"cogl_position_in",
|
|
sizeof (vertex_data[0]),
|
|
offsetof (CoglVertexP2T2, x),
|
|
2, /* n_components */
|
|
COGL_ATTRIBUTE_TYPE_FLOAT);
|
|
tex_coord_attribute =
|
|
cogl_attribute_new (buffer,
|
|
"cogl_tex_coord_in",
|
|
sizeof (vertex_data[0]),
|
|
offsetof (CoglVertexP2T2, s),
|
|
2, /* n_components */
|
|
COGL_ATTRIBUTE_TYPE_FLOAT);
|
|
|
|
cogl_framebuffer_clear4f (test_fb,
|
|
COGL_BUFFER_BIT_COLOR,
|
|
0, 0, 0, 1);
|
|
|
|
cogl_framebuffer_vdraw_attributes (test_fb,
|
|
pipeline,
|
|
COGL_VERTICES_MODE_TRIANGLE_STRIP,
|
|
0, /* first_vertex */
|
|
4, /* n_vertices */
|
|
pos_attribute,
|
|
tex_coord_attribute,
|
|
NULL);
|
|
|
|
/* Top left pixel should be the one that is replaced to be green */
|
|
test_utils_check_pixel (test_fb, 1, 1, 0x00ff00ff);
|
|
/* The other three corners should be left as red */
|
|
test_utils_check_pixel (test_fb, fb_width - 2, 1, 0xff0000ff);
|
|
test_utils_check_pixel (test_fb, 1, fb_height - 2, 0xff0000ff);
|
|
test_utils_check_pixel (test_fb, fb_width - 2, fb_height - 2, 0xff0000ff);
|
|
|
|
cogl_object_unref (buffer);
|
|
cogl_object_unref (pos_attribute);
|
|
cogl_object_unref (tex_coord_attribute);
|
|
|
|
cogl_object_unref (pipeline);
|
|
cogl_object_unref (tex);
|
|
|
|
if (cogl_test_verbose ())
|
|
g_print ("OK\n");
|
|
}
|