[Mesa-dev] [Bug 84566] Unify the format conversion code

2015-01-21 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 Jason Ekstrand changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-11-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #55 from Jason Ekstrand --- (In reply to Iago Toral from comment #54) > Hi Jason, > > I think we are close to completing this, so here is a summary of the current > state and how we intend to submit the patches for review, since we a

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-11-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #54 from Iago Toral --- Hi Jason, I think we are close to completing this, so here is a summary of the current state and how we intend to submit the patches for review, since we assume that you will be reviewing most of these: (In r

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-11-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #53 from Jason Ekstrand --- (In reply to Samuel Iglesias from comment #52) > (In reply to Jason Ekstrand from comment #51) > > (In reply to Samuel Iglesias from comment #48) > > > (In reply to Jason Ekstrand from comment #34) > > > >

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-11-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #52 from Samuel Iglesias --- (In reply to Jason Ekstrand from comment #51) > (In reply to Samuel Iglesias from comment #48) > > (In reply to Jason Ekstrand from comment #34) > > > (In reply to Samuel Iglesias from comment #33) > > > >

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-11-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #51 from Jason Ekstrand --- (In reply to Samuel Iglesias from comment #48) > (In reply to Jason Ekstrand from comment #34) > > (In reply to Samuel Iglesias from comment #33) > > > Jason, I would like to know your opinion about the int

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-11-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #50 from Jason Ekstrand --- (In reply to Iago Toral from comment #49) > Jason you had this commit in your original branch: "MAYBEREVERT: Fill X > components with 1" > > That basically makes packing to padded formats (like RGBX) set X

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-11-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #49 from Iago Toral --- Jason you had this commit in your original branch: "MAYBEREVERT: Fill X components with 1" That basically makes packing to padded formats (like RGBX) set X=1. In the commit message you mention that you are not

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-11-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #48 from Samuel Iglesias --- (In reply to Jason Ekstrand from comment #34) > (In reply to Samuel Iglesias from comment #33) > > Jason, I would like to know your opinion about the integer RGBA clamping > > done in pack.c (_mesa_pack_rg

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #47 from Iago Toral --- (In reply to Jason Ekstrand from comment #46) > (In reply to Iago Toral from comment #43) > > Jason, we are running into some issues when attempting to use > > _mesa_format_convert for glReadPixels and glGetTex

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-29 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #46 from Jason Ekstrand --- (In reply to Iago Toral from comment #43) > Jason, we are running into some issues when attempting to use > _mesa_format_convert for glReadPixels and glGetTexImage. > > Generally, one thing that is differe

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-29 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #45 from Jason Ekstrand --- (In reply to Iago Toral from comment #44) > (In reply to Iago Toral from comment #43) > (...) > > 3) Luminance formats have special requirements. A conversion to Luminance > > from RGBA requires to do L=R+G

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-28 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #44 from Iago Toral --- (In reply to Iago Toral from comment #43) (...) > 3) Luminance formats have special requirements. A conversion to Luminance > from RGBA requires to do L=R+G+B for example. This is something that > _mesa_format_

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-28 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #43 from Iago Toral --- Jason, we are running into some issues when attempting to use _mesa_format_convert for glReadPixels and glGetTexImage. Generally, one thing that is different in this case is that the current implementation nev

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #23 from Iago Toral --- (In reply to Jason Ekstrand from comment #21) > (In reply to Iago Toral from comment #19) > > Jason, for conversions where we cannot use a fast path in the master > > converter (that is, where we need to unpack

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #42 from Jason Ekstrand --- (In reply to Iago Toral from comment #40) > Jason, I think we may want to be a bit more conservative with the > pack/unpack fast paths when we consider the internal base format. > > Right now, if the src o

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #41 from Jason Ekstrand --- (In reply to Iago Toral from comment #39) > From your explanations I think this is what we should do too. For example, > when the source type is RGBA ubyte, we will do something like this: > > if (src_arra

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #40 from Iago Toral --- Jason, I think we may want to be a bit more conservative with the pack/unpack fast paths when we consider the internal base format. Right now, if the src or dst formats are RGBA we will try the pack/unpack fas

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #39 from Iago Toral --- (In reply to Jason Ekstrand from comment #37) > (In reply to Iago Toral from comment #35) > > Jason, you wrote some fast paths in _mesa_format_convert for the cases where > > we can directly pack or unpack to t

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #38 from Jason Ekstrand --- (In reply to Samuel Iglesias from comment #36) > (In reply to Jason Ekstrand from comment #32) > > (In reply to Iago Toral from comment #31) > > > Jason, there is an had-hoc path to deal with MESA_FORMAT_YC

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #37 from Jason Ekstrand --- (In reply to Iago Toral from comment #35) > Jason, you wrote some fast paths in _mesa_format_convert for the cases where > we can directly pack or unpack to the destination format. In these paths, if > we h

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #36 from Samuel Iglesias --- (In reply to Jason Ekstrand from comment #32) > (In reply to Iago Toral from comment #31) > > Jason, there is an had-hoc path to deal with MESA_FORMAT_YCBCR* in > > texstore.c. Seems that this path only al

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #35 from Iago Toral --- Jason, you wrote some fast paths in _mesa_format_convert for the cases where we can directly pack or unpack to the destination format. In these paths, if we have to pack from or unpack to RGBA UINT, you hav

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-21 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #34 from Jason Ekstrand --- (In reply to Samuel Iglesias from comment #33) > Jason, I would like to know your opinion about the integer RGBA clamping > done in pack.c (_mesa_pack_rgba_span_from_ints()). > > glReadPixels() and glGetTe

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #33 from Samuel Iglesias --- Jason, I would like to know your opinion about the integer RGBA clamping done in pack.c (_mesa_pack_rgba_span_from_ints()). glReadPixels() and glGetTexImage() specs said (for example, in OpenGL 3.3. core

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #32 from Jason Ekstrand --- (In reply to Iago Toral from comment #31) > Jason, there is an had-hoc path to deal with MESA_FORMAT_YCBCR* in > texstore.c. Seems that this path only allows conversion between these > formats, which is mos

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #31 from Iago Toral --- Jason, there is an had-hoc path to deal with MESA_FORMAT_YCBCR* in texstore.c. Seems that this path only allows conversion between these formats, which is mostly a memcpy with some byte swapping when necessary.

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-17 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #30 from Jason Ekstrand --- (In reply to Iago Toral from comment #29) > (In reply to Jason Ekstrand from comment #26) > > (In reply to Iago Toral from comment #24) > > > Jason, what do you think we should do for GL_COLOR_INDEX + GL_BI

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #29 from Iago Toral --- (In reply to Jason Ekstrand from comment #26) > (In reply to Iago Toral from comment #24) > > Jason, what do you think we should do for GL_COLOR_INDEX + GL_BITMAP? > > > > Currently this is handled as a specia

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #28 from Iago Toral --- (In reply to Kenneth Graunke from comment #25) > I thought we killed color indexing with fire a while back. Did we miss some? I see multiple references to GL_COLOR_INDEX in src/mesa/main and at least these pi

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #27 from Jason Ekstrand --- (In reply to Iago Toral from comment #23) > (In reply to Jason Ekstrand from comment #21) > > (In reply to Iago Toral from comment #19) > > > Jason, for conversions where we cannot use a fast path in the ma

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #26 from Jason Ekstrand --- (In reply to Iago Toral from comment #24) > Jason, what do you think we should do for GL_COLOR_INDEX + GL_BITMAP? > > Currently this is handled as a special case in the texstore.c ubyte path > (store_ubyte

