Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1729009

Re: [PATCH v11 4/6] target-arm: kvm64: detect guest RAS EXTENSION feature

From Peter Maydell <peter.maydell@linaro.org>
Newsgroups linux.kernel
Subject Re: [PATCH v11 4/6] target-arm: kvm64: detect guest RAS EXTENSION feature
Date 2017-09-08 17:00 +0200
Message-ID <unsVc-8sm-15@gated-at.bofh.it> (permalink)
References <unssa-8fk-23@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On 8 September 2017 at 15:26, gengdongjiu <gengdongjiu@huawei.com> wrote:


>> Shouldn't we need to also tell the kernel that we actually want
>> it to expose RAS to the guest? Compare the PMU code in this function,
>> where we set a kvm_init_features bit to do this.

> In the PMU code, it indeed sets a kvm_init_features bit. Here ARM
> James has a concern that we are depend on the host CPU RAS extension,
> He means that if userspace receives the SIGBUS delivered by host
> memory_failure(), user space should record the CPER for guest
> and handling the error regardless whether host CPU supports RAS
> extension. But I think if user space receives the SIGBUS signal,
> that means
> host CPU RAS module detects the error or CPU consumes the poison
> data, thus we should check whether physical CPU support RAS extension.

I don't understand what you have in mind here. If the host does
not support the CPU RAS extension then we should never get a
SIGBUS in the first place.

In any case this doesn't seem relevant to the question of whether it
should be optional to expose the RAS extension to the *guest*.
Even if the host does support RAS, you should be able to run a
VM that knows nothing about RAS.

thanks
-- PMM

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread


Thread

Re: [PATCH v11 4/6] target-arm: kvm64: detect guest RAS EXTENSION  feature gengdongjiu <gengdongjiu@huawei.com> - 2017-09-08 16:30 +0200
  Re: [PATCH v11 4/6] target-arm: kvm64: detect guest RAS EXTENSION feature Peter Maydell <peter.maydell@linaro.org> - 2017-09-08 17:00 +0200

csiph-web