Re: [Qemu-devel] [PATCH v2 1/6] Add cfgend parameter for ARM CPU selection.

2017-01-19 Thread Peter Maydell
On 19 January 2017 at 18:02, Julian Brown wrote: > We're using integratorcp That board model is extremely ancient and unloved, by the way. If we ever make good on the suggestion from last year to start deprecating and eventually removing unmaintained board models, it's probably going to be on the

Re: [Qemu-devel] [PATCH v2 1/6] Add cfgend parameter for ARM CPU selection.

2017-01-19 Thread Julian Brown
On Thu, 19 Jan 2017 15:12:49 + Peter Maydell wrote: > I'm actually looking at a patch at the moment that attempts > to fix this by adding a cpu_generic_init_unrealized(), > which does everything that cpu_generic_init() does except > the final "set the realized prop to true" step, so that you

Re: [Qemu-devel] [PATCH v2 1/6] Add cfgend parameter for ARM CPU selection.

2017-01-19 Thread Peter Maydell
On 18 January 2017 at 16:52, Eduardo Habkost wrote: > On Tue, Jan 17, 2017 at 11:52:24AM +, Peter Maydell wrote: >> Oops, I'd forgotten there was board-specific code for handling >> cpu feature flags. I think we want to change the other boards >> to do it like the virt board does, but maybe I'

Re: [Qemu-devel] [PATCH v2 1/6] Add cfgend parameter for ARM CPU selection.

2017-01-18 Thread Eduardo Habkost
On Tue, Jan 17, 2017 at 11:52:24AM +, Peter Maydell wrote: > On 12 January 2017 at 14:28, Julian Brown wrote: > > On Wed, 11 Jan 2017 12:35:52 + > > Julian Brown wrote: > > > >> I'm a little confused, I think -- these changes seemed to be necessary > >> to allow the parsing of the command

Re: [Qemu-devel] [PATCH v2 1/6] Add cfgend parameter for ARM CPU selection.

2017-01-17 Thread Peter Maydell
On 12 January 2017 at 14:28, Julian Brown wrote: > On Wed, 11 Jan 2017 12:35:52 + > Julian Brown wrote: > >> I'm a little confused, I think -- these changes seemed to be necessary >> to allow the parsing of the command-line syntax you suggested earlier >> (-mcpu=foo,cfgend=bar): >> [...] >> C

Re: [Qemu-devel] [PATCH v2 1/6] Add cfgend parameter for ARM CPU selection.

2017-01-12 Thread Julian Brown
On Wed, 11 Jan 2017 12:35:52 + Julian Brown wrote: > I'm a little confused, I think -- these changes seemed to be necessary > to allow the parsing of the command-line syntax you suggested earlier > (-mcpu=foo,cfgend=bar): > [...] > Can the existing properties be set like that via the command

Re: [Qemu-devel] [PATCH v2 1/6] Add cfgend parameter for ARM CPU selection.

2017-01-11 Thread Julian Brown
On Thu, 5 Jan 2017 17:17:13 + Peter Maydell wrote: > > +qdev_property_add_static(DEVICE(obj), &arm_cpu_cfgend_property, > > + &error_abort); > > + > > +qdev_prop_set_globals(DEVICE(obj)); > > + > > +if (object_property_get_bool(obj, "cfgend", NULL)) { >

Re: [Qemu-devel] [PATCH v2 1/6] Add cfgend parameter for ARM CPU selection.

2017-01-05 Thread Peter Maydell
On 7 December 2016 at 14:48, Julian Brown wrote: > The new "cfgend" parameter can be given after the CPU name, > e.g. "-cpu=arm926,cfgend=yes", and is used to control the reset value > of the SCTLR register in an architecture-version appropriate way (in a > similar way to the CFGEND input on physi

[Qemu-devel] [PATCH v2 1/6] Add cfgend parameter for ARM CPU selection.

2016-12-07 Thread Julian Brown
The new "cfgend" parameter can be given after the CPU name, e.g. "-cpu=arm926,cfgend=yes", and is used to control the reset value of the SCTLR register in an architecture-version appropriate way (in a similar way to the CFGEND input on physical cores). Using the option for CPUs implementing the AR