[Mesa-dev] gallium: Reduce trace_dump_box_bytes size by box->x.

[Cherniak, Bruce]

On Jul 4, 2017, at 9:40 AM, Jose Fonseca <jfonseca at vmware.com<mailto:jfonseca at vmware.com>> wrote:

On 29/06/17 11:02, Michel Dänzer wrote:
On 29/06/17 01:54 AM, Cherniak, Bruce wrote:
On Jun 26, 2017, at 2:10 PM, Marek Olšák <maraeo at gmail.com<mailto:maraeo at gmail.com>> wrote:

In my opinion, dumping resources isn't very useful. I think it would
be better to remove that completely.

From Michel's response, sounds like dumping resources is useful, so...
FWIW, after my previous post it occurred to me that resource dumping
might not be necessary after all; dumping transfers might be sufficient
for a replayable trace.
Adding José, I hope he can clarify.

Right, to preserve ability to replay, it should suffice to serialize the write transfers.

Of course, this ignores the issue of persistent mappings.  To handle persistent mappings the trace driver would need to, on every draw, walk over every bound buffer and see if any was changed by keeping a shadow memory, or some other trick mentioned on https://github.com/apitrace/apitrace/issues/232

With regards to averting the crash that I experience in trace_dump_box_bytes when tracing certain applications, what is the best path forward?
I can keep my patch (or a patch that stubs this function altogether) handy for local application, but that doesn't help anyone else out.

Thanks,
Bruce

Jose

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170705/dfc62b85/attachment.html>