[PATCH libdrm 0/5] modetest: fix misc when terminates modetest
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
[PATCH libdrm 0/5] modetest: fix misc when terminates modetest
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
[PATCH libdrm 0/5] modetest: fix misc when terminates modetest
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.
[PATCH libdrm 0/5] modetest: fix misc when terminates modetest
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
[PATCH libdrm 0/5] modetest: fix misc when terminates modetest
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(-)