On Wed, Sep 25, 2013 at 5:01 AM, Matt Fleming <matt@xxxxxxxxxxxxxxxxx> wrote: > On Sun, 22 Sep, at 05:24:28PM, H. Peter Anvin wrote: >> On 09/22/2013 04:07 PM, Roy Franz wrote: >> > On Sun, Sep 22, 2013 at 3:54 PM, H. Peter Anvin <hpa@xxxxxxxxx> wrote: >> >> Sorry this version is broken and doesn't even compile due to remaining options_size references. >> > >> > I compiled and tested this series on both x86_64 (using OVMF) and on >> > the ARM simulator. I just doubled checked >> > my kernel .config to verify this was not being omitted and I'm pretty >> > sure this doesn't have any compilation problems. >> > I did make a few changes to get the untested version you sent out to >> > compile, but they all seemed to be straightforward typo type fixes. >> > I'll gladly address any defects in this patch, but I don't see an >> > compilation problems. >> > >> >> Ah yes, I see now... you fixed up the compile problem but did so >> incorrectly. > > Folks, I'm gonna drop this patch for now. Feel free to resend it once > everyone's happy with it. There's plenty of time to get this patch > applied, it just doesn't make sense to hold up the rest of the series. I'll get an updated (and independent) version out in the next couple days that addresses HPA's feedback. Thanks, Roy > > -- > Matt Fleming, Intel Open Source Technology Center -- To unsubscribe from this list: send the line "unsubscribe linux-efi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
