<div dir="ltr"><div>BTW, is there any reason for get_sample_pixel_grid to be in pipe_context? It seems that pipe_screen is a better place, because the returned values are immutable. pipe_context also doesn't like functions that return something, because it limits multithreading.<br><br></div>Marek<br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 8, 2018 at 9:32 PM, Marek Olšák <span dir="ltr"><<a href="mailto:maraeo@gmail.com" target="_blank">maraeo@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div>The cso_context changes are unnecessary. It looks like that meta ops don't enable MSAA, so they are unaffected by sample locations, and thus no need to have any code for them in cso_context.<br><br></div>The names don't have to use the _state suffix, i.e. set_sample_locations.<br><br></div><div>I wouldn't add struct pipe_sample_locations. The whole array can be passed to set_sample_locations via a pointer, and NULL can mean disabled.<span class="m_4788207791457343349HOEnZb"><font color="#888888"><br></font></span></div><span class="m_4788207791457343349HOEnZb"><font color="#888888"><br></font></span></div><span class="m_4788207791457343349HOEnZb"><font color="#888888">Marek</font></span><br></div></blockquote></div><br></div></div>