[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH v2 01/15] xen/arm: register mmio handler at runtime
On Tue, Apr 8, 2014 at 3:47 PM, Julien Grall <julien.grall@xxxxxxxxxx> wrote: > On 04/08/2014 05:47 AM, Vijay Kilari wrote: >>>>> >>>>>> >>>>>> -const struct mmio_handler vgic_distr_mmio_handler = { >>>>>> +static struct mmio_handler vgic_distr_mmio_handler = { >>>>> >>>>> Why did you remove the const? >>>> >>>> I think, compiler is allowing forward declaration with const >>> >>> You meant doesn't allow? If so, please move the whole structure. I'd >>> prefer code movement rather than removing a const because of compiler issue. >>> >> Yes, compiler does not allow. I will move the code > > This is from the specification... if you declare something const you > have to assign the value directly. > Also we cannot keep it as const for the reason that register_mmio_handler fails to assign this const pointer to handler list. Anyway we will get rid of this structure completely after implementing check handler where read & write handler are passed as parameter to register_mmio_handler() call. >>>> >>>>> >>>>> [..] >>>>> >>>>>> diff --git a/xen/include/asm-arm/domain.h b/xen/include/asm-arm/domain.h >>>>>> index 50b9b54..23dac85 100644 >>>>>> --- a/xen/include/asm-arm/domain.h >>>>>> +++ b/xen/include/asm-arm/domain.h >>>>>> @@ -116,6 +116,7 @@ struct arch_domain >>>>>> struct hvm_domain hvm_domain; >>>>>> xen_pfn_t *grant_table_gpfn; >>>>>> >>>>>> + struct io_handler *io_handlers; >>>>> >>>>> Why do you need a pointer here? I think can can directly use >>>>> struct io_handler iohandlers. >>>>> >>>> I just adds to increase the size of this arch_domain struct. >>>> so allocated memory at runtime. >>> >>> Do you hit the page size? >> Yes, I have faced page size exhausted issue which I have reported earlier > > Did you still have the issue when you moved out GICv stuff outside? Now I am not getting this issue. But I feel we are on the edge. So it is always better to allocate runtime. > > Regards, > > -- > Julien Grall _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |