From mboxrd@z Thu Jan 1 00:00:00 1970 From: Cyrill Gorcunov Subject: [PATCH] kvm tools: Drop unused vars from int10.c code Date: Mon, 23 May 2011 18:39:17 +0400 Message-ID: <4DDA7195.4030507@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Sasha Levin , kvm-vger , Ingo Molnar , Prasad Joshi To: Pekka Enberg Return-path: Received: from mail-ew0-f46.google.com ([209.85.215.46]:59118 "EHLO mail-ew0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755253Ab1EWOjV (ORCPT ); Mon, 23 May 2011 10:39:21 -0400 Received: by ewy4 with SMTP id 4so1837142ewy.19 for ; Mon, 23 May 2011 07:39:20 -0700 (PDT) Sender: kvm-owner@vger.kernel.org List-ID: There is a couple of functions which defines 'ah' variable but never use it in real so that gcc 4.6.x series does complain on me as CC bios/bios-rom.bin bios/int10.c: In function =E2=80=98int10_putchar=E2=80=99: bios/int10.c:86:9: error: variable =E2=80=98ah=E2=80=99 set but not u= sed [-Werror=3Dunused-but-set-variable] bios/int10.c: In function =E2=80=98int10_vesa=E2=80=99: bios/int10.c:96:9: error: variable =E2=80=98ah=E2=80=99 set but not u= sed [-Werror=3Dunused-but-set-variable] cc1: all warnings being treated as errors so get rid of them. Signed-off-by: Cyrill Gorcunov CC: Sasha Levin --- tools/kvm/bios/int10.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) Index: linux-2.6.git/tools/kvm/bios/int10.c =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D --- linux-2.6.git.orig/tools/kvm/bios/int10.c +++ linux-2.6.git/tools/kvm/bios/int10.c @@ -83,22 +83,18 @@ static inline void outb(unsigned short p */ static inline void int10_putchar(struct int10_args *args) { - u8 al, ah; - - al =3D args->eax & 0xFF; - ah =3D (args->eax & 0xFF00) >> 8; + u8 al =3D args->eax & 0xFF; outb(0x3f8, al); } static void int10_vesa(struct int10_args *args) { - u8 al, ah; + u8 al; struct vesa_general_info *destination; struct vminfo *vi; al =3D args->eax; - ah =3D args->eax >> 8; switch (al) { case 0: