All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stefan Bader <stefan.bader@canonical.com>
To: xen-devel@lists.xen.org
Subject: Re: [PATCH] mm/x86: Patch out arch_flush_lazy_mmu_mode() when running on bare metal
Date: Thu, 21 Mar 2013 15:39:57 +0100	[thread overview]
Message-ID: <514B1BBD.2030405@canonical.com> (raw)
In-Reply-To: <20130321140757.GC28648@phenom.dumpdata.com>


[-- Attachment #1.1: Type: text/plain, Size: 2145 bytes --]

On 21.03.2013 15:07, Konrad Rzeszutek Wilk wrote:
> On Wed, Mar 13, 2013 at 09:44:43AM -0400, Josh Boyer wrote:
>> On Wed, Mar 13, 2013 at 09:25:44AM -0400, Boris Ostrovsky wrote:
>>> On 03/01/2013 07:14 AM, Josh Boyer wrote:
>>>> On Thu, Feb 28, 2013 at 04:52:20PM -0800, H. Peter Anvin wrote:
>>>>> On 02/28/2013 04:42 PM, Josh Boyer wrote:
>>>>>> On Fri, Mar 01, 2013 at 01:36:29AM +0100, Borislav Petkov wrote:
>>>>>>> On Thu, Feb 28, 2013 at 04:15:45PM -0800, H. Peter Anvin wrote:
>>>>>>>>> I'll try to get someone to test this tomorrow.
>>>>>>> Btw, you'd need to apply that other patch too
>>>>>>>
>>>>>>> http://marc.info/?l=xen-devel&m=136206183814547&w=2
>>>>>>>
>>>>>>> so that arch_flush_lazy_mmu_mode() has at least one caller on x86_64.
>>>>>> Yeah, we already have that applied.  It stops crashes in xen
>>>>>> environments so we pulled it in as a bugfix.  Thanks though!
>>>>>>
>>>>> Who are "we"?
>>>> Sorry, Fedora.  That patch has a link to a bug in it.  We applied the
>>>> patch for that bug.  I'll apply Boris' patch on top and get the same
>>>> people to test it.
>>>
>>> Josh, have you had a chance to test this?
>>
>> I've tested it on bare metal for a while now.  No problems noticed at
>> all.  I've not heard back from Krishna who was testing it in the Xen
>> environment.  Krishna?
> 
> Roping in Stephan from Canonical as they are seeing it on a regular basis
> and is affecting their distro.

That is unfortunately overstated. Just was asked about it by someone who seems
affected and who had found the redhat bug. From that I found the other thread
(which added the lazy_mmu_flush).
Well, I passed on the link to this thread so maybe this will get some confirmation.

> 
> Stephan - there are two patches for it - the paravirt of lazy_mmu mode
> and then the other one.
> 
> This thread should provide the right URLs (I hope). Could you (if it is possible)
> try them out and see if they fix the issue?
> 
>>
>> josh
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel
> 



[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 899 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

  reply	other threads:[~2013-03-21 14:39 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-28 22:55 [PATCH] mm/x86: Patch out arch_flush_lazy_mmu_mode() when running on bare metal Boris Ostrovsky
2013-02-28 23:37 ` Borislav Petkov
2013-02-28 23:37 ` Borislav Petkov
2013-03-01  0:05   ` Josh Boyer
2013-03-01  0:15     ` H. Peter Anvin
2013-03-01  0:15     ` H. Peter Anvin
2013-03-01  0:36       ` Borislav Petkov
2013-03-01  0:36       ` Borislav Petkov
2013-03-01  0:42         ` Josh Boyer
2013-03-01  0:42         ` Josh Boyer
2013-03-01  0:52           ` H. Peter Anvin
2013-03-01  0:52           ` H. Peter Anvin
2013-03-01 12:14             ` Josh Boyer
2013-03-01 12:14             ` Josh Boyer
2013-03-13 13:25               ` Boris Ostrovsky
2013-03-13 13:44                 ` Josh Boyer
2013-03-13 13:44                 ` Josh Boyer
2013-03-21 14:07                   ` Konrad Rzeszutek Wilk
2013-03-21 14:39                     ` Stefan Bader [this message]
2013-03-22  1:11                       ` Konrad Rzeszutek Wilk
2013-03-21 14:07                   ` Konrad Rzeszutek Wilk
2013-03-13 13:25               ` Boris Ostrovsky
2013-03-01  0:05   ` Josh Boyer
2013-03-04 15:13 ` Konrad Rzeszutek Wilk
2013-03-04 15:13 ` Konrad Rzeszutek Wilk
  -- strict thread matches above, loose matches on Subject: below --
2013-02-28 22:55 Boris Ostrovsky
2013-03-20 13:53 Boris Ostrovsky
2013-03-21  0:08 ` Josh Boyer
2013-03-21  0:08 ` Josh Boyer
2013-03-22 20:09   ` Konrad Rzeszutek Wilk
2013-03-22 20:25     ` H. Peter Anvin
2013-03-22 20:25     ` H. Peter Anvin
2013-03-22 20:09   ` Konrad Rzeszutek Wilk
2013-03-20 13:53 Boris Ostrovsky

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=514B1BBD.2030405@canonical.com \
    --to=stefan.bader@canonical.com \
    --cc=xen-devel@lists.xen.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 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.