[PATCH libdrm 0/5] modetest: fix misc when terminates modetest

[PATCH libdrm 0/5] modetest: fix misc when terminates modetest

[Joonyoung Shim]

Hi all,

This patchset is to fix properly about buffer and framebuffer resources
when terminates modetest of libdrm.

Thanks.

Joonyoung Shim (5):
      modetest: fix Segmentation fault
      modetest: add to use drmModeRmFB
      modetest: fix to proper error path handling
      modetest: clear buffer and framebuffer of planes
      modetest: destroy buffer of cursor

 tests/modetest/modetest.c | 49 ++++++++++++++++++++++++++++++++++++++++++-----
 1 file changed, 44 insertions(+), 5 deletions(-)
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH libdrm 0/5] modetest: fix misc when terminates modetest

[Emil Velikov]

Hi Joonyoung,

On 13/04/15 08:31, Joonyoung Shim wrote:
> Hi all,
> 
> This patchset is to fix properly about buffer and framebuffer resources
> when terminates modetest of libdrm.
> 
The series looks great (incl. the MAKE_RGB_INFO fix) and barring any
objections I'll push it by the end of the week. Just a small sidenote -
some of the commit messages are a bit hard to read. Hope you don't mind
if I change them slightly before pushing.

Thanks
Emil

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH libdrm 0/5] modetest: fix misc when terminates modetest

[Joonyoung Shim]

Hi Emil,

On 04/22/2015 05:16 AM, Emil Velikov wrote:
> Hi Joonyoung,
> 
> On 13/04/15 08:31, Joonyoung Shim wrote:
>> Hi all,
>>
>> This patchset is to fix properly about buffer and framebuffer resources
>> when terminates modetest of libdrm.
>>
> The series looks great (incl. the MAKE_RGB_INFO fix) and barring any
> objections I'll push it by the end of the week. Just a small sidenote -
> some of the commit messages are a bit hard to read. Hope you don't mind
> if I change them slightly before pushing.

It's ok, i guess i should thank you.

Thanks.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH libdrm 0/5] modetest: fix misc when terminates modetest

[Emil Velikov]

On 22/04/15 02:05, Joonyoung Shim wrote:
> Hi Emil,
> 
> On 04/22/2015 05:16 AM, Emil Velikov wrote:
>> Hi Joonyoung,
>>
>> On 13/04/15 08:31, Joonyoung Shim wrote:
>>> Hi all,
>>>
>>> This patchset is to fix properly about buffer and framebuffer resources
>>> when terminates modetest of libdrm.
>>>
>> The series looks great (incl. the MAKE_RGB_INFO fix) and barring any
>> objections I'll push it by the end of the week. Just a small sidenote -
>> some of the commit messages are a bit hard to read. Hope you don't mind
>> if I change them slightly before pushing.
> 
> It's ok, i guess i should thank you.
> 
I seriously won't be offended if you don't, so it's entirely up-to you.

Cheers
Emil

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH libdrm 0/5] modetest: fix misc when terminates modetest

[Emil Velikov]

On 22/04/15 02:05, Joonyoung Shim wrote:
> Hi Emil,
> 
> On 04/22/2015 05:16 AM, Emil Velikov wrote:
>> Hi Joonyoung,
>>
>> On 13/04/15 08:31, Joonyoung Shim wrote:
>>> Hi all,
>>>
>>> This patchset is to fix properly about buffer and framebuffer resources
>>> when terminates modetest of libdrm.
>>>
>> The series looks great (incl. the MAKE_RGB_INFO fix) and barring any
>> objections I'll push it by the end of the week. Just a small sidenote -
>> some of the commit messages are a bit hard to read. Hope you don't mind
>> if I change them slightly before pushing.
> 
> It's ok, i guess i should thank you.
> 
In case you've missed it - I corrected the value for invalid fb_id.
According to the kernel and other users - 0 should be used, as (unsigned
int)-1 (used in the series) is valid one, albeit hard to hit.

Thanks again for the fixes.

-Emil
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel