{"push_id":1018243205,"size":33,"distinct_size":33,"ref":"refs/heads/master","head":"1f3af32a3e76d0ab30e32f61dac2dae254c16047","before":"4dd5479121caff13483f1b80995773ece1ed289a","commits":[{"sha":"06bb83c84f65fdec10063e53e7e287de399f7390","author":{"email":"a847aabaac44184372f6b1cd1a59aa254daaeb0b@igalia.com","name":"Iago Toral Quiroga"},"message":"shaders: add test for recursive variable indexing into arrays\n\nv2:\n  - Move to tests/spec/glsl-1.30 (Timothy)\n  - Fix code style and rename some variables (Iago)\n\nReviewed-by: Timothy Arceri <timothy.arceri@collabora.com>","distinct":true,"url":"https://api.github.com/repos/jvesely/piglit/commits/06bb83c84f65fdec10063e53e7e287de399f7390"},{"sha":"3fd2e7c3168741816276ec04f4fe8e6c5191cd07","author":{"email":"5dfedca1889afcca58e9b95acd46f78f54f166ed@igalia.com","name":"Alejandro Piñeiro"},"message":"arb_internalformat_query: remove references to arb_internalformat_query2\n\narb_internalformat_query tests was taking into account query2, in\norder to avoid testing some pnames/internalformats valid on query2.\n\nBut taking into account how deeply arb_internalformat_query2 extended\nand changed the behaviour of GetInternalformativ, it is really more\nclear to isolate both.\n\nWith this commit arb_internalformat_query tests will test when only\narb_internalformat_query is present, and arb_internalformat_query2\nwill test when both are present (as arb_internalformat_query2 has\narb_internalformat_query as a requirement).\n\nThis patch makes this one obsolete:\nhttp://lists.freedesktop.org/archives/piglit/2015-October/017746.html\n\nAcked-by: Dave Airlie <airlied@redhat.com>","distinct":true,"url":"https://api.github.com/repos/jvesely/piglit/commits/3fd2e7c3168741816276ec04f4fe8e6c5191cd07"},{"sha":"1e1ba3ce9e9000cb73ff3ec7794ca63a2689464c","author":{"email":"5dfedca1889afcca58e9b95acd46f78f54f166ed@igalia.com","name":"Alejandro Piñeiro"},"message":"arb_internalformat_query2: query2 api-error check\n\nSimilar to the equivalent for arb_internalformat_query, but in\naddition to test when a invalid pname/target combination should return\nINVALID_ENUM, it also checks that a valid combination returns\nNO_ERROR, and testing both GetInternalformativ and\nGetInternalformati64v.\n\nThe rationale of this is that is really likely that the implementation\nof arb_internal_format_query2 would reuse a lot of bits of\narb_internal_format_query, so we want to be sure that a combination\nthat was invalid with arb_internal_format_query is not considered\ninvalid by arb_internal_format_query2.\n\nTested on NVIDIA GeForce GTX 950 - NVIDIA 352.55: pass\n\nAcked-by: Dave Airlie <airlied@redhat.com>","distinct":true,"url":"https://api.github.com/repos/jvesely/piglit/commits/1e1ba3ce9e9000cb73ff3ec7794ca63a2689464c"},{"sha":"ba05b3caa67481cfacccbe5a71b7b4fa9c20c2a0","author":{"email":"5dfedca1889afcca58e9b95acd46f78f54f166ed@igalia.com","name":"Alejandro Piñeiro"},"message":"arb_internalformat_query2: test for the most generic pname checks\n\nThe spec includes conditions for all the pnames. On most cases, it is\ngeneric like this:\n\n \"Possible values returned are <set>. If the resource is not\n  supported, or if the operation is not supported, NONE is\n  returned.\"\n\nSo this test checks that for those pnames:\n * If it is not supported (using INTERNALFORMAT_SUPPORTED), the\n   returned value is zero.\n * If it is supported, and there is available a list of possible\n   values, the returned value is among one of those values.\n\nTested on NVIDIA GeForce GTX 950 - NVIDIA 352.55: pass.\n\nv2: IMAGE_TEXEL_SIZE, IMAGE_COMPATIBILITY_CLASS, IMAGE_PIXEL_TYPE,\n    IMAGE_PIXEL_FORMAT tested on a specific test (Antía)\n\nAcked-by: Dave Airlie <airlied@redhat.com>","distinct":true,"url":"https://api.github.com/repos/jvesely/piglit/commits/ba05b3caa67481cfacccbe5a71b7b4fa9c20c2a0"},{"sha":"e8ab86534977812c93358202700bb165f92b33fe","author":{"email":"5dfedca1889afcca58e9b95acd46f78f54f166ed@igalia.com","name":"Alejandro Piñeiro"},"message":"arb_internalformat_query2: test for SAMPLES and NUM_SAMPLE_COUNTS pnames\n\nFor those pnames, it verify that the defined on the spec is fulfilled:\n  NUM_SAMPLE_COUNTS\n    <skip>\n    If <internalformat> is not color-renderable, depth-renderable, or\n    stencil-renderable (as defined in section 4.4.4), or if <target>\n    does not support multiple samples (ie other than\n    TEXTURE_2D_MULTISAMPLE, TEXTURE_2D_MULTISAMPLE_ARRAY, or\n    RENDERBUFFER), 0 is returned.\n\n  SAMPLES\n    <skip>\n    If <internalformat> is not color-renderable, depth-renderable, or\n    stencil-renderable (as defined in section 4.4.4), or if <target>\n    does not support multiple samples (ie other than\n    TEXTURE_2D_MULTISAMPLE, TEXTURE_2D_MULTISAMPLE_ARRAY, or\n    RENDERBUFFER), <params> is not modified.\n\nThis test is really similar to api-errors test at\narb_internalformat_query, with the difference that on that case an\nerror was defined, and here was \"softed\" to return some specific\nvalues. It also covers query1 overrun test.\n\nThose two tests fails on the following proprietary drivers:\n  * 4.5.13399 on AMD Radeon (TM) R9 380 Series\n  * 4.5.0 NVIDIA 352.55 on GeForce GTX 950/PCIe/SSE2\n\nFor NUM_SAMPLES_COUNTS, they report a value different of 0 (5 on the\nNVIDIA, 4 on ATI) for target/internalformat that should return 0\n(example: GL_TEXTURE_1D+GL_COMPRESSED_RGB).  It is worth to note that\nfor those cases, INTERNALFORMAT_SUPPORTED returns TRUE. It is not\nclear if they provide multi-sample capabilities for those targets.\n\nThat seems to suggest that the paragraph was C&P from query1, and it\nis being too restrictive, when the purpose of the query2 itself is \"a)\nprovide a mechanism for implementations to declare support *above* the\nminimum required by the specification\".\n\nFor the SAMPLES case, it changes the first element of <params>,\nfor cases where it should remain unmodified.\n\nAcked-by: Dave Airlie <airlied@redhat.com>","distinct":true,"url":"https://api.github.com/repos/jvesely/piglit/commits/e8ab86534977812c93358202700bb165f92b33fe"},{"sha":"517d44db644ea2033ba405deee0918b0ae8f3e8f","author":{"email":"5dfedca1889afcca58e9b95acd46f78f54f166ed@igalia.com","name":"Alejandro Piñeiro"},"message":"arb_internalformat_query2: test for INTERNALFORMAT_{X}_SIZE pnames\n\nFrom spec:\n\"  INTERNALFORMAT_RED_SIZE\n   INTERNALFORMAT_GREEN_SIZE\n   INTERNALFORMAT_BLUE_SIZE\n   INTERNALFORMAT_ALPHA_SIZE\n   INTERNALFORMAT_DEPTH_SIZE\n   INTERNALFORMAT_STENCIL_SIZE\n   INTERNALFORMAT_SHARED_SIZE\n\n   For textures this query will return the same information as\n   querying GetTexLevelParameter{if}v for TEXTURE_*_SIZE would return.\n   If the internal format is unsupported, or if a particular component\n   is not present in the format, 0 is written to <params>.\"\n\nSo this test calls GetInternalformat with INTERNALFORMAT_SUPPORTED:\n  * If it is false, it checks that the returned value is 0.\n  * If it is true, for texture targets, checks that the returned value\n    is the same that calling GetTexLevelParameter{if}v.\n\nIn order to call GetTexLevelParameter, it requires to create a texture\nfor the given internalformat/target combination. If that fails it is\nassumed that the current implementation doesn't support that\ncombination, and it is tested against 0 too. Note that this means\nthat the test would output some error messages even if the test\npasses.\n\nTested on NVIDIA GeForce GTX 950 - NVIDIA 352.55: fails on the next\ncases:\n\n  * Fails when trying to create a texture with some per-spec valid\n    internalformat, like 1D/3D textures with\n    GL_COMPRESSED_SIGNED_RED_RGTC1. Returns INVALID_ENUM, as if\n    TexCompressedImage1D were called.\n\n * GL_TEXTURE_BUFFER has a limited ranged of internalformat supported\n   (table 3.15 from spec 4.2). So for example, creating a texture with\n   target GL_TEXTURE_BUFFER and internalformat GL_DEPTH_STENCIL would\n   fail. On those cases the query should return 0, but it is not\n   always the case.\n\nAcked-by: Dave Airlie <airlied@redhat.com>","distinct":true,"url":"https://api.github.com/repos/jvesely/piglit/commits/517d44db644ea2033ba405deee0918b0ae8f3e8f"},{"sha":"116294f06a666f074e72442590a63ea250bff757","author":{"email":"5dfedca1889afcca58e9b95acd46f78f54f166ed@igalia.com","name":"Alejandro Piñeiro"},"message":"arb_internalformat_query2: test for INTERNALFORMAT_{X}_TYPE pnames\n\nFrom spec:\n\n\" INTERNALFORMAT_RED_TYPE\n  INTERNALFORMAT_GREEN_TYPE\n  INTERNALFORMAT_BLUE_TYPE\n  INTERNALFORMAT_ALPHA_TYPE\n  INTERNALFORMAT_DEPTH_TYPE\n  INTERNALFORMAT_STENCIL_TYPE\n\n  For uncompressed internal formats, queries for these values return\n  the data type used to store the component.  For compressed internal\n  formats the types returned specify how components are interpreted\n  after decompression.  For textures this query returns the same\n  information as querying GetTexLevelParameter{if}v for TEXTURE_*TYPE\n  would return.  Possible values return include, NONE,\n  SIGNED_NORMALIZED, UNSIGNED_NORMALIZED, FLOAT, INT, UNSIGNED_INT,\n  representing missing, signed normalized fixed point, unsigned\n  normalized fixed point, floating-point, signed unnormalized integer\n  and unsigned unnormalized integer components. NONE is returned for\n  all component types if the format is unsupported.\"\n\nSo this test calls GetInternalformat with INTERNALFORMAT_SUPPORTED:\n  * If it is false, checks that the returned value is zero\n  * If it is true, checks that all returned values are from that\n    specific set of values\n  * If it is true, checks that all the values are the same that\n    GetTexLevelParameter\n\nTested on NVIDIA GeForce GTX 950 - NVIDIA 352.55: fails for the\nsame reasons that for INTERNALFORMAT_{X}_SIZE tests.\n\nAcked-by: Dave Airlie <airlied@redhat.com>","distinct":true,"url":"https://api.github.com/repos/jvesely/piglit/commits/116294f06a666f074e72442590a63ea250bff757"},{"sha":"4cbf29e75349949ea86a7870a5169afdb5a88e5a","author":{"email":"5dfedca1889afcca58e9b95acd46f78f54f166ed@igalia.com","name":"Alejandro Piñeiro"},"message":"arb_internalformat_query2: IMAGE_FORMAT_COMPATIBILITY_TYPE test\n\nFrom spec:\n   \"IMAGE_FORMAT_COMPATIBILITY_TYPE: The matching criteria use for the\n    resource when used as an image textures is returned in\n    <params>. This is equivalent to calling GetTexParameter with\n    <value> set to IMAGE_FORMAT_COMPATIBILITY_TYPE. Possible values\n    are IMAGE_FORMAT_COMPATIBILITY_BY_SIZE or\n    IMAGE_FORMAT_COMPATIBILITY_BY_CLASS.  If the resource is not\n    supported for image textures, or if image textures are not\n    supported, NONE is returned.\"\n\nTested on NVIDIA GeForce GTX 950 - NVIDIA 352.55: fails for the same\nreasons it fails for INTERNALFORMAT_{X}_SIZE and INTERNALFORMAT_{X}_TYPE.\n\nAcked-by: Dave Airlie <airlied@redhat.com>","distinct":true,"url":"https://api.github.com/repos/jvesely/piglit/commits/4cbf29e75349949ea86a7870a5169afdb5a88e5a"},{"sha":"98906d0acbcff43fcfa93041bba423feeae9e00a","author":{"email":"5dfedca1889afcca58e9b95acd46f78f54f166ed@igalia.com","name":"Alejandro Piñeiro"},"message":"arb_internalformat_query2: max dimensions related checks\n\nAdd a check for the following pnames (a subtest for each one)\n\n* MAX_WIDTH. From spec:\n   \"The maximum supported width for the resource is returned in\n    <params>. For resources with only one-dimension, this one\n    dimension is considered the width. If the resource is unsupported,\n    zero is returned.\"\n\n    It is only tested that returns zero if not supported. It makes\n    sense to include it on this test because it is related to the\n    other max dimension pnames.\n\n* MAX_HEIGHT. From spec:\n   \"The maximum supported height for the resource is returned in\n    <params>. For resources with two or more dimensions, the second\n    dimension is considered the height. If the resource does not have\n    at least two dimensions, or if the resource is unsupported, zero\n    is returned.\"\n\n  So in addition to the usual zero-test if not supported, it is tested\n  that if the target has less that two dimensions, it returns zero\n  too, even if supported.\n\n* MAX_DEPTH: From spec:\n   \"The maximum supported depth for the resource is returned in\n    <params>. For resources with three or more dimensions, the third\n    dimension is considered the depth. If the resource does not have\n    at least three dimensions, or if the resource is unsupported, zero\n    is returned.\n\n  So in addition to the usual zero-test if not supported, it is tested\n  that if the target has less that three dimensions, it returns zero\n  too, even if supported.\n\n* MAX_LAYERS: From spec:\n   \"The maximum supported number of layers for the resource is\n    returned in <params>. For 1D array targets, the value returned is\n    the same as the MAX_HEIGHT. For 2D and cube array targets, the\n    value returned is the same as the MAX_DEPTH. If the resource does\n    not support layers, or if the resource is unsupported, zero is\n    returned.\"\n\n  In addition to the usual zero-test if not supported, it is tested\n  that the value returned is the same that the one returned by\n  MAX_HEIGHT or MAX_DEPTH for the array texture targets.\n\nTested on NVIDIA GeForce GTX 950 - NVIDIA 352.55: passes.\n\nTested on 4.5.13399 on AMD Radeon (TM) R9 380 Series: the tests\ndoesn't pass for this ATI proprietary drivers for any of the\nsubtests. It returns a non-zero value for unsupported combinations.\n\nAcked-by: Dave Airlie <airlied@redhat.com>","distinct":true,"url":"https://api.github.com/repos/jvesely/piglit/commits/98906d0acbcff43fcfa93041bba423feeae9e00a"},{"sha":"e7d81a9389be904ac149a4b202c171fd59683a3b","author":{"email":"5dfedca1889afcca58e9b95acd46f78f54f166ed@igalia.com","name":"Alejandro Piñeiro"},"message":"arb_internalformat_query2: check returned max-dimensions values against GetInteger\n\nIn addition to the previous checks, this checks that the returned\nvalues when supported are the same that the ones you receive calling\nGetIntegerv with equivalent pnames like GL_MAX_TEXTURE_SIZE,\nGL_MAX_3D_TEXTURE_SIZE, etc.\n\nAll those are internal format-independent, meanwhile GetInternalformat\nallows to specify the internal format. So in theory there is the\npossibility of being different for some internal format. But in\npractice, this is not happening on any driver at this moment. Query2\nspec mentions this case:\n\n    \"7) There some <pnames> which it makes no sense to be qualified by\n     a per-format/target scope, how should we handle them?\n     e.g. MAX_WIDTH and MAX_HEIGHT might be the same for all formats.\n     e.g. properties like AUTO_GENERATE_MIPMAP and\n     MANUAL_GENERATE_MIPMAP might depend only on the GL version.\n\n     <skip>\n\n     A) Just use this entry point as is, if there are no per-format or\n     target differences, it is perfectly acceptable to have the\n     implementation return the same information for all valid\n     parameters. This does allow implementations to report caveats\n     that may exist for some formats but not others, even though all\n     formats/targets may be supported.\"\n\nSo at this point, taking into account the current implementation, it\nmakes sense to check against those values. Probably in the future this\ncheck needs to be removed.\n\nTested on NVIDIA GeForce GTX 950 - NVIDIA 352.55: pass\n\nTested on 4.5.13399 on AMD Radeon (TM) R9 380 Series: fails:\n  * For MAX_WIDTH:\n    * For 3d textures is returning MAX_TEXTURE_SIZE\n    * For texture buffers is returning crap. Different values for\n      each internalformat, including negative values.\n  * For MAX_HEIGHT:\n    * Ditto for 3d textures.\n    * For RENDERBUFFER and TEXTURE_2D_MULTISAMPLE_ARRAY is returning\n      0 even for supported values.\n\nAnyway, take into account that even the basic dimension test (0 for\nunsupported) was failing on ATI proprietary drivers.\n\nAcked-by: Dave Airlie <airlied@redhat.com>","distinct":true,"url":"https://api.github.com/repos/jvesely/piglit/commits/e7d81a9389be904ac149a4b202c171fd59683a3b"},{"sha":"8de0411a9f3d963ca83ff8ed1ae195803e7368d5","author":{"email":"5dfedca1889afcca58e9b95acd46f78f54f166ed@igalia.com","name":"Alejandro Piñeiro"},"message":"arb_internalformat_query2: add MAX_COMBINED_DIMENSIONS subtest to max-dimensions\n\nFrom spec:\n\n\"MAX_COMBINED_DIMENSIONS: The maximum combined dimensions for the\n  resource is returned in <params>. The combined dimensions is the\n  product of the individual dimensions of the resource. For\n  multisampled surfaces the number of samples is considered an\n  additional dimension. Note that the value returned can be >= 2^32\n  and should be queried with the 64-bit query.\n  <skip>\n  If the resource is unsupported, zero is returned.\"\n\nIn this case the returning value is a combination of the values of\nMAX_WIDTH, MAX_HEIGHT, MAX_DEPTH, SAMPLES and the number of faces, so\nthe test can check that the returned value is correct in any case.\n\nNote that there are cases that the combined value is greater that\n2^32, but the test is also testing the 32-bit query. The spec doesn't\nspecify what the query should return in that case, so it is assumed\nthat any value would be correct.\n\nTested on NVIDIA GeForce GTX 950 - NVIDIA 352.55: fails for cube\nmap related targets. From spec:\n\n\"For cube map targets this is the maximum combined width, height and\n faces\"\n\nBut NVIDIA propietary drivers is just combining width and height.\n\nAcked-by: Dave Airlie <airlied@redhat.com>","distinct":true,"url":"https://api.github.com/repos/jvesely/piglit/commits/8de0411a9f3d963ca83ff8ed1ae195803e7368d5"},{"sha":"fd5b970f6a2d4449781b8d3f97eb47f4251f6063","author":{"email":"cb7235b689ed9503d8ca40656e7a04b2a0428d43@igalia.com","name":"Antia Puentes"},"message":"arb_internalformat_query2: Check if the resource is supported in max-dimensions\n\nThe ARB_internalformat_query2 specification says about\nMAX_WIDTH, MAX_HEIGHT, MAX_DEPTH and MAX_COMBINED_DIMENSIONS <pnames>:\n\n   \"If the resource is unsupported, zero is returned.\"\n\nFrom the same specification:\n   \"In the following descriptions, the term /resource/ is used\n    to generically refer to an object of the appropriate type that has\n    been created with <internalformat> and <target>.\"\n\nWe check if the /resource/ is supported by trying to create the object,\nif an error is raised, we consider the /resource/ as \"unsupported\".\nBefore, we were only checking if the <internalformat> was supported\nindependently of the <target>.\n\nTested on NVIDIA GeForce GTX 950 - NVIDIA 352.55: after this commit,\nall the subtests on this test fails.\n\nNow that a stricter check for unsupported resources are in place, the\ntest fails on NVIDIA because for some unsupported resources it returns\na value different to zero.\n\nAcked-by: Dave Airlie <airlied@redhat.com>","distinct":true,"url":"https://api.github.com/repos/jvesely/piglit/commits/fd5b970f6a2d4449781b8d3f97eb47f4251f6063"},{"sha":"bc95a98cab878b30f868d8ca0b9f704d78b08a29","author":{"email":"5dfedca1889afcca58e9b95acd46f78f54f166ed@igalia.com","name":"Alejandro Piñeiro"},"message":"arb_internalformat_query2: add a generic check for TEXTURE_IMAGE_FORMAT\n\nFrom spec:\n\n\"Possible values include any value that is legal to pass for the\n <format> parameter to the Tex*Image*D commands, or NONE if the\n resource is not supported for this operation.\"\n\nTested on NVIDIA GeForce GTX 950 - NVIDIA 352.55: fails:\nIn some cases is returns GL_R11F_G11F_B10F or GL_RGB9_ES5, that are\ninternalformats, not valid formats.\n\nTested on AMD Radeon (TM) R9 380 Series: fails:\nIt returns values different to NONE on cases where the\ntarget/internalformat are not supported (via\nINTERNALFORMAT_SUPPORTED).\n\nv2: rebased after changes in other commits\n\nAcked-by: Dave Airlie <airlied@redhat.com>","distinct":true,"url":"https://api.github.com/repos/jvesely/piglit/commits/bc95a98cab878b30f868d8ca0b9f704d78b08a29"},{"sha":"3ca0b8bf3b9d4c0ff9666db71d0c9f17eca6cdea","author":{"email":"5dfedca1889afcca58e9b95acd46f78f54f166ed@igalia.com","name":"Alejandro Piñeiro"},"message":"arb_internalformat_query2: add a generic check for TEXTURE_IMAGE_TYPE\n\n \"TEXTURE_IMAGE_TYPE:\n  <skip>\n  Possible values include any value that is legal to pass for the\n  <type> parameter to the Tex*Image*D commands, or NONE if the\n  resource is not supported for this operation.\"\n\nTested on NVIDIA GeForce GTX 950 - NVIDIA 352.55: pass.\n\nAcked-by: Dave Airlie <airlied@redhat.com>","distinct":true,"url":"https://api.github.com/repos/jvesely/piglit/commits/3ca0b8bf3b9d4c0ff9666db71d0c9f17eca6cdea"},{"sha":"360d5c1628635919b93f07345731a604c700d0de","author":{"email":"5dfedca1889afcca58e9b95acd46f78f54f166ed@igalia.com","name":"Alejandro Piñeiro"},"message":"arb_internalformat_query2: add generic check for GET_TEXTURE_IMAGE_FORMAT\n\nFrom query2 spec:\n\"GET_TEXTURE_IMAGE_FORMAT:\n <skip>\n Possible values include any value that is legal to pass for the\n <format> parameter to GetTexImage, or NONE if the resource does not\n support this operation, or if GetTexImage is not supported.\"\n\nThe possible list of values is the same that testing\nTEXTURE_IMAGE_FORMAT, but depending on the gl version,\nGL_STENCIL_INDEX is allowed too. This commits adds another list of\npossible values. We could have added a clone method that added extra\nvalues, but for now that would be an overkill. We could add more if we\nfind more similar cases.\n\nTested on NVIDIA GeForce GTX 950 - NVIDIA 352.55: fails.\n\nAs with TEXTURE_IMAGE_FORMAT in some cases returns GL_R11F_G11F_B10F\nor GL_RGB9_E5, that are internalformats, not formats.\n\nAcked-by: Dave Airlie <airlied@redhat.com>","distinct":true,"url":"https://api.github.com/repos/jvesely/piglit/commits/360d5c1628635919b93f07345731a604c700d0de"},{"sha":"2ca19cab1ae5dd56c258738d066e4d10dc204978","author":{"email":"5dfedca1889afcca58e9b95acd46f78f54f166ed@igalia.com","name":"Alejandro Piñeiro"},"message":"arb_internalformat_query2: add generic check for GET_TEXTURE_IMAGE_TYPE\n\nFrom spec:\n \"GET_TEXTURE_IMAGE_TYPE:\n  <skip>\n\n  Possible values include any value that is legal to pass for the\n  <type> parameter to GetTexImage, or NONE if the resource does not\n  support this operation, or if GetTexImage is not supported.\"\n\nTested on NVIDIA GeForce GTX 950 - NVIDIA 352.55: pass.\n\nAcked-by: Dave Airlie <airlied@redhat.com>","distinct":true,"url":"https://api.github.com/repos/jvesely/piglit/commits/2ca19cab1ae5dd56c258738d066e4d10dc204978"},{"sha":"3bf46c1decf175a3d6a582065283bc6968d3cc53","author":{"email":"5dfedca1889afcca58e9b95acd46f78f54f166ed@igalia.com","name":"Alejandro Piñeiro"},"message":"arb_internalformat_query2: test for COLOR_ENCODING\n\nFrom spec:\n \"COLOR_ENCODING:\n  <skip>\n  Possible values for color buffers are LINEAR or SRGB, for linear or\n  sRGB-encoded color components, respectively. For non-color formats\n  (such as depth or stencil), or for unsupported resources, the value\n  NONE is returned.\"\n\nTested on NVIDIA GeForce GTX 950 - NVIDIA 352.55: pass.\n\nAcked-by: Dave Airlie <airlied@redhat.com>","distinct":true,"url":"https://api.github.com/repos/jvesely/piglit/commits/3bf46c1decf175a3d6a582065283bc6968d3cc53"},{"sha":"4c00ec68dab0662f02ea90254e13d9f78f846d7b","author":{"email":"5dfedca1889afcca58e9b95acd46f78f54f166ed@igalia.com","name":"Alejandro Piñeiro"},"message":"arb_internalformat_query2: test for TEXTURE_COMPRESSED_BLOCK_<X>\n\nThis test add a check for the following pnames:\n  * TEXTURE_COMPRESSED_BLOCK_WIDTH\n  * TEXTURE_COMPRESSED_BLOCK_HEIGHT\n  * TEXTURE_COMPRESSED_BLOCK_SIZE\n\nOn all those three, query2 spec says the following:\n\"If the internal format is not compressed, or the resource is not\n supported, 0 is returned.\"\n\nWe could have classified the existing internalformats on\ncompressed/non-compressed (similar to color-format/non-color-format\nfor COLOR_ENCONDING), but that seems pointless taking into account\nthat we already have TEXTURE_COMPRESSED to query if a internalformat\nis compressed.\n\nSo this test queries TEXTURE_COMPRESSED and INTERNALFORMAT_SUPPORTED,\nand if any of them is false, checks that the returned value for any of\nthose three pnames is zero.\n\nTested on NVIDIA GeForce GTX 950 - NVIDIA 352.55: pass.\n\nAcked-by: Dave Airlie <airlied@redhat.com>","distinct":true,"url":"https://api.github.com/repos/jvesely/piglit/commits/4c00ec68dab0662f02ea90254e13d9f78f846d7b"},{"sha":"da71ea161a5652e619db43a0db35ce9b2c17d1ed","author":{"email":"5dfedca1889afcca58e9b95acd46f78f54f166ed@igalia.com","name":"Alejandro Piñeiro"},"message":"arb_internalformat_query2: port of query1 minmax test to query2\n\nIt is basically the same, but testing with both GetInternalformativ\nand GetInternalformati64v, using the test_data struct defined at\ncommon.h\n\nTested on NVIDIA GeForce GTX 950 - NVIDIA 352.55: pass.\n\nAcked-by: Dave Airlie <airlied@redhat.com>","distinct":true,"url":"https://api.github.com/repos/jvesely/piglit/commits/da71ea161a5652e619db43a0db35ce9b2c17d1ed"},{"sha":"2936f14eeb27a9d739b5c152845c292bada12525","author":{"email":"cb7235b689ed9503d8ca40656e7a04b2a0428d43@igalia.com","name":"Antia Puentes"},"message":"arb_internalformat_query2: Added tests for image textures\n\nChecks that:\n  * TEXEL_SIZE\n  * IMAGE_COMPATIBILITY_CLASS\n  * IMAGE_PIXEL_FORMAT\n  * IMAGE_PIXEL_TYPE\nreturn the values defined for them in Table 3.22 of the OpenGL 4.2\nspecification depending on the \"Image Format\" passed.\n\nFor unsupported resources, this test checks that the returned value\nis equal to the 'unsupported' response defined by the extension\nspecification.\n\nv2: check for unsupported values, so testing this pnames could be\n    removed from generic-pnames\n\nTested on NVIDIA GeForce GTX 950 - NVIDIA 352.55: fails:\n\n* IMAGE_TEXEL_SIZE\n  From spec:\n      The size of a texel when the resource when used as an image\n      texture is returned in <params>.  This is the value from the\n      /Size/ column in Table 3.22. If the resource is not supported\n      for image textures, or if image textures are not supported, zero\n      is returned.\n\n  NVIDIA proprietary drivers returns 1, 2 or 4, a values not present\n  on Table 3.22.\n\n* IMAGE_PIXEL_TYPE:\n  From spec:\n      The pixel type of the resource when used as an image texture is\n      returned in <params>.  This is the value from the /Pixel type/\n      column in Table 3.22. If the resource is not supported for image\n      textures, or if image textures are not supported, NONE is\n      returned.\n\n  NVIDIA proprietary drivers returns in some cases\n  GL_UNSIGNED_INT_8_8_8_8_REV, that although defined on core spec 4.2,\n  it is not included on Table 3.22\n\n* IMAGE_PIXEL_FORMAT:\n  From spec:\n      The pixel format of the resource when used as an image texture\n      is returned in <params>.  This is the value from the /Pixel\n      format/ column in Table 3.22. If the resource is not supported\n      for image textures, or if image textures are not supported, NONE\n      is returned.\n\n  NVIDIA proprietary drivers returns in some cases GL_R11F_G11F_B10F,\n  that although defined on core spec 4.2, it is not included on Table\n  3.22\n\nAcked-by: Dave Airlie <airlied@redhat.com>","distinct":true,"url":"https://api.github.com/repos/jvesely/piglit/commits/2936f14eeb27a9d739b5c152845c292bada12525"}]}