linux-kernel - Re: [GIT PULL 0/6] perf/urgent fixes

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Mon, 13 Jan 2020 15:14:40 +0530
From:   Ravi Bangoria <ravi.bangoria@...ux.ibm.com>
To:     Jiri Olsa <jolsa@...hat.com>
Cc:     Arnaldo Carvalho de Melo <acme@...nel.org>,
        Jiri Olsa <jolsa@...nel.org>, Ingo Molnar <mingo@...nel.org>,
        Thomas Gleixner <tglx@...utronix.de>,
        Namhyung Kim <namhyung@...nel.org>,
        Clark Williams <williams@...hat.com>,
        linux-kernel@...r.kernel.org, linux-perf-users@...r.kernel.org,
        Adrian Hunter <adrian.hunter@...el.com>,
        Arnaldo Carvalho de Melo <acme@...hat.com>,
        Ravi Bangoria <ravi.bangoria@...ux.ibm.com>
Subject: Re: [GIT PULL 0/6] perf/urgent fixes



On 1/13/20 2:55 PM, Jiri Olsa wrote:
> On Mon, Jan 13, 2020 at 01:58:59PM +0530, Ravi Bangoria wrote:
> 
> SNIP
> 
>>          | ^~~~
>>    In file included from /usr/include/glib-2.0/gobject/gobject.h:24,
>>                     from /usr/include/glib-2.0/gobject/gbinding.h:29,
>>                     from /usr/include/glib-2.0/glib-object.h:23,
>>                     from /usr/include/glib-2.0/gio/gioenums.h:28,
>>                     from /usr/include/glib-2.0/gio/giotypes.h:28,
>>                     from /usr/include/glib-2.0/gio/gio.h:26,
>>                     from /usr/include/gtk-2.0/gdk/gdkapplaunchcontext.h:30,
>>                     from /usr/include/gtk-2.0/gdk/gdk.h:32,
>>                     from /usr/include/gtk-2.0/gtk/gtk.h:32,
>>                     from test-gtk2.c:3:
>>    /usr/include/glib-2.0/gobject/gtype.h:679:1: note: declared here
>>      679 | {
>>          | ^
>>    In file included from /usr/include/gtk-2.0/gtk/gtktoolitem.h:31,
>>                     from /usr/include/gtk-2.0/gtk/gtktoolbutton.h:30,
>>                     from /usr/include/gtk-2.0/gtk/gtkmenutoolbutton.h:30,
>>                     from /usr/include/gtk-2.0/gtk/gtk.h:126,
>>                     from test-gtk2.c:3:
>>    /usr/include/gtk-2.0/gtk/gtktooltips.h:73:3: error: ‘GTimeVal’ is deprecated: Use 'GDateTime' instead [-Werror=deprecated-declarations]
>>       73 |   GTimeVal last_popdown;
>>          |   ^~~~~~~~
>>    In file included from /usr/include/glib-2.0/glib/galloca.h:32,
>>                     from /usr/include/glib-2.0/glib.h:30,
>>                     from /usr/include/glib-2.0/gobject/gbinding.h:28,
>>                     from /usr/include/glib-2.0/glib-object.h:23,
>>                     from /usr/include/glib-2.0/gio/gioenums.h:28,
>>                     from /usr/include/glib-2.0/gio/giotypes.h:28,
>>                     from /usr/include/glib-2.0/gio/gio.h:26,
>>                     from /usr/include/gtk-2.0/gdk/gdkapplaunchcontext.h:30,
>>                     from /usr/include/gtk-2.0/gdk/gdk.h:32,
>>                     from /usr/include/gtk-2.0/gtk/gtk.h:32,
>>                     from test-gtk2.c:3:
>>    /usr/include/glib-2.0/glib/gtypes.h:551:8: note: declared here
>>      551 | struct _GTimeVal
>>          |        ^~~~~~~~~
>>    cc1: all warnings being treated as errors
>>
> 
> patch below fixes that for me.. please let me know
> if it works for you and I'll post full patch
> 
> jirka
> 

LGTM. You can add:

Tested-by: Ravi Bangoria <ravi.bangoria@...ux.ibm.com>