[Mesa-dev] [PATCH] gallium/{r600, radeonsi}: Fix segfault with color format (v2)

Denis Pauk pauk.denis at gmail.com
Thu Sep 14 05:24:24 UTC 2017


Thank you.

I have closed https://bugs.freedesktop.org/show_bug.cgi?id=102552

Best regards,
                  Denis.

On Sep 14, 2017 2:01 AM, "Marek Olšák" <maraeo at gmail.com> wrote:

> I pushed the patch. Thanks.
>
> Marek
>
> On Wed, Sep 13, 2017 at 9:27 PM, Denis Pauk <pauk.denis at gmail.com> wrote:
> > Could you please commit this changes?
> >
> > I have not found what i need to do next after recieve "approve" for
> changes
> > in https://www.mesa3d.org/submittingpatches.html#reviewing.
> > Do i need to send new mail with "Reviewed-by: " mark?
> >
> > On Wed, Sep 13, 2017 at 6:36 PM, Marek Olšák <maraeo at gmail.com> wrote:
> >>
> >> I've changed my mind. The patch is OK:
> >>
> >> Reviewed-by: Marek Olšák <marek.olsak at amd.com>
> >>
> >> Marek
> >>
> >> On Wed, Sep 13, 2017 at 4:45 PM, Denis Pauk <pauk.denis at gmail.com>
> wrote:
> >> > What do you think about replace all checks in patch to asserts?
> >> >
> >> >
> >> > Best regards,
> >> >                   Denis.
> >> >
> >> > On Sep 13, 2017 1:00 PM, "Marek Olšák" <maraeo at gmail.com> wrote:
> >> >>
> >> >> On Wed, Sep 13, 2017 at 6:54 AM, Денис Паук <pauk.denis at gmail.com>
> >> >> wrote:
> >> >> > Do you mean delete check in u_format.c:: util_format_is_supported?
> >> >> > Could
> >> >> > you
> >> >> > please explain more?
> >> >>
> >> >> I mean delete everything. Invalid formats shouldn't be passed to
> >> >> pipe_screen::is_format_supported. The code doing it needs to be
> fixed.
> >> >>
> >> >> Marek
> >
> >
> >
> >
> > --
> > Best regards,
> >                   Denis.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170914/989cfaa7/attachment.html>


More information about the mesa-dev mailing list