Re: [kvmarm] [PATCH] bootwrapper: CPU hotplug aware boot protocol

2013-01-07 Thread Jon Medhurst (Tixy)
On Mon, 2013-01-07 at 13:47 +, Dave Martin wrote: > (Minor point: you can merge the "b start" with the first entry of > the vectors, because this slot is never used for any other purpose in > any vector table. The code works anyway, though.) I considered that when I wrote the patch but though

Re: [kvmarm] [PATCH] bootwrapper: CPU hotplug aware boot protocol

2013-01-07 Thread Jon Medhurst (Tixy)
On Mon, 2013-01-07 at 13:59 +, Peter Maydell wrote: > On 7 January 2013 13:47, Dave Martin wrote: > > On Thu, Dec 20, 2012 at 12:23:48PM +, Jon Medhurst (Tixy) wrote: > >> Does anyone have any further outstanding concerns or comments about my > >> proposed patch? > > > > Since this doesn't

Re: [kvmarm] [PATCH] bootwrapper: CPU hotplug aware boot protocol

2013-01-07 Thread Peter Maydell
On 7 January 2013 13:47, Dave Martin wrote: > On Thu, Dec 20, 2012 at 12:23:48PM +, Jon Medhurst (Tixy) wrote: >> Does anyone have any further outstanding concerns or comments about my >> proposed patch? > > Since this doesn't seem to be merged yet, I'll just comment that this > all looks sens

Re: [kvmarm] [PATCH] bootwrapper: CPU hotplug aware boot protocol

2013-01-07 Thread Dave Martin
On Thu, Dec 20, 2012 at 12:23:48PM +, Jon Medhurst (Tixy) wrote: > On Fri, 2012-12-14 at 15:54 +, Jon Medhurst (Tixy) wrote: > > On Fri, 2012-12-14 at 10:10 -0500, Christoffer Dall wrote: > > > On Fri, Dec 14, 2012 at 3:49 AM, Jon Medhurst (Tixy) > > > wrote: > > > > On Thu, 2012-12-13 at

Re: [kvmarm] [PATCH] bootwrapper: CPU hotplug aware boot protocol

2012-12-20 Thread Jon Medhurst (Tixy)
On Fri, 2012-12-14 at 15:54 +, Jon Medhurst (Tixy) wrote: > On Fri, 2012-12-14 at 10:10 -0500, Christoffer Dall wrote: > > On Fri, Dec 14, 2012 at 3:49 AM, Jon Medhurst (Tixy) > > wrote: > > > On Thu, 2012-12-13 at 13:52 -0800, Christoffer Dall wrote: > > >> On Friday, December 7, 2012, Jon M

Re: [kvmarm] [PATCH] bootwrapper: CPU hotplug aware boot protocol

2012-12-17 Thread Christoffer Dall
On Fri, Dec 14, 2012 at 3:49 AM, Jon Medhurst (Tixy) wrote: > On Thu, 2012-12-13 at 13:52 -0800, Christoffer Dall wrote: >> On Friday, December 7, 2012, Jon Medhurst (Tixy) wrote: >> >> > To enable CPU hotplug the need to provide some boot code at the >> reset >> > vector and which survives after

Re: [kvmarm] [PATCH] bootwrapper: CPU hotplug aware boot protocol

2012-12-14 Thread Nicolas Pitre
On Fri, 14 Dec 2012, Marc Zyngier wrote: > On 14/12/12 15:54, Jon Medhurst (Tixy) wrote: > > I must admit, my patch is based on someone elses work, and I've not seen > > documentation which says that NOR flash can be written to simply by the > > application being loaded by the models having code l

Re: [kvmarm] [PATCH] bootwrapper: CPU hotplug aware boot protocol

2012-12-14 Thread Jon Medhurst (Tixy)
On Fri, 2012-12-14 at 16:00 +, Marc Zyngier wrote: > On 14/12/12 15:54, Jon Medhurst (Tixy) wrote: > > On Fri, 2012-12-14 at 10:10 -0500, Christoffer Dall wrote: > >> On Fri, Dec 14, 2012 at 3:49 AM, Jon Medhurst (Tixy) > >> wrote: > >>> On Thu, 2012-12-13 at 13:52 -0800, Christoffer Dall wro

Re: [kvmarm] [PATCH] bootwrapper: CPU hotplug aware boot protocol

2012-12-14 Thread Marc Zyngier
On 14/12/12 15:54, Jon Medhurst (Tixy) wrote: > On Fri, 2012-12-14 at 10:10 -0500, Christoffer Dall wrote: >> On Fri, Dec 14, 2012 at 3:49 AM, Jon Medhurst (Tixy) wrote: >>> On Thu, 2012-12-13 at 13:52 -0800, Christoffer Dall wrote: On Friday, December 7, 2012, Jon Medhurst (Tixy) wrote:

Re: [kvmarm] [PATCH] bootwrapper: CPU hotplug aware boot protocol

2012-12-14 Thread Alexander Spyridakis
On 14 December 2012 16:10, Christoffer Dall wrote: > What I was trying to figure out was why you can get rid of the "a > location less likely to be overridden" and why you're sure this memory > stays unmodified - I thought that was because you were loading the > code in the beginning of the physic

Re: [kvmarm] [PATCH] bootwrapper: CPU hotplug aware boot protocol

2012-12-14 Thread Jon Medhurst (Tixy)
On Fri, 2012-12-14 at 10:10 -0500, Christoffer Dall wrote: > On Fri, Dec 14, 2012 at 3:49 AM, Jon Medhurst (Tixy) wrote: > > On Thu, 2012-12-13 at 13:52 -0800, Christoffer Dall wrote: > >> On Friday, December 7, 2012, Jon Medhurst (Tixy) wrote: > >> > >> > To enable CPU hotplug the need to provide

Re: [kvmarm] [PATCH] bootwrapper: CPU hotplug aware boot protocol

2012-12-14 Thread Christoffer Dall
On Friday, December 7, 2012, Jon Medhurst (Tixy) wrote: > To enable CPU hotplug the need to provide some boot code at the reset > vector and which survives after the kernel has booted without being > overwritten. We achieve this by the getting the linker script to place > the code in boot.S at add

Re: [kvmarm] [PATCH] bootwrapper: CPU hotplug aware boot protocol

2012-12-14 Thread Jon Medhurst (Tixy)
On Thu, 2012-12-13 at 13:52 -0800, Christoffer Dall wrote: > On Friday, December 7, 2012, Jon Medhurst (Tixy) wrote: > > > To enable CPU hotplug the need to provide some boot code at the > reset > > vector and which survives after the kernel has booted without being > > overwritten. We achieve thi

Re: [kvmarm] [PATCH] bootwrapper: CPU hotplug aware boot protocol

2012-12-11 Thread Christoffer Dall
Hi, I will get back to you on this one at the end of the week - hang in there:) -Christoffer On Tue, Dec 11, 2012 at 4:52 AM, Jon Medhurst (Tixy) wrote: > Adding some named people to the 'To' list to try and elicit review > comments... :-) > > On Fri, 2012-12-07 at 11:35 +, Jon Medhurst (Ti