All of lore.kernel.org
 help / color / mirror / Atom feed
From: Igor Mammedov <imammedo@redhat.com>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: Dongjiu Geng <gengdongjiu@huawei.com>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	Xiao Guangrong <xiaoguangrong.eric@gmail.com>,
	Shannon Zhao <shannon.zhaosl@gmail.com>,
	Fam Zheng <fam@euphon.net>, Richard Henderson <rth@twiddle.net>,
	Eduardo Habkost <ehabkost@redhat.com>,
	Marcelo Tosatti <mtosatti@redhat.com>,
	QEMU Developers <qemu-devel@nongnu.org>,
	kvm-devel <kvm@vger.kernel.org>, qemu-arm <qemu-arm@nongnu.org>,
	Paolo Bonzini <pbonzini@redhat.com>,
	James Morse <james.morse@arm.com>,
	Laszlo Ersek <lersek@redhat.com>,
	Jonathan Cameron <jonathan.cameron@huawei.com>,
	Shameerali Kolothum Thodi  <shameerali.kolothum.thodi@huawei.com>,
	Zheng Xiang <zhengxiang9@huawei.com>
Subject: Re: [PATCH v24 02/10] hw/arm/virt: Introduce a RAS machine option
Date: Tue, 25 Feb 2020 11:53:22 +0100	[thread overview]
Message-ID: <20200225115322.4344e589@redhat.com> (raw)
In-Reply-To: <CAFEAcA9zjoa48Mth5aaOnhjDyY_qyrg9Nz5=0YEa2aE_aFcCug@mail.gmail.com>

On Tue, 25 Feb 2020 08:54:07 +0000
Peter Maydell <peter.maydell@linaro.org> wrote:

> On Tue, 25 Feb 2020 at 08:34, Igor Mammedov <imammedo@redhat.com> wrote:
> >
> > On Mon, 17 Feb 2020 21:12:40 +0800
> > Dongjiu Geng <gengdongjiu@huawei.com> wrote:
> >  
> > > RAS Virtualization feature is not supported now, so add a RAS machine  
> >  
> > > option and disable it by default.  
> >              ^^^^
> >
> > this doesn't match the patch.  
> 
> Hmm? It seems right to me -- the patch adds a machine option
> and makes it disabled-by-default, doesn't it?

Right, I misread 'and' as 'to' somehow.
My apologies

> 
> thanks
> -- PMM
> 

WARNING: multiple messages have this Message-ID (diff)
From: Igor Mammedov <imammedo@redhat.com>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: Fam Zheng <fam@euphon.net>,
	Xiao Guangrong <xiaoguangrong.eric@gmail.com>,
	kvm-devel <kvm@vger.kernel.org>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	Marcelo Tosatti <mtosatti@redhat.com>,
	QEMU Developers <qemu-devel@nongnu.org>,
	Eduardo Habkost <ehabkost@redhat.com>,
	Dongjiu Geng <gengdongjiu@huawei.com>,
	Shannon Zhao <shannon.zhaosl@gmail.com>,
	Zheng Xiang <zhengxiang9@huawei.com>,
	qemu-arm <qemu-arm@nongnu.org>, James Morse <james.morse@arm.com>,
	Shameerali Kolothum Thodi <shameerali.kolothum.thodi@huawei.com>,
	Jonathan Cameron <jonathan.cameron@huawei.com>,
	Paolo Bonzini <pbonzini@redhat.com>,
	Laszlo Ersek <lersek@redhat.com>,
	Richard Henderson <rth@twiddle.net>
Subject: Re: [PATCH v24 02/10] hw/arm/virt: Introduce a RAS machine option
Date: Tue, 25 Feb 2020 11:53:22 +0100	[thread overview]
Message-ID: <20200225115322.4344e589@redhat.com> (raw)
In-Reply-To: <CAFEAcA9zjoa48Mth5aaOnhjDyY_qyrg9Nz5=0YEa2aE_aFcCug@mail.gmail.com>

On Tue, 25 Feb 2020 08:54:07 +0000
Peter Maydell <peter.maydell@linaro.org> wrote:

> On Tue, 25 Feb 2020 at 08:34, Igor Mammedov <imammedo@redhat.com> wrote:
> >
> > On Mon, 17 Feb 2020 21:12:40 +0800
> > Dongjiu Geng <gengdongjiu@huawei.com> wrote:
> >  
> > > RAS Virtualization feature is not supported now, so add a RAS machine  
> >  
> > > option and disable it by default.  
> >              ^^^^
> >
> > this doesn't match the patch.  
> 
> Hmm? It seems right to me -- the patch adds a machine option
> and makes it disabled-by-default, doesn't it?

Right, I misread 'and' as 'to' somehow.
My apologies

> 
> thanks
> -- PMM
> 



  reply	other threads:[~2020-02-25 10:53 UTC|newest]

