mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Jan Marek <linux@hazard.jcu.cz>
To: linux-kernel <linux-kernel@vger.kernel.org>
Subject: [linux@hazard.jcu.cz: Re: [miniPATCH][RFC] Compilation fixes in the 2.5.44]
Date: Fri, 25 Oct 2002 13:29:23 +0200	[thread overview]
Message-ID: <20021025112923.GB1073@hazard.jcu.cz> (raw)

[-- Attachment #1: Type: text/plain, Size: 56 bytes --]

Hallo l-k,

I resend my answer to l-k too...

Jan Marek

[-- Attachment #2: Type: message/rfc822, Size: 2143 bytes --]

From: Jan Marek <linux@hazard.jcu.cz>
To: Andrey Panin <pazke@orbita1.ru>
Subject: Re: [miniPATCH][RFC] Compilation fixes in the 2.5.44
Date: Fri, 25 Oct 2002 12:15:19 +0200
Message-ID: <20021025101518.GA1073@hazard.jcu.cz>

Hallo l-k,

On Fri, Oct 25, 2002 at 12:01:57PM +0400, Andrey Panin wrote:
> On Fri, Oct 25, 2002 at 09:44:21AM -0200, Denis Vlasenko wrote:
> > On 25 October 2002 04:28, Jan Marek wrote:
> > > Hallo l-k,
> > >
> > > I'm beginner in the kernel hacking (or fixing ;-))).
> > >
> > > I have small patch, which is fixing some compilation errors (I'm
> > > using gcc-2.95.4-17 from Debian sid).
> > >
> > > The first chunk fixed this warning:
> > >
> > > arch/i386/kernel/irq.c: In function `do_IRQ':
> > > arch/i386/kernel/irq.c:331: warning: unused variable `esp'
> > >
> > > I move declaration of variable esp to the #ifdef blok, where it is
> > > using...
> > 
> > 
> >         unsigned int status;
> > -       long esp;
> >  
> >         irq_enter();
> >  
> >  #ifdef CONFIG_DEBUG_STACKOVERFLOW
> >         /* Debugging check for stack overflow: is there less than 1KB free? */
> > +       long esp;
> > 
> > Most C compilers don't allow you to mix declarations and code.
> > This is allowed only in new C standards. But GCC 3 seems to cope,
> > so it's probably fine for new kernels.
> 
> This fragment must be fixed, look at Documentation/Changes:

gcc-2.95.4-17 on my Debian works fine on that and without any
messages... You can try it, if you have other version of compiler...

> 
> "The recommended compiler for the kernel is gcc 2.95.x (x >= 3)"
> 
> Best regards.
>  
> -- 
> Andrey Panin            | Embedded systems software developer
> pazke@orbita1.ru        | PGP key: wwwkeys.eu.pgp.net

Sincerely
Jan Marek
-- 
Ing. Jan Marek
University of South Bohemia
Academic Computer Centre
Phone: +420-38-7772080

             reply	other threads:[~2002-10-25 11:23 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-10-25 11:29 Jan Marek [this message]
2002-10-25 13:18 ` Dave Jones
2002-10-25 13:41   ` Jan Marek
2002-10-25 14:13     ` Jan Marek

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20021025112923.GB1073@hazard.jcu.cz \
    --to=linux@hazard.jcu.cz \
    --cc=linux-kernel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®