All of lore.kernel.org
 help / color / mirror / Atom feed
From: Borislav Petkov <borislav.petkov@amd.com>
To: Jeff Dike <jdike@addtoit.com>
Cc: "Borislav Petkov" <bp@alien8.de>,
	"H. Peter Anvin" <hpa@zytor.com>,
	"Geert Uytterhoeven" <geert@linux-m68k.org>,
	"Toralf Förster" <toralf.foerster@gmx.de>,
	"user-mode-linux-devel@lists.sourceforge.net"
	<user-mode-linux-devel@lists.sourceforge.net>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] x86, hweight: Fix UML boot crash
Date: Mon, 31 May 2010 15:51:32 +0200	[thread overview]
Message-ID: <20100531135132.GA4915@aftab> (raw)
In-Reply-To: <20100531023212.GA22673@c-98-229-117-200.hsd1.ma.comcast.net>

From: Jeff Dike <jdike@addtoit.com>
Date: Sun, May 30, 2010 at 10:32:12PM -0400

> On Sun, May 30, 2010 at 09:39:56PM +0200, Borislav Petkov wrote:
> > Which begs the question why _is_ UML sucking in x86 stuff and can anyone
> > provide us with some sensible reasons? Because if there aren't any, it
> > is their includes that should be fixed. Let me see what I can do to
> > redirect hweight stuff properly...
> 
> Generally, UML pulls in the host arch headers because they work.  When
> they are only architecture-dependent (and not, say, depending on the
> host task struct or something), they're fine.
> 
> What's the include path from UML to the x86 hweight stuff?

<arch/x86/include/asm/bitops.h> includes <asm/arch_hweight.h> which are
the optimized variants.

I have a patch which with which UML falls back to the defaults:
http://marc.info/?l=linux-kernel&m=127525067908139&w=2 but hpa's concern
is still valid: UML shouldn't choke on the optimized variants. Anyways,
here's the original commit d61931d89be506372d01a90d1755f6d0a9fafe2d -
you might be able to find something which interferes with UML in there.

Thanks.

-- 
Regards/Gruss,
Boris.

Operating Systems Research Center
Advanced Micro Devices, Inc.



  reply	other threads:[~2010-05-31 13:51 UTC|newest]

Thread overview: 52+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-27 17:44 current git kernel crashes UML system during boot Toralf Förster
2010-05-30 11:39 ` Geert Uytterhoeven
2010-05-30 11:57   ` Geert Uytterhoeven
2010-05-30 15:02     ` Borislav Petkov
2010-05-30 15:18       ` Geert Uytterhoeven
2010-05-30 15:46         ` Borislav Petkov
2010-05-30 15:28       ` Toralf Förster
2010-05-30 17:03         ` [PATCH] x86, hweight: Fix UML boot crash Borislav Petkov
2010-05-30 18:36           ` H. Peter Anvin
2010-05-30 19:39             ` Borislav Petkov
2010-05-30 20:17               ` Borislav Petkov
2010-05-30 21:09                 ` H. Peter Anvin
2010-06-12 13:34                   ` [uml-devel] " Paolo Giarrusso
2010-06-12 14:18                     ` Borislav Petkov
2010-06-12 16:01                       ` Paolo Giarrusso
2010-06-12 16:34                         ` Borislav Petkov
2010-06-12 18:37                           ` Geert Uytterhoeven
2010-06-13  6:58                             ` Borislav Petkov
2010-05-31 13:55                 ` Toralf Förster
2010-05-31 14:10                   ` Borislav Petkov
2010-05-31 14:36                     ` Toralf Förster
2010-05-31 14:36                       ` Toralf Förster
2010-06-14  9:49                     ` Toralf Förster
2010-06-14  9:49                       ` Toralf Förster
2010-06-14 10:26                       ` [uml-devel] " Paolo Giarrusso
2010-06-14 12:54                         ` Toralf Förster
2010-06-14 13:00                       ` Borislav Petkov
2010-06-14 13:00                         ` Borislav Petkov
2010-06-14 13:44                         ` Toralf Förster
2010-06-14 13:44                           ` Toralf Förster
2010-06-14 14:39                           ` Geert Uytterhoeven
2010-06-14 15:11                             ` Toralf Förster
2010-06-14 15:20                               ` H. Peter Anvin
2010-05-31 14:25                   ` [uml-devel] " Boaz Harrosh
2010-05-31  2:32               ` Jeff Dike
2010-05-31 13:51                 ` Borislav Petkov [this message]
2010-05-31 15:56                   ` Jeff Dike
2010-05-31 16:29                     ` Borislav Petkov
     [not found]           ` <tip-cc7f0a7b3004a4ca0bfef0e1ca79f2e0da6ca1b0@git.kernel.org>
2010-06-09 22:43             ` [tip:x86/urgent] " H. Peter Anvin
2010-06-10  5:32               ` Geert Uytterhoeven
2010-06-10  6:26                 ` H. Peter Anvin
2010-06-09  8:04 ` [PATCH resend] um, " Boaz Harrosh
2010-06-09  8:09   ` [uml-devel] " Boaz Harrosh
2010-06-09  8:46     ` Geert Uytterhoeven
2010-06-09  9:08       ` Boaz Harrosh
2010-06-09  9:32       ` Américo Wang
2010-06-09 18:29       ` H. Peter Anvin
2010-06-14 16:08         ` Boaz Harrosh
2010-06-14 16:41           ` H. Peter Anvin
2010-06-14 18:18             ` Boaz Harrosh
2010-06-09 16:39   ` Boaz Harrosh
2010-06-10 23:43   ` [tip:x86/urgent] um, hweight: Fix UML boot crash due to x86 optimized hweight tip-bot for Borislav Petkov

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=20100531135132.GA4915@aftab \
    --to=borislav.petkov@amd.com \
    --cc=bp@alien8.de \
    --cc=geert@linux-m68k.org \
    --cc=hpa@zytor.com \
    --cc=jdike@addtoit.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=toralf.foerster@gmx.de \
    --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.