Re: [PATCH] pseries: asm/pci-bridge.h CONFIG_ minor cleanup

On Tue, 2007-05-22 at 12:00 -0500, Linas Vepstas wrote:

On Tue, May 22, 2007 at 10:40:59AM +1000, Michael Ellerman wrote:

quoted

On Mon, 2007-05-21 at 18:18 -0500, Linas Vepstas wrote:

quoted

Use the correct CONFIG_ option to mark off the EEH bits.

You're making the struct 4 bytes larger by creating a hole here :(

Hmm. Perhaps I could shuffle one of the ints down ...  

I figured there might be some vague cache-line benefits to
getting the eeh stuff out of the way. 

Yeah who knows. The hole doesn't really matter unless the struct gets
bigger, having the EEH stuff at the bottom makes it read better which is
probably more important.

quoted

quoted

 	u32	config_space[16];	/* saved PCI config space */
+#endif

It looks correct, but I think it's worth mentioning in the changelog
that config_space was previously unconditionally defined, but is now
within CONFIG_EEH.

I could just tell that someone would remark on this; why I didn't
mention it, I don't know.

Me neither :)

cheers

-- 
Michael Ellerman
OzLabs, IBM Australia Development Lab

wwweb: http://michael.ellerman.id.au
phone: +61 2 6212 1183 (tie line 70 21183)

We do not inherit the earth from our ancestors,
we borrow it from our children. - S.M.A.R.T Person