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

Groups > linux.kernel > #1200110 > unrolled thread

Re: [PATCH V9 0/5] map GHES memory region according to EFI memory map

Started by"Zhang, Jonathan Zhixiong" <zjzhang@codeaurora.org>
First post2015-08-04 17:50 +0200
Last post2015-08-05 18:20 +0200
Articles 6 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH V9 0/5] map GHES memory region according to EFI memory map "Zhang, Jonathan Zhixiong" <zjzhang@codeaurora.org> - 2015-08-04 17:50 +0200
    Re: [PATCH V9 0/5] map GHES memory region according to EFI memory map Matt Fleming <matt@codeblueprint.co.uk> - 2015-08-05 11:30 +0200
      Re: [PATCH V9 0/5] map GHES memory region according to EFI memory map "Zhang, Jonathan Zhixiong" <zjzhang@codeaurora.org> - 2015-08-05 18:00 +0200
        Re: [PATCH V9 0/5] map GHES memory region according to EFI memory map Matt Fleming <matt@codeblueprint.co.uk> - 2015-08-05 18:20 +0200
          Re: [PATCH V9 0/5] map GHES memory region according to EFI memory map "Zhang, Jonathan Zhixiong" <zjzhang@codeaurora.org> - 2015-08-06 15:50 +0200
        Re: [PATCH V9 0/5] map GHES memory region according to EFI memory map Matt Fleming <matt@codeblueprint.co.uk> - 2015-08-05 18:20 +0200

#1200110 — Re: [PATCH V9 0/5] map GHES memory region according to EFI memory map

From"Zhang, Jonathan Zhixiong" <zjzhang@codeaurora.org>
Date2015-08-04 17:50 +0200
SubjectRe: [PATCH V9 0/5] map GHES memory region according to EFI memory map
Message-ID<pTMDw-6sx-23@gated-at.bofh.it>
On 8/3/2015 9:25 PM, Borislav Petkov wrote:
> On Mon, Aug 03, 2015 at 05:23:54PM +0100, Matt Fleming wrote:
>> Rafael, Boris?
>
> The ghes.c change looks fine I guess. The whole patchset makes sense
> now, with the arch bits extracted. So
>
> Acked-by: Borislav Petkov <bp@suse.de>
Thank you Boris/Matt/Will for the support.

>
> However, we probably should work towards adhering to EFI memory
> attributes on x86, long term, as we talked. But that's a future thing.
>
> Thanks.
>

-- 
Jonathan (Zhixiong) Zhang
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]

#1200578

FromMatt Fleming <matt@codeblueprint.co.uk>
Date2015-08-05 11:30 +0200
Message-ID<pU3bm-5xL-29@gated-at.bofh.it>
In reply to#1200110
On Tue, 04 Aug, at 08:41:36AM, Zhang, Jonathan Zhixiong wrote:
> 
> On 8/3/2015 9:25 PM, Borislav Petkov wrote:
> >On Mon, Aug 03, 2015 at 05:23:54PM +0100, Matt Fleming wrote:
> >>Rafael, Boris?
> >
> >The ghes.c change looks fine I guess. The whole patchset makes sense
> >now, with the arch bits extracted. So
> >
> >Acked-by: Borislav Petkov <bp@suse.de>
> Thank you Boris/Matt/Will for the support.

Jonathan, if you fixup the compiler error in PATCH 2 I'll take this
series through the EFI tree.

-- 
Matt Fleming, Intel Open Source Technology Center
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1200935

