linux-kernel - Re: [Suggestion] arch: arm64: xen: "ln -s" the paravirt.h from arm.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Tue, 25 Jun 2013 16:54:53 +0100
From:	Stefano Stabellini <stefano.stabellini@...citrix.com>
To:	Catalin Marinas <catalin.marinas@....com>
CC:	Stefano Stabellini <stefano.stabellini@...citrix.com>,
	Chen Gang <gang.chen@...anux.com>,
	Will Deacon <Will.Deacon@....com>,
	Tony Lindgren <tony@...mide.com>,
	"olof@...om.net" <olof@...om.net>,
	Santosh Shilimkar <santosh.shilimkar@...com>,
	"linux-arm-kernel@...ts.infradead.org" 
	<linux-arm-kernel@...ts.infradead.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Linux-Arch <linux-arch@...r.kernel.org>
Subject: Re: [Suggestion] arch: arm64: xen: "ln -s" the paravirt.h from
 arm.

On Tue, 25 Jun 2013, Catalin Marinas wrote:
> On Mon, Jun 24, 2013 at 05:50:46PM +0100, Stefano Stabellini wrote:
> > On Mon, 24 Jun 2013, Stefano Stabellini wrote:
> > > On Mon, 24 Jun 2013, Chen Gang wrote:
> > > > Hello Maintainers:
> > > > 
> > > > if 'CONFIG_XEN'
> > > > 
> > > >   CC      arch/arm64/xen/../../arm/xen/enlighten.o
> > > > arch/arm64/xen/../../arm/xen/enlighten.c:19:26: fatal error: asm/paravirt.h: No such file or directory
> > > > 
> > > > The related .config file for next-20130624 is in attachment.
> > > > 
> > > > 
> > > > If "ln -s ../../../arm/include/asm/paravirt.h paravirt.h", it can pass
> > > > compiling, but I do not know how to make a patch for it ("ln -s ..."),
> > > > Do we have another more suitable ways for it (or another fixing ways) ?
> > > > 
> > > > Welcome any suggestions or completions.
> > > 
> > > The problem is caused by:
> > > 
> > > commit 3a885582a366caf868b0782041c44854ff4c3568
> > > Author: Stefano Stabellini <stefano.stabellini@...citrix.com>
> > > Date:   Wed May 29 10:56:34 2013 +0000
> > > 
> > >     xen/arm: account for stolen ticks
> > > 
> > > that is in my tree for linux-next (even though I have not received any
> > > replies from the ARM maintainers so I don't know when and if it is going
> > > to go upstream).
> > > 
> > > I think that the best thing to do would be to add a couple of ifdef
> > > CONFIG_PARAVIRT in arch/arm/xen/enlighten.c. I'll add them to my tree.
> > 
> > Actually now that we have XEN support under arm64, we can just introduce the
> > same pv_time_op struct and paravirt header that this patch
> > 
> > http://marc.info/?l=linux-arm-kernel&m=136992435301890&w=2
> > 
> > is introducing under arm.
> 
> I'm not sure whether I replied to your Xen pull request but the patches
> have been merged into my 'upstream' branch on git.kernel.org and they'll
> be going in for 3.11-rc1.

Great, thanks!


> > diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig
> > index 19c1cde..47bd26b 100644
> > --- a/arch/arm64/Kconfig
> > +++ b/arch/arm64/Kconfig
> > @@ -207,6 +207,25 @@ config FORCE_MAX_ZONEORDER
> >  	default "14" if (ARM64_64K_PAGES && TRANSPARENT_HUGEPAGE)
> >  	default "11"
> >  
> > +config PARAVIRT
> > +	bool "Enable paravirtualization code"
> > +	---help---
> > +	  This changes the kernel so it can modify itself when it is run
> > +	  under a hypervisor, potentially improving performance significantly
> > +	  over full virtualization.
> > +
> > +config PARAVIRT_TIME_ACCOUNTING
> > +	bool "Paravirtual steal time accounting"
> > +	select PARAVIRT
> > +	default n
> 
> No need for "default n".
> 
> If you want, I can apply this patch on top (or send me another pull
> request but don't rebase your original Xen branch which I merged).

Thanks, but unfortunately the corresponding arm part of this patch
(linked above) didn't manage to get enough reviews and acks.
I might as well send another iteration of "CONFIG_PARAVIRT and stolen
ticks accounting" with this change in it.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/