Re: [Xen-devel] [PATCH v6 09/24] xsplice: Implement payload loading

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 08.04.16 at 23:10, <konrad.wilk@xxxxxxxxxx> wrote:
>> > +int arch_xsplice_perform_rela(struct xsplice_elf *elf,
>> > +                              const struct xsplice_elf_sec *base,
>> > +                              const struct xsplice_elf_sec *rela)
>> > +{
>> > +    const Elf_RelA *r;
>> > +    unsigned int symndx, i;
>> > +    uint64_t val;
>> > +    uint8_t *dest;
>> > +
>> > +    if ( !rela->sec->sh_entsize || !rela->sec->sh_size ||
>> > +         rela->sec->sh_entsize != sizeof(Elf_RelA) )
>> > +    {
>> > +        dprintk(XENLOG_DEBUG, XSPLICE "%s: Section relative header is 
>> > corrupted!\n",
>> > +                elf->name);
>> 
>> XENLOG_ERR surely? (and the other examples).
> 
> Yes! I modified all of those that return an error. One of them I made
> an printk (the one about more than 64 sections).

Why would that be any worse than other check failures? I think
these log messages should all be issued consistently.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel