Hi Nicolai,
2016-12-01 13:00 GMT+01:00 Nicolai Hähnle <nhaeh...@gmail.com>: > Congratulations on a huge amount of work! Obviously I can't say much about > the driver itself. Some things that I noticed for the renderonly library. > > On 30.11.2016 14:44, Christian Gmeiner wrote: >> >> This a very lightweight library to add basic support for >> renderonly GPUs. It does all the magic regarding in/exporting >> buffers etc. This library will likely break android support and >> hopefully will get replaced with a better solution based on gbm2. > > > Some more comments would be _really_ helpful. What is the purpose of a > "scanout" object? What does for_prime vs. for_resource mean? What does > intermediate_rendering mean and what is it good for? > Makes sense and I will try to come up with a nice way to document it. > > The lifecycle of the renderonly object itself looks wrong to me: > renderonly_screen_create calls the actual driver's screen_create, but the > actual driver's screen_create frees the renderonly struct? Please make it > consistent: Either have a proper wrapper or (better, since this is so > lightweight) have the driver's screen_create call the > renderonly_screen_create. > I will rework this part for v2. thanks a lot for your comments! -- Christian Gmeiner, MSc https://soundcloud.com/christian-gmeiner _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
