Re: [Xen-devel] [PATCH v2] x86/xen: init %gs very early to avoid page faults with stack protector

2018-02-02 Thread Chris Patterson 
On Fri, Feb 2, 2018 at 12:56 AM, Juergen Gross <jgr...@suse.com> wrote:
> On 02/02/18 01:36, Chris Patterson wrote:
>> Works great, tested it and it fixes booting Linux v4.15 kernel for me :)
>
> Can I add your "Tested-by:" to the patch when committing it?
>
>

Sure thing, have a great day.

Re: [Xen-devel] [PATCH v2] x86/xen: init %gs very early to avoid page faults with stack protector

2018-02-02 Thread Chris Patterson 
On Fri, Feb 2, 2018 at 12:56 AM, Juergen Gross  wrote:
> On 02/02/18 01:36, Chris Patterson wrote:
>> Works great, tested it and it fixes booting Linux v4.15 kernel for me :)
>
> Can I add your "Tested-by:" to the patch when committing it?
>
>

Sure thing, have a great day.

Re: [Xen-devel] [PATCH v2] x86/xen: init %gs very early to avoid page faults with stack protector

2018-02-01 Thread Chris Patterson 
Works great, tested it and it fixes booting Linux v4.15 kernel for me :)

Cheers!

On Thu, Feb 1, 2018 at 3:17 PM, Boris Ostrovsky
 wrote:
> On 02/01/2018 07:40 AM, Juergen Gross wrote:
>> When running as Xen pv guest %gs is initialized some time after
>> C code is started. Depending on stack protector usage this might be
>> too late, resulting in page faults.
>>
>> So setup %gs and MSR_GS_BASE in assembly code already.
>>
>> Cc: sta...@vger.kernel.org
>> Signed-off-by: Juergen Gross 
>
> Reviewed-by: Boris Ostrovsky 
>
>
>
> ___
> Xen-devel mailing list
> xen-de...@lists.xenproject.org
> https://lists.xenproject.org/mailman/listinfo/xen-devel

Re: [Xen-devel] [PATCH v2] x86/xen: init %gs very early to avoid page faults with stack protector

2018-02-01 Thread Chris Patterson 
Works great, tested it and it fixes booting Linux v4.15 kernel for me :)

Cheers!

On Thu, Feb 1, 2018 at 3:17 PM, Boris Ostrovsky
 wrote:
> On 02/01/2018 07:40 AM, Juergen Gross wrote:
>> When running as Xen pv guest %gs is initialized some time after
>> C code is started. Depending on stack protector usage this might be
>> too late, resulting in page faults.
>>
>> So setup %gs and MSR_GS_BASE in assembly code already.
>>
>> Cc: sta...@vger.kernel.org
>> Signed-off-by: Juergen Gross 
>
> Reviewed-by: Boris Ostrovsky 
>
>
>
> ___
> Xen-devel mailing list
> xen-de...@lists.xenproject.org
> https://lists.xenproject.org/mailman/listinfo/xen-devel

Re: [Xen-devel] [PATCH] kernel-parameters: document earlycon=xenboot

2016-04-05 Thread Chris Patterson 
On Tue, Apr 5, 2016 at 12:42 PM, Konrad Rzeszutek Wilk
<konrad.w...@oracle.com> wrote:
> On Tue, Apr 05, 2016 at 11:43:36AM -0400, Chris Patterson wrote:
>> On Tue, Apr 5, 2016 at 9:49 AM, Konrad Rzeszutek Wilk
>> <konrad.w...@oracle.com> wrote:
>> > On Mon, Apr 04, 2016 at 10:48:11PM -0400, Chris Patterson wrote:
>> >> Add earlycon=xenboot option to Documentation/kernel-parameters.txt.
>> >>
>> >
>> > But it is not true.
>> >
>> > I tried this on x86: "earlycon=xenboot console=tty0"
>> >
>>
>> I can investigate further, but hopefully Stefano can chime in more
>> definitively on what is supported.
>>
>> I used "earlycon=xenboot console=hvc0" with success for dom0 on my
>> ARM64 platform.
>
> Perhaps the documentation should also have [ARM], like some of the
> other ones?

Absolutely.  I will test some combinations and be explicit about what
should work in v2.  Thanks for the feedback.

Cheers,
-Chris

Re: [Xen-devel] [PATCH] kernel-parameters: document earlycon=xenboot

