From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matt Fleming Date: Fri, 11 Jan 2013 13:52:38 +0000 Subject: Re: [patch] x86, efi: precedence error in setup_efi_pci() Message-Id: <1357912358.8203.316.camel@mfleming-mobl1.ger.corp.intel.com> List-Id: References: <20121207061649.GA18220@elgon.mountain> <50C2AE91.9060800@zytor.com> <20121210084426.GQ22569@mwanda> In-Reply-To: <20121210084426.GQ22569@mwanda> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Dan Carpenter Cc: "H. Peter Anvin" , Matthew Garrett , Thomas Gleixner , Ingo Molnar , x86-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, linux-efi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, kernel-janitors-u79uwXL29TY76Z2rM5mHXA@public.gmane.org On Mon, 2012-12-10 at 11:44 +0300, Dan Carpenter wrote: > On Fri, Dec 07, 2012 at 07:05:53PM -0800, H. Peter Anvin wrote: > > On 12/06/2012 10:16 PM, Dan Carpenter wrote: > > >Negate has higher precedence than bitwise AND so the condition is always > > >false. > > > > > >Signed-off-by: Dan Carpenter > > > > > >diff --git a/arch/x86/boot/compressed/eboot.c b/arch/x86/boot/compressed/eboot.c > > >index b1942e2..18e329c 100644 > > >--- a/arch/x86/boot/compressed/eboot.c > > >+++ b/arch/x86/boot/compressed/eboot.c > > >@@ -302,7 +302,7 @@ static efi_status_t setup_efi_pci(struct boot_params *params) > > > if (status != EFI_SUCCESS) > > > continue; > > > > > >- if (!attributes & EFI_PCI_IO_ATTRIBUTE_EMBEDDED_ROM) > > >+ if (!(attributes & EFI_PCI_IO_ATTRIBUTE_EMBEDDED_ROM)) > > > continue; > > > > > > if (!pci->romimage || !pci->romsize) > > > > > > > What is the baseline for this patch? > > > > Sorry, this is a linux-next bug. Introduced in: > > commit dd5fc854de5fd37adfcef8a366cd21a55aa01d3d > Author: Matthew Garrett > Date: Wed Dec 5 14:33:26 2012 -0700 > > EFI: Stash ROMs if they're not in the PCI BAR Thanks Dan, I've picked this up.