[Xen-merge] questions (resend)

["Jan Beulich" <JBeulich@xxxxxxxxxx>]

Since they remained un-answered so far, I'm resending the questions I
had embedded in the proposal for i386's include file merges. Without
answers I cannot properly deal with the respective files.

include/asm-i386/mach-xen/asm/fixmap.h
- Can any of the fixmaps actually live at machine addresses beyond 4G?
- Can't FIX_ACPI_RSDP_PAGE be replaced by use of FIX_ISAMAP + n?

include/asm-i386/mach-xen/asm/io.h
- whether merging is appropriate needs to be discussed; an alternative
  would be to break out the *_to_* translations (into a new
  include/asm-i386/mach-*/mach_xlat.h)
- What is __UNSAFE_IO__?

include/asm-i386/mach-xen/asm/pgtable-3level.h
- BUG()/panic() in pfn_pmd() needs attention from whoever put it there

include/asm-i386/mach-xen/asm/pgtable-?level-defs.h
- possibly replace HAVE_SHARED_KERNEL_PMD by
CONFIG_XEN/CONFIG_X86_XEN?

include/asm-i386/mach-xen/asm/setup.h
- What is the 'unsigned long long' override for PFN_PHYS good for (and
  if for anything, why isn't standard Linux in need of this in PAE
  mode)?

include/asm-i386/mach-xen/asm/synch_bitops.h
- don't use ; to separate lock prefix (only / or blank are proper
  syntax)
- replace "q" constraint on word/long/quad cmpxchg with "r" (mainline
  change)
- include/asm-i386/mach-xen/asm/synch_bitops.h ->
  include/asm-x86_64/mach-xen/asm/ (copy, minor cleanup to remove
  32-bit conditional, maybe use 64-bit ops where useful)
- include/asm-i386/mach-xen/asm/synch_bitops.h ->
  include/asm-i386/synch_bitops.h (minor cleanup to remove 64-bit
  conditional)

Thanks, Jan

_______________________________________________
Xen-merge mailing list
Xen-merge@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-merge