2016-04-05 Thread Chris Patterson 
On Tue, Apr 5, 2016 at 12:42 PM, Konrad Rzeszutek Wilk
 wrote:
> On Tue, Apr 05, 2016 at 11:43:36AM -0400, Chris Patterson wrote:
>> On Tue, Apr 5, 2016 at 9:49 AM, Konrad Rzeszutek Wilk
>>  wrote:
>> > On Mon, Apr 04, 2016 at 10:48:11PM -0400, Chris Patterson wrote:
>> >> Add earlycon=xenboot option to Documentation/kernel-parameters.txt.
>> >>
>> >
>> > But it is not true.
>> >
>> > I tried this on x86: "earlycon=xenboot console=tty0"
>> >
>>
>> I can investigate further, but hopefully Stefano can chime in more
>> definitively on what is supported.
>>
>> I used "earlycon=xenboot console=hvc0" with success for dom0 on my
>> ARM64 platform.
>
> Perhaps the documentation should also have [ARM], like some of the
> other ones?

Absolutely.  I will test some combinations and be explicit about what
should work in v2.  Thanks for the feedback.

Cheers,
-Chris

Re: [Xen-devel] [PATCH] kernel-parameters: document earlycon=xenboot

2016-04-05 Thread Chris Patterson 
On Tue, Apr 5, 2016 at 9:49 AM, Konrad Rzeszutek Wilk
<konrad.w...@oracle.com> wrote:
> On Mon, Apr 04, 2016 at 10:48:11PM -0400, Chris Patterson wrote:
>> Add earlycon=xenboot option to Documentation/kernel-parameters.txt.
>>
>
> But it is not true.
>
> I tried this on x86: "earlycon=xenboot console=tty0"
>

I can investigate further, but hopefully Stefano can chime in more
definitively on what is supported.

I used "earlycon=xenboot console=hvc0" with success for dom0 on my
ARM64 platform.

Re: [Xen-devel] [PATCH] kernel-parameters: document earlycon=xenboot

2016-04-05 Thread Chris Patterson 
On Tue, Apr 5, 2016 at 9:49 AM, Konrad Rzeszutek Wilk
 wrote:
> On Mon, Apr 04, 2016 at 10:48:11PM -0400, Chris Patterson wrote:
>> Add earlycon=xenboot option to Documentation/kernel-parameters.txt.
>>
>
> But it is not true.
>
> I tried this on x86: "earlycon=xenboot console=tty0"
>

I can investigate further, but hopefully Stefano can chime in more
definitively on what is supported.

I used "earlycon=xenboot console=hvc0" with success for dom0 on my
ARM64 platform.

[PATCH] kernel-parameters: document earlycon=xenboot

2016-04-04 Thread Chris Patterson 
Add earlycon=xenboot option to Documentation/kernel-parameters.txt.

Signed-off-by: Chris Patterson <cjp...@gmail.com>
---
 Documentation/kernel-parameters.txt | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/kernel-parameters.txt 
b/Documentation/kernel-parameters.txt
index ecc74fa..e01ec39 100644
--- a/Documentation/kernel-parameters.txt
+++ b/Documentation/kernel-parameters.txt
@@ -1078,6 +1078,8 @@ bytes respectively. Such letter suffixes can also be 
entirely omitted.
address. The serial port must already be setup
and configured. Options are not yet supported.
 
+   xenboot Use Xen hypervisor console for early console.
+
earlyprintk=[X86,SH,BLACKFIN,ARM,M68k]
earlyprintk=vga
earlyprintk=efi
-- 
2.1.4

[PATCH] kernel-parameters: document earlycon=xenboot

2016-04-04 Thread Chris Patterson 
Add earlycon=xenboot option to Documentation/kernel-parameters.txt.

Signed-off-by: Chris Patterson 
---
 Documentation/kernel-parameters.txt | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/kernel-parameters.txt 
b/Documentation/kernel-parameters.txt
index ecc74fa..e01ec39 100644
--- a/Documentation/kernel-parameters.txt
+++ b/Documentation/kernel-parameters.txt
@@ -1078,6 +1078,8 @@ bytes respectively. Such letter suffixes can also be 
entirely omitted.
address. The serial port must already be setup
and configured. Options are not yet supported.
 
+   xenboot Use Xen hypervisor console for early console.
+
earlyprintk=[X86,SH,BLACKFIN,ARM,M68k]
earlyprintk=vga
earlyprintk=efi
-- 
2.1.4