Thread overview: 52+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-02-17 13:12 [PATCH v24 00/10] Add ARMv8 RAS virtualization support in QEMU Dongjiu Geng
2020-02-17 13:12 ` Dongjiu Geng
2020-02-17 13:12 ` [PATCH v24 01/10] acpi: nvdimm: change NVDIMM_UUID_LE to a common macro Dongjiu Geng
2020-02-17 13:12   ` Dongjiu Geng
2020-02-21 14:07   ` Peter Maydell
2020-02-21 14:07     ` Peter Maydell
2020-02-17 13:12 ` [PATCH v24 02/10] hw/arm/virt: Introduce a RAS machine option Dongjiu Geng
2020-02-17 13:12   ` Dongjiu Geng
2020-02-25  8:34   ` Igor Mammedov
2020-02-25  8:34     ` Igor Mammedov
2020-02-25  8:54     ` Peter Maydell
2020-02-25  8:54       ` Peter Maydell
2020-02-25 10:53       ` Igor Mammedov [this message]
2020-02-25 10:53         ` Igor Mammedov
2020-02-17 13:12 ` [PATCH v24 03/10] docs: APEI GHES generation and CPER record description Dongjiu Geng
2020-02-17 13:12   ` Dongjiu Geng
2020-02-17 13:12 ` [PATCH v24 04/10] ACPI: Build related register address fields via hardware error fw_cfg blob Dongjiu Geng
2020-02-17 13:12   ` Dongjiu Geng
2020-02-25  8:48   ` Igor Mammedov
2020-02-25  8:48     ` Igor Mammedov
2020-02-25 13:57     ` Igor Mammedov
2020-02-25 13:57       ` Igor Mammedov
2020-02-17 13:12 ` [PATCH v24 05/10] ACPI: Build Hardware Error Source Table Dongjiu Geng
2020-02-17 13:12   ` Dongjiu Geng
2020-02-25 13:23   ` Igor Mammedov
2020-02-25 13:23     ` Igor Mammedov
2020-02-17 13:12 ` [PATCH v24 06/10] ACPI: Record the Generic Error Status Block address Dongjiu Geng
2020-02-17 13:12   ` Dongjiu Geng
2020-02-25 14:11   ` Igor Mammedov
2020-02-25 14:11     ` Igor Mammedov
2020-02-17 13:12 ` [PATCH v24 07/10] KVM: Move hwpoison page related functions into kvm-all.c Dongjiu Geng
2020-02-17 13:12   ` Dongjiu Geng
2020-02-17 13:12 ` [PATCH v24 08/10] ACPI: Record Generic Error Status Block(GESB) table Dongjiu Geng
2020-02-17 13:12   ` Dongjiu Geng
2020-02-25 16:58   ` Igor Mammedov
2020-02-25 16:58     ` Igor Mammedov
2020-02-17 13:12 ` [PATCH v24 09/10] target-arm: kvm64: handle SIGBUS signal from kernel or KVM Dongjiu Geng
2020-02-17 13:12   ` Dongjiu Geng
2020-02-21 14:02   ` Peter Maydell
2020-02-21 14:02     ` Peter Maydell
2020-02-17 13:12 ` [PATCH v24 10/10] MAINTAINERS: Add ACPI/HEST/GHES entries Dongjiu Geng
2020-02-17 13:12   ` Dongjiu Geng
2020-02-21 14:09 ` [PATCH v24 00/10] Add ARMv8 RAS virtualization support in QEMU Peter Maydell
2020-02-21 14:09   ` Peter Maydell
2020-02-24  8:37   ` gengdongjiu
2020-02-24  8:37     ` gengdongjiu
2020-02-25 16:59     ` Igor Mammedov
2020-02-25 16:59       ` Igor Mammedov
2020-02-26 16:34       ` gengdongjiu
2020-02-26 16:34         ` gengdongjiu
  -- strict thread matches above, loose matches on Subject: below --
2020-02-25 12:51 [PATCH v24 02/10] hw/arm/virt: Introduce a RAS machine option gengdongjiu
2020-02-25 12:51 ` gengdongjiu

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=20200225115322.4344e589@redhat.com \
    --to=imammedo@redhat.com \
    --cc=ehabkost@redhat.com \
    --cc=fam@euphon.net \
    --cc=gengdongjiu@huawei.com \
    --cc=james.morse@arm.com \
    --cc=jonathan.cameron@huawei.com \
    --cc=kvm@vger.kernel.org \
    --cc=lersek@redhat.com \
    --cc=mst@redhat.com \
    --cc=mtosatti@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-arm@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=rth@twiddle.net \
    --cc=shameerali.kolothum.thodi@huawei.com \
    --cc=shannon.zhaosl@gmail.com \
    --cc=xiaoguangrong.eric@gmail.com \
    --cc=zhengxiang9@huawei.com \
    /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.