Re: [Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-16 Thread Roland Scheidegger
On 10/16/2014 08:33 AM, bugzilla-dae...@freedesktop.org wrote: *Comment # 25 I thought we killed color indexing with fire a while back. Did we miss some? You can't kill glBitmap (in a non-core context, anyway). Strictly speaking these aren't color indexed textures, but still hit all the same

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #25 from Kenneth Graunke --- I thought we killed color indexing with fire a while back. Did we miss some? -- You are receiving this mail because: You are the QA Contact for the bug. ___ m

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #24 from Iago Toral --- Jason, what do you think we should do for GL_COLOR_INDEX + GL_BITMAP? Currently this is handled as a special case in the texstore.c ubyte path (store_ubyte_texture), specifically, the special case handles extr

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #22 from Iago Toral --- (In reply to Jason Ekstrand from comment #20) > (In reply to Iago Toral from comment #18) > > (In reply to Jason Ekstrand from comment #17) > > > (In reply to Iago Toral from comment #16) > > > > We also need n

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #21 from Jason Ekstrand --- (In reply to Iago Toral from comment #19) > Jason, for conversions where we cannot use a fast path in the master > converter (that is, where we need to unpack the src to rgba and then pack > from rgba to ds

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #20 from Jason Ekstrand --- (In reply to Iago Toral from comment #18) > (In reply to Jason Ekstrand from comment #17) > > (In reply to Iago Toral from comment #16) > > > We also need new mesa_format enums and pack/unpack functions for

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #19 from Iago Toral --- Jason, for conversions where we cannot use a fast path in the master converter (that is, where we need to unpack the src to rgba and then pack from rgba to dst) you differentiate between 3 paths: uint (integer)

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-14 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #18 from Iago Toral --- (In reply to Jason Ekstrand from comment #17) > (In reply to Iago Toral from comment #16) > > We also need new mesa_format enums and pack/unpack functions for byte > > swapped variants of non-array types. > >

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-14 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #17 from Jason Ekstrand --- (In reply to Iago Toral from comment #16) > We also need new mesa_format enums and pack/unpack functions for byte > swapped variants of non-array types. Why? Is this for the byte_swapped flag on GL upload

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-14 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #16 from Iago Toral --- We also need new mesa_format enums and pack/unpack functions for byte swapped variants of non-array types. -- You are receiving this mail because: You are the QA Contact for the bug. _

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #15 from Jason Ekstrand --- (In reply to Iago Toral from comment #14) > (...) > > > I don't know yet. For that I would have enable the master convertion > > > function for all code paths, then run all the piglit tests and then check >

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #14 from Iago Toral --- (In reply to Jason Ekstrand from comment #13) > (In reply to Iago Toral from comment #12) > > (In reply to Jason Ekstrand from comment #11) > > > (In reply to Iago Toral from comment #10) > > > > (In reply to I

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #13 from Jason Ekstrand --- (In reply to Iago Toral from comment #12) > (In reply to Jason Ekstrand from comment #11) > > (In reply to Iago Toral from comment #10) > > > (In reply to Iago Toral from comment #9) > > > > Jason, piglit t

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #12 from Iago Toral --- (In reply to Jason Ekstrand from comment #11) > (In reply to Iago Toral from comment #10) > > (In reply to Iago Toral from comment #9) > > > Jason, piglit tests hit cases where they attempt to convert GL format

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #11 from Jason Ekstrand --- (In reply to Iago Toral from comment #10) > (In reply to Iago Toral from comment #9) > > Jason, piglit tests hit cases where they attempt to convert GL format and > > data type combinations that do not matc

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #10 from Iago Toral --- (In reply to Iago Toral from comment #9) > Jason, piglit tests hit cases where they attempt to convert GL format and > data type combinations that do not match any of the existing mesa formats. > > For example

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #9 from Iago Toral --- Jason, piglit tests hit cases where they attempt to convert GL format and data type combinations that do not match any of the existing mesa formats. For example GL_RGB + GL_UNSIGNED_BYTE_2_3_3_REV (BBGG GRRR).

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 Eduardo Lima Mitev changed: What|Removed |Added CC||el...@igalia.com --- Comment #8 fro

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #7 from Iago Toral --- (In reply to Jason Ekstrand from comment #6) > (In reply to Iago Toral from comment #5) > > Jason, in your initial implementation of the master function you have this > > assert: > > > > if (src_array_format.as

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #6 from Jason Ekstrand --- (In reply to Iago Toral from comment #5) > Jason, in your initial implementation of the master function you have this > assert: > > if (src_array_format.as_uint && dst_array_format.as_uint) { >assert(sr

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #5 from Iago Toral --- Jason, in your initial implementation of the master function you have this assert: if (src_array_format.as_uint && dst_array_format.as_uint) { assert(src_array_format.normalized == dst_array_format.normalize

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #4 from Jason Ekstrand --- (In reply to Iago Toral from comment #3) > (...) > As far as I can see, functions in textstore do not receive a mesa_format > for their src argument, instead they receive the GL format and data type, so > i

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #3 from Iago Toral --- (In reply to Jason Ekstrand from comment #0) (...) > 5) Make a masater conversion function that takes a void*, format, width, > height, and stride for both source and destination and just does the > conversion.

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 Matt Turner changed: What|Removed |Added QA Contact||mesa-dev@lists.freedesktop.

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 Iago Toral changed: What|Removed |Added CC||sigles...@igalia.com Assignee|mes

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #2 from Iago Toral --- Hi Jason: Samuel and I will continue your work. We will ask here if we need input from your side. -- You are receiving this mail because: You are the assignee for the bug. _

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-01 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 Jason Ekstrand changed: What|Removed |Added CC||ja...@jlekstrand.net -- You are receiv

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-10-01 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #1 from Jason Ekstrand --- I've already started a lot of this work. I've got an implementation of an array format enum and master conversion function in this branch: http://cgit.freedesktop.org/~jekstrand/mesa/log/?h=wip/format-conv