Re: [Xen-devel] Xen 4.7.0 boot PANIC on kernel 4.7.0-4 + UEFI ?

[lists@xxxxxxxxxxxx]

On Fri, Jul 29, 2016, at 09:03 AM, Konrad Rzeszutek Wilk wrote:
> It may very well be added.
> 
> But having extra test-confirmation is always good.

looking at the patch

        diff --git a/include/linux/efi.h b/include/linux/efi.h
        index c2db3ca..f196dd0 100644
        --- a/include/linux/efi.h
        +++ b/include/linux/efi.h
        @@ -1005,7 +1005,7 @@  extern int efi_memattr_apply_permissions(struct 
mm_struct *mm,
         /* Iterate through an efi_memory_map */
         #define for_each_efi_memory_desc_in_map(m, md)                         
   \
                for ((md) = (m)->map;                                           
   \
        -            (md) <= (efi_memory_desc_t *)((m)->map_end - 
(m)->desc_size); \
        +            ((void *)(md) + (m)->desc_size) <= (m)->map_end;           
   \
                     (md) = (void *)(md) + (m)->desc_size)

         /**

and in the source used in the distro build

        wget 
https://build.opensuse.org/source/Kernel:stable/kernel-default/linux-4.7.tar.xz?rev=88182fafc706b2366e0251692bd4b7e7
        tar zxf linux-4.7.tar.xz
        cd linux-4.7/include/linux/
        edit efi.h

it looks like the patch is in there already

        1005    /* Iterate through an efi_memory_map */
                        #define for_each_efi_memory_desc_in_map(m, md)          
                   \
                                for ((md) = (m)->map;                           
                   \
>>>                                  ((void *)(md) + (m)->desc_size) <= 
>>> (m)->map_end;              \
                                     (md) = (void *)(md) + (m)->desc_size)

even if it's not mentioned in the changelog.

The binary built with this source crashes as reported.

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