From mboxrd@z Thu Jan 1 00:00:00 1970 From: Namhyung Kim Subject: Re: [PATCH] fix compilation error of perf/core Date: Tue, 25 Sep 2012 17:50:30 +0900 Message-ID: <87haqmwko9.fsf@sejong.aot.lge.com> References: <1348561285-25587-1-git-send-email-haodong@linux.vnet.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: acme@infradead.org, linux-kernel@vger.kernel.org, kvm@vger.kernel.org, xiaoguangrong@linux.vnet.ibm.com, runzhen@linux.vnet.ibm.com To: Dong Hao Return-path: In-Reply-To: <1348561285-25587-1-git-send-email-haodong@linux.vnet.ibm.com> (Dong Hao's message of "Tue, 25 Sep 2012 16:21:25 +0800") Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org Hi Dong, On Tue, 25 Sep 2012 16:21:25 +0800, Dong Hao wrote: > From: Dong Hao > > The newest branch of perf/core should have compilation error! > Error log includes: > > builtin-test.c: In function =E2=80=98perf_evsel__test_field=E2=80=99: > builtin-test.c:1216:6: error: > variable =E2=80=98ret=E2=80=99 set but not used [-Werror=3Dunused-bu= t-set-variable] > builtin-test.c: In function =E2=80=98perf_evsel__tp_sched_test=E2=80=99= : > builtin-test.c:1242:6: error: > variable =E2=80=98ret=E2=80=99 set but not used [-Werror=3Dunused-bu= t-set-variable] > cc1: all warnings being treated as errors > make: *** [builtin-test.o] Error 1 > --------------------------------------------------------------------- > Fix it by replacing return value from 0 to ret. > Then this branch can be compiled successfully. > > Signed-off-by: Dong Hao I already sent the same patch: https://lkml.org/lkml/2012/9/24/689 Thanks for fixing it anyway! Namhyung