From"Zhang, Jonathan Zhixiong" <zjzhang@codeaurora.org>
Date2015-08-05 18:00 +0200
Message-ID<pU9gL-5S2-29@gated-at.bofh.it>
In reply to#1200578
On 8/5/2015 2:21 AM, Matt Fleming wrote:
> On Tue, 04 Aug, at 08:41:36AM, Zhang, Jonathan Zhixiong wrote:
>>
>> On 8/3/2015 9:25 PM, Borislav Petkov wrote:
>>> On Mon, Aug 03, 2015 at 05:23:54PM +0100, Matt Fleming wrote:
>>>> Rafael, Boris?
>>>
>>> The ghes.c change looks fine I guess. The whole patchset makes sense
>>> now, with the arch bits extracted. So
>>>
>>> Acked-by: Borislav Petkov <bp@suse.de>
>> Thank you Boris/Matt/Will for the support.
>
> Jonathan, if you fixup the compiler error in PATCH 2 I'll take this
> series through the EFI tree.
Yes, absolutely. It has been in my mind, sorry I was tied up in the last
few days. I wonder what I may missed, I certainly do x86 build test with
versions of the patch set.

I will try today and report back.

-- 
Jonathan (Zhixiong) Zhang
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1200950

FromMatt Fleming <matt@codeblueprint.co.uk>
Date2015-08-05 18:20 +0200
Message-ID<pU9A5-6u8-1@gated-at.bofh.it>
In reply to#1200935
On Wed, 05 Aug, at 05:10:00PM, Matt Fleming wrote:
> On Wed, 05 Aug, at 08:58:52AM, Zhang, Jonathan Zhixiong wrote:
> > 
> > Yes, absolutely. It has been in my mind, sorry I was tied up in the last
> > few days. I wonder what I may missed, I certainly do x86 build test with
> > versions of the patch set.
> > 
> > I will try today and report back.
> 
> No problem.
> 
> I'd suggest moving the implementation into include/acpi/apei.h and
> leaving it as static inline instead of implementing it out of line in
> arch/x86/kernel/acpi/apei.c.

Yeah, I didn't mean include/acpi/apei.h but the architecture-specific
equivalent.

-- 
Matt Fleming, Intel Open Source Technology Center
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1201733

From"Zhang, Jonathan Zhixiong" <zjzhang@codeaurora.org>
Date2015-08-06 15:50 +0200
Message-ID<pUtIt-271-1@gated-at.bofh.it>
In reply to#1200950

On 8/5/2015 9:13 AM, Matt Fleming wrote:
> On Wed, 05 Aug, at 05:10:00PM, Matt Fleming wrote:
>> On Wed, 05 Aug, at 08:58:52AM, Zhang, Jonathan Zhixiong wrote:
>>>
>>> Yes, absolutely. It has been in my mind, sorry I was tied up in the last
>>> few days. I wonder what I may missed, I certainly do x86 build test with
>>> versions of the patch set.
>>>
>>> I will try today and report back.
>>
>> No problem.
>>
>> I'd suggest moving the implementation into include/acpi/apei.h and
>> leaving it as static inline instead of implementing it out of line in
>> arch/x86/kernel/acpi/apei.c.
>
> Yeah, I didn't mean include/acpi/apei.h but the architecture-specific
> equivalent.
>
Thank you Matt. It builds well on x86 with default configuration, but
not if APEI feature is turned on. This is my fault.
I just sent a new version of the patch set out.

-- 
Jonathan (Zhixiong) Zhang
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1200951

FromMatt Fleming <matt@codeblueprint.co.uk>
Date2015-08-05 18:20 +0200
Message-ID<pU9A5-6u8-3@gated-at.bofh.it>
In reply to#1200935
On Wed, 05 Aug, at 08:58:52AM, Zhang, Jonathan Zhixiong wrote:
> 
> Yes, absolutely. It has been in my mind, sorry I was tied up in the last
> few days. I wonder what I may missed, I certainly do x86 build test with
> versions of the patch set.
> 
> I will try today and report back.

No problem.

I'd suggest moving the implementation into include/acpi/apei.h and
leaving it as static inline instead of implementing it out of line in
arch/x86/kernel/acpi/apei.c.

-- 
Matt Fleming, Intel Open Source Technology Center
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web