From: Jan Kiszka <jan.kiszka@web.de>
To: Lukas Czerner <lczerner@redhat.com>
Cc: Jeff Dike <jdike@addtoit.com>,
user-mode-linux-devel@lists.sourceforge.net
Subject: Re: [uml-devel] [PATCH] um: Fix compile errors for um architecture
Date: Tue, 20 Apr 2010 00:02:04 +0200 [thread overview]
Message-ID: <4BCCD2DC.7020508@web.de> (raw)
In-Reply-To: <1271700367-14127-1-git-send-email-lczerner@redhat.com>
[-- Attachment #1.1: Type: text/plain, Size: 1704 bytes --]
Lukas Czerner wrote:
> arch/um/drivers/line.c
> - need to include linux/slab.h since the code uses kmalloc.
> - fix warning message with unused variable tty
>
> arch/um/os-Linux/helper.c
> - there is no need to include linux/slab.h since allocations are
> done with uml_kmalloc
The build fixes are already queued in Tejun's percpu tree, the variable
removal is part of my just posted trivial series. But what an amazing
activity here! :)
Thanks,
Jan
>
> Signed-off-by: Lukas Czerner <lczerner@redhat.com>
> ---
> arch/um/drivers/line.c | 2 +-
> arch/um/os-Linux/helper.c | 1 -
> 2 files changed, 1 insertions(+), 2 deletions(-)
>
> diff --git a/arch/um/drivers/line.c b/arch/um/drivers/line.c
> index 64cda95..7f7338c 100644
> --- a/arch/um/drivers/line.c
> +++ b/arch/um/drivers/line.c
> @@ -6,6 +6,7 @@
> #include "linux/irqreturn.h"
> #include "linux/kd.h"
> #include "linux/sched.h"
> +#include "linux/slab.h"
> #include "chan_kern.h"
> #include "irq_kern.h"
> #include "irq_user.h"
> @@ -18,7 +19,6 @@ static irqreturn_t line_interrupt(int irq, void *data)
> {
> struct chan *chan = data;
> struct line *line = chan->line;
> - struct tty_struct *tty;
>
> if (line)
> chan_interrupt(&line->chan_list, &line->task, line->tty, irq);
> diff --git a/arch/um/os-Linux/helper.c b/arch/um/os-Linux/helper.c
> index 06d6ccf..b6b1096 100644
> --- a/arch/um/os-Linux/helper.c
> +++ b/arch/um/os-Linux/helper.c
> @@ -8,7 +8,6 @@
> #include <errno.h>
> #include <sched.h>
> #include <linux/limits.h>
> -#include <linux/slab.h>
> #include <sys/socket.h>
> #include <sys/wait.h>
> #include "kern_constants.h"
[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 257 bytes --]
[-- Attachment #2: Type: text/plain, Size: 345 bytes --]
------------------------------------------------------------------------------
Download Intel® Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
[-- Attachment #3: Type: text/plain, Size: 194 bytes --]
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel
prev parent reply other threads:[~2010-04-19 22:02 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-04-19 18:06 [uml-devel] [PATCH] um: Fix compile errors for um architecture Lukas Czerner
2010-04-19 22:02 ` Jan Kiszka [this message]
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=4BCCD2DC.7020508@web.de \
--to=jan.kiszka@web.de \
--cc=jdike@addtoit.com \
--cc=lczerner@redhat.com \
--cc=user-mode-linux-devel@lists.sourceforge.net \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.