<div dir="ltr"><div>Suggestions:<br></div><div>- PIPE_CAP_TILED_BASED_MSAA_OVERSAMPLING</div><div>- pipe_surface::tile_based_oversample_count</div><div><br></div><div>I'm assuming this feature isn't possible without tile-based rendering.<br></div><div><br></div><div>Marek<br></div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Nov 30, 2018 at 1:23 PM Kristian Høgsberg <<a href="mailto:hoegsberg@gmail.com" target="_blank">hoegsberg@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri, Nov 30, 2018 at 10:17 AM Marek Olšák <<a href="mailto:maraeo@gmail.com" target="_blank">maraeo@gmail.com</a>> wrote:<br>
><br>
> On Fri, Nov 30, 2018 at 1:13 PM Kristian Høgsberg <<a href="mailto:hoegsberg@gmail.com" target="_blank">hoegsberg@gmail.com</a>> wrote:<br>
>><br>
>> On Fri, Nov 16, 2018 at 7:48 PM Marek Olšák <<a href="mailto:maraeo@gmail.com" target="_blank">maraeo@gmail.com</a>> wrote:<br>
>> ><br>
>> > I think the name PIPE_CAP_MULTISAMPLED_RENDER_TO_TEXTURE is slightly misleading, because it doesn't imply anything about the OpenGL ES behavior, which is that a texture is multisampled in the cache, but single-sampled in memory. This should be mentioned somewhere.<br>
>><br>
>> Not sure exactly which change you're recommending, but in retrospect,<br>
>> I think it would be better to name the cap in term of how it changes<br>
>> the gallium API instead of the GLES extension it enables. How about<br>
>> PIPE_CAP_SURFACE_SAMPLE_COUNT, to indicate that it allows overriding<br>
>> sample counts in pipe_surface?<br>
><br>
><br>
> That's an interesting idea, but how does it convey that multisampled surfaces are never multisampled in memory?<br>
<br>
How are you going to convey all that in a cap enum name? In general,<br>
the cap names are concise hints and not super descriptive - you have<br>
to go read the documentation to understand the semantics.<br>
<br>
Do you have a specific name you'd like to see or can we go with<br>
PIPE_CAP_SURFACE_SAMPLE_COUNT?<br>
<br>
><br>
> Marek<br>
</blockquote></div>