netfront accel: Reduce length of accel_watch workqueue name

As reported reported (see link below) the first argument to
create_workqueue is limited to 10 character on RHEL4U2.  This patch
shortens the length and makes the workqueue name a little more
descriptive.

http://lists.xensource.com/archives/html/xen-devel/2008-03/msg00417.html

Signed-off-by: Neil Turton <nturton@solarflare.com>

xen/x86: simplify synch_test_bit()

There really is no need for a redundant implementation here, just keep
the alternative name for allowing consumers to use consistent naming.

Signed-off-by: Jan Beulich <jbeulich@novell.com>

xen/i386: re-add and use pre_setup_arch_hook()

It was only during the 2.6.25 merge that I realized that there was a
difference to native code that was not only unnecessary, but even
preventing the Xen version from being better readable and closer to
native both in terms of source code and behavior:
pre_setup_arch_hook() can do everything that (or equivalent to what)
x86-64 does in head64-xen.c. Apart from that it simplifies forward
porting, since certain pieces set up here are required to be available
much earlier in newer Linux.

Signed-off-by: Jan Beulich <jbeulich@novell.com>

xen: satisfy newer modpost requirements, part 2

References to __devinit probe functions are considered valid by newer
modprobe if the containing structure is named in certain ways. Use the
_driver suffix where necessary.

Signed-off-by: Jan Beulich <jbeulich@novell.com>

xen/x86-64: slightly simplify smp-xen.c

Two Xen conditionals can be folded into one, as the code between the
two is also not used on Xen.

Signed-off-by: Jan Beulich <jbeulich@novell.com>

xen/x86-64: ia32entry.S doesn't need xen_entry.S anymore
Signed-off-by: Jan Beulich <jbeulich@novell.com>

xen/i386: change the way trampoline.S building is suppressed

Instead of using the disabled-obj-$() mechanism, preventing
trampoline.S from being built can be done simply be adding a config
option dependency.

Signed-off-by: Jan Beulich <jbeulich@novell.com>

xen pcifront: satisfy newer modpost requirements

pcifront_scan_root() calls pci_scan_bus_parented() possibly after core
kernel initialization, but that latter function is marked __devinit.
While HOTPLUG can be turned off only under EMBEDDED, 2.6.25's modpost
still (validly) catches this as an incorrect reference. Marking
pcifront_scan_root() __init_refok seems too dangerous, however, so
instead the much more streamlined pcifront_backend_changed() is being
marked so, and the rest of the offending dependencies is being marked
__devinit, and XEN_PCIDEV_FRONTEND now selects HOTPLUG to make it
independend of any changes in HOTPLUG's prompt visibility.

Signed-off-by: Jan Beulich <jbeulich@novell.com>

xen/x86: put trap initialization code/data into cpuinit space
Signed-off-by: Jan Beulich <jbeulich@novell.com>

linux/x86-64: make original definition of VGCF_IN_SYSCALL usable in assembly
Signed-off-by: Jan Beulich <jbeulich@novell.com>

xen: validate type and value of the dtor argument of SetPageForeign()

Linux 2.6.25 changes the protoype of pte_free() etc., resulting in
those functions no longer be suitable as a PageForeign destructor. I
had to find out by way of analysing a crash, but for the future it'd
be much better if the build would already indicate a problem with
this.

At the same time, also check the destructor supplied is not NULL.

Signed-off-by: Jan Beulich <jbeulich@novell.com>

xen,x86: fix warnings introduced by c/s 468

I overlooked the compiler warnings resulting from cases where the
argument of __pte_ma() is a plain zero.

Signed-off-by: Jan Beulich <jbeulich@novell.com>

xen, x86: get_phys_id hypercall interface change.
Signed-off-by: Keir Fraser <keir.fraser@citrix.com>

xenbus: properly qualify all static 'struct xenbus_device_id' instances

include/xen/xenbus.h nicely declares struct xenbus_driver's 'ids'
member as pointer to const...

Signed-off-by: Jan Beulich <jbeulich@novell.com>

xenbus: prevent warnings on unhandled enumeration values

XenbusStateReconfiguring/XenbusStateReconfigured were introduced by
c/s 437, but aren't handled in many switch statements. This c/s also
introduced a possibly un-referenced label, which also gets eliminated
here.

Signed-off-by: Jan Beulich <jbeulich@novell.com>

linux: fix pv driver build

When building with -Werror-implicit-function-declaration, the addition
of is_initial_xen_domain() checks in drivers/xen/netfront/accel.c
causes the build to fail.

Additionally, drivers/xen/netfront/netfront.c illegally (and
needlessly) includes xen/hypercall.h directly.

Signed-off-by: Jan Beulich <jbeulich@novell.com>

linux/i386-pae: fix __pte_ma()

While at present there's no use of the macro that would suffer from
this problem, this is a latent bug and should therefore be fixed (just
like __pte() in the native kernel).

Signed-off-by: Jan Beulich <jbeulich@novell.com>

linux/x86-64: remove stale MCA_bus definition

Apparently a merge oversight that pre-dates 2.6.16.

Signed-off-by: Jan Beulich <jbeulich@novell.com>

x86 xen: New vcpu_op call to get physical CPU identity.

Some AMD machines have APIC IDs that not equal to CPU IDs. In
the default Xen configuration, ACPI calls on these machines
can get confused. This shows up most noticeably when running
AMD PowerNow!. The only solution is for dom0 to get the
hypervisor's cpuid to apicid table when needed (ie, when dom0
vcpus are pinned).

Add a vcpu op to Xen to allow dom0 to query the hypervisor for
architecture dependent physical cpu information if dom0 vcpus are
pinned.

Signed-off-by: Mark Langsdorf <mark.langsdorf@amd.com>
Signed-off-by: Keir Fraser <keir.fraser@citrix.com>

xen suspend: Fix write-after-read error on fast_suspend flag.
Signed-off-by: Keir Fraser <keir.fraser@citrix.com>

Fix section type conflict in hrtimer.c

Backport from git d316c57ff6bfad9557462b9100f25c6260d2b774

Signed-off-by: Aron Griffis <aron@hp.com>

Fix section type mismatch in gameport.c

Backport from git f5ad58675149077b2046905d54fb831873288058

Signed-off-by: Aron Griffis <aron@hp.com>

Fix section type conflict in skbuff.c

Backport from git 2cd052e44329dd2b42eb958f8f346b053de6e2cd

Signed-off-by: Aron Griffis <aron@hp.com>

moduleparam: fix ia64 compile failure

Backport from git 91d35dd93e14c34539a8005183ea500f25caad02

Signed-off-by: Aron Griffis <aron@hp.com>

Fix section type mismatch in neighbor.c

Backport from git e92b43a3455d3e817c13481bb3ea3cd29d0a47f4

Signed-off-by: Aron Griffis <aron@hp.com>

Fix section type mismatch in rsxface.c

Backport from git cd5bfea278987ebfe60f3ff92a01696b17c4f978

Signed-off-by: Aron Griffis <aron@hp.com>

Revert 419:0034d9389 and 425:9dea529497.
Signed-off-by: Keir Fraser <keir.fraser@citrix.com>

merge with linux-2.6.18-xen.hg

[IA64] kexec: Unpin TLB in the hypervisor

In the case where kexec is being run for a xen dom0 or hypervisor,
this code, present in dom0, will be called from the hypervisor
after ia64_machine_kexec and before going into purgatory.

This code makes a large number of assumptions about various compile
time constants, and thus assumes that these constants are the
same for the hypervisor and dom0. Despite extensive #ifdef work this
has proved to be both fragile and incomplete.

This patch changes things around so that the unpinning work is done
by code provided by the hypervisor, reusing existing code there.
Apart from being a solution that works, its also likely
a much more maintainable solution, as as TLB changes in the hypervisor
code are made, the code paths in the hypervisor are much more likely
to be checked than this one which lies in a completely different tree.

With this change, and the corresponding change to the hypervisor,
the CPU will be running in "physical mode" when it enters this
relocate_new_kernel code. Previously the code was called with the
CPU in "virtual mode". This is still the case when the code is
called from a regular Linux kernel.

Code that switches the CPU into "physical mode" is still called.
This is primarily left in tact to ensure that the register stack
(bspstore) and stack (sp) are updated correctly. It could probably
be trimmed down a bit, but there seems little point.

When called from the hypervisor, in3 is ignored. The number
of parameters could hence be shrunk from 4 to 3, but again,
there seems little point.

This code still assumes that PAGE_SIZE and PAGE_MASK are the
same in the hypervisor and dom0. I am reluctant to fix this
as I think that it is unlikely this will change. But if it does,
the fix should be as easy as passing PAGE_SIZE as in3. I am quite
happy to create a patch if/when it is needed.

As noted above, there is a hypervisor portion of this patch
supplied separately.

Signed-off-by: Simon Horman <horms@verge.net.au>

[IA64] kexec: is for privileged guests only

This makes the KEXEC Kconfig option depend on !XEN_UNPRIVILEGED_GUEST, so
that it is not available to unprivelaged guests. Or in other words,
it is only available to non-xen linux or privileged guests.

Some minor #defines relating to kexec have also been
updated.

linux/kexec.h is only needed in contig.c if both XEN and KEXEC
are in operation.

iomem_machine_resource is only used if PROC_IOMEM_MACHINE is in effect.
This does depend on XEN, but also depends on KEXEC and IA64.

Throughout the code #if CONFIG_XEN is used to guard regions.
This is ok, because the relevant code is only active if
KEXEC is configured, and thus implicitly the code is
being compiled with XEN_PRIVILEGED_GUEST

This is in line with the use of Kconfig on x86_64
(and presumably x86_32, though I did not check)

Signed-off-by: Simon Horman <horms@verge.net.au>

[IA64] Enable XEN_SYSCTL_availheap

This enables per node memory information to be displayed

From: Duan Ronghui <ronghui.duan@intel.com>
Signed-off-by: Alex Williamson <alex.williamson@hp.com>

merge with linux-2.6.18-xen.hg

Avoid using a separate watch thread due to uninitialised watch->flags.

The xenbus_dev code isn't setup to handle the case where
XBWF_new_thread is set so there is a potetial crash if this flag is
erroneously set. Therefore initialise flags to zero by using kzalloc
rather than kmalloc.

Signed-off-by: Ian Campbell <ian.campbell@citrix.com>

fbfront: Slightly streamline handling of wheel (z-axis) mouse events.
Signed-off-by: Keir Fraser <keir.fraser@citrix.com>

Add license stanza to a few Xen header files.
Signed-off-by: Keir Fraser <keir.fraser@citrix.com>

[IA64] Fix vulnerability of privcmd_mmap

empty_zero_page can be polluted by writing to a page through
privcmd_mmap().  i.e. a user program can hang a privileged
domain (dom0), although root privilege is required.

Resetting the VM_PFNMAP flag is a little bit kludgy, but
fixes the issue.

After this patch is applied, other patches to Qemu become
necessary to create a HVM domain.

Signed-off-by: Kouya Shimura <kouya@jp.fujitsu.com>

xen: Fix PV resume race against another back-to-back suspend request.
Previously the next suspend request was being dropped on the floor.
Signed-off-by: Keir Fraser <keir.fraser@citrix.com>

Reduce diff between arch/i386/pci/irq.c and irq-xen.c.
Signed-off-by: Keir Fraser <keir.fraser@citrix.com>

xenkbd: Fix z-axis mouse-wheel event reporting.
From: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Keir Fraser <keir.fraser@citrix.com>

merge

kexec: dont initialise regions in reserve_memory()

There is no need to initialise efi_memmap_res and boot_param_res in
reserve_memory() for the initial xen domain as it is done in
machine_kexec_setup_resources() using values from the kexec hypercall.

Signed-off-by: Simon Horman <horms@verge.net.au>

kexec: read more iomem regions from hypervisor

This sets the location of the efi memmap and boot parameter
regions using information provided by the hypervisor,
overriding values derived by dom0 from the virtualised
efi memory regions.

It also creates a xen heap region and uses this as the parent
of per-cpu regions - they belong in hypervisor memory not
dom0 kernel memory.

The xen heap region is inserted into /proc/iomem_machine

* There is also a hypervisor portion of this patch.
* In order for the regions to show up after kexec patches
  to kexec-tools are required. I have posted them
  to the kexec mailing list and intend to merge them.

  http://lists.infradead.org/pipermail/kexec/2008-February/001348.html

Signed-off-by: Simon Horman <horms@verge.net.au>

kexec: add xen_machine_kexec_register_resources() and machine_kexec_register_resources()

Add xen_machine_kexec_register_resources() and
machine_kexec_register_resources() to allow architecture specific
handling of iomem resources.

At this time xen_machine_kexec_register_resources() does the
same parenting of per-cpu resources on all architectures.
And machine_kexec_register_resources does nothing on all
architectures.

Signed-off-by: Simon Horman <horms@verge.net.au>

kexec: add parent to per-cpu regions at setup time.

This is slightly more efficient as xen_machine_kexec_setup_resorces()
is called once (before xen_machine_kexec_register_resources()). While
xen_machine_kexec_register_resources() is called once for each EFI
memory region seen by a domain.

More cosmetic than anything else, but it seems more logical to me.

Signed-off-by: Simon Horman <horms@verge.net.au>

kexec: Use error path if crash region range can't be accessed

Although the error handling path in xen_machine_kexec_setup_resource()
is somewhat minmal, it ought to be used if HYPERVISOR_kexec_op() fails
whengetting the crash kernel region, as this indicates that an error
occured, not that the crash kernel region is empty.

Signed-off-by: Simon Horman <horms@verge.net.au>

Fully initialise watch data structure in pcifront by using kzalloc.

This avoids creating a watch with XBWF_new_thread set spuriously.

Signed-off-by: Ian Campbell <ian.campbell@citrix.com>

netfront accel: Fix request_module/modprobe deadlock

There would seem to be a potential deadlock in the netfront accelerator
plugin support.  When the configured accelerator changes in xenstore,
netfront tries to load the new plugin using request_module().  It does
this from a workqueue work item.  request_module() will invoke
modprobe which in some circumstances (I'm not sure exactly what - I've
not managed to reproduce it myself) seems to try to flush the
workqueue, and so it deadlocks.   This patch fixes the problem by
giving the accel watch work item its own workqueue, and so modprobe
can successfully flush the system-wide one.

Signed-off-by Kieran Mansley <kmansley@solarflare.com>

Xen paravirtualised PCI hotplug.
Signed-off-by: Yosuke Iwamatsu <y-iwamatsu@ab.jp.nec.com>

Solarflare: use simplifying header-offset macros where possible
Signed-off-by: Jan Beulich <jbeulich@novell.com>

Solarflare: adjust Kconfig additions

Clean up drivers/xen/Kconfig after the Solarflare additions:
- placement of new items should not disturb menu hierarchy
- dependencies of XEN_NETDEV_ACCEL_SFC_BACKEND were missing
- use tabs for indentation

Signed-off-by: Jan Beulich <jbeulich@novell.com>
Signed-off-by: Kieran Mansley <kmansley@solarflare.com>

Solarflare: adjust module_param() permissions

Using permissions with bit 1 set (i.e. 0666) is prohibited on module
parameters on 2.6.20+.

Signed-off-by: Jan Beulich <jbeulich@novell.com>
Signed-off-by: Kieran Mansley <kmansley@solarflare.com>

Solarflare: Various build fixes, and make SFC drivers dependent on x86
From: Kieran Mansley <kmansley@solarflare.com>
Signed-off-by: Keir Fraser <keir.fraser@citrix.com>

Solarflare: Add -I. to sfc module builds
Signed-off-by: Kieran Mansley <kmansley@solarflare.com>

[IA64] Update buildconfigs

Solarflare doesn't build on ia64 yet

Signed-off-by: Alex Williamson <alex.williamson@hp.com>

merge with linux-2.6.18-xen.hg

merge with linux-2.6.18-xen.hg

Xen dom0 arbitrarily assigns APIC ID x to CPU ID x.  Make dom0 also
assign the APIC ID to ACPI ID mapping in the same way.
Signed-off-by: Mark Langsdorf <mark.langsdorf@amd.com>

fbfront: Fix no-kbd-event case
out_cons shouldn't even be used.

Signed-off-by: Samuel Thibault <samuel.thibault@eu.citrix.com>

Added tag xen-3.2.0 for changeset 08e85e79c65d0316bfda5e77e8a0dc7ab9ca181a

CVE-2008-0600: Fix exploitable hole in vmsplice() syscall.
Fix is Al Viro's suggested patch for RHEL5.
Signed-off-by: Keir Fraser <keir.fraser@citrix.com>

fbfront: Clear ring contents on save/restore. Otherwise in some cases
a restored domain loses mouse and keyboard.
Signed-off-by: Kazuhiro Suzuki <kaz@jp.fujitsu.com>

block: backport Jens Axboe's commit from
Tue, 16 Oct 2007 09:03:56 +0000 (11:03 +0200)
bf2de6f5a4faf0197268f18d08969b003b87b6e8
Initial support for data-less (or empty) barrier support

blkback: permit and implement empty barrier.
Signed-off-by: Samuel Thibault <samuel.thibault@eu.citrix.com>

blkback: Fix st_wr_sect accounting by including write-barrier requests.
Signed-off-by: Samuel Thibault <samuel.thibault@eu.citrix.com>

net accel: Prevent frontend network accelerator plugin loading in dom0

Generate error if frontend plugin tries to connect on dom0

Signed-off-by: Kieran Mansley <kmansley@solarflare.com>

net accel: Fix double-probe of accelerator on suspend_cancel

Fixes a bug in the network acceleration stuff where an accelerator
could get probed with the same interface twice on a suspend-cancel -
once manually in the suspend_cancel handler, and once when the watch
on the accel configuration option fired after being reinstated.

Signed-off-by: Kieran Mansley <kmansley@solarflare.com>

xen balloon: allocate and free cold pages

To reduce the performance side effects of ballooning, use and return
cold pages. To limit the impact scrubbing of these (and other) pages
has on the cache, also implement a dedicated scrubbing function on x86
which uses non-temporal stores (when available).

Signed-off-by: Jan Beulich <jbeulich@novell.com>

Create hgignore file: ignore patch droppings and .valid-src.

xen/x86: fix and improve xen_limit_pages_to_max_mfn()

- don't do multicall when nr_mcl is zero (and specifically don't
  access cr_mcl[nr_mcl - 1] in that case)
- fix CONFIG_XEN_COMPAT <=3D 0x030002 handling
- don't exchange pages already meeting the restriction (likely
  avoiding exchanging anything at all)
- avoid calling kmap functions without CONFIG_XEN_SCRUB_PAGES
- eliminate a few local variables

Signed-off-by: Jan Beulich <jbeulich@novell.com>

merge with linux-2.6.18-xen.hg

linux/x86: don't hardcode is_running_on_xen() for pv-on-hvm drivers

Allowing graceful failure of these modules when inadvertently loaded
on native kernels.

Signed-off-by: Jan Beulich <jbeulich@novell.com>

[IA64] Remove dead code

Remove ski simulator related stuff since it is for early
Xen development stage, and no longer necessary for now.

Signed-off-by: YaoZu (Eddie) Dong <eddie.dong@intel.com>

netback: Fix BUG_ON() on page-flip receive path which would always
trigger and crash the kernbel.
Tracked down by Joakim Dahlstedt <jda@bea.com>
Signed-off-by: Keir Fraser <keir.fraser@citrix.com>

[IA64] Split xencomm_arch_hypercall_suspend to its own file

This allows us to use it from the unmodified driver, eliminating the following
error:

  Loading xen-platform-pci.ko module
  xen_platform_pci: Unknown symbol xencomm_hypercall_suspend
  insmod: error inserting '/lib/xen-platform-pci.ko':
   -1 Unknown symbol in module
  ERROR: /bin/insmod exited abnormally! (pid 210)

Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp>

Fix x86 SMP non-ACPI build.
Signed-off-by: Keir Fraser <keir.fraser@citrix.com>

Update Xen/x86 defconfig files.
Signed-off-by: Keir Fraser <keir.fraser@citrix.com>

Solarflare: PV netback accelerator.
Signed-off-by: Kieran Mansley <kmansley@solarflare.com>

Solarflare: PV frontend accelerator and front/back common util driver.
Signed-off-by: Kieran Mansley <kmansley@solarflare.com>

Solarflare: Resource driver.
Signed-off-by: Kieran Mansley <kmansley@solarflare.com>

Solarflare: Standard network driver.
Signed-off-by: Kieran Mansley <kmansley@solarflare.com>

ebtables: don't compute gap until we know we have an ebt_entry
Original upstream Linux patch by Chuck Ebbert.
Signed-off-by: Keir Fraser <keir.fraser@citrix.com>

linux/x86: make xen_change_pte_range() compatible with CONFIG_HIGHPTE

Cannot use virt_to_machine() on a kmap()-ed address.

Signed-off-by: Jan Beulich <jbeulich@novell.com>

linux/x86: Fix a missing hypercall WARN_ON().
Signed-off-by: Jan Beulich <jbeulich@novell.com>

Do not allocate vcpu_guest_context on the stack when initialising a
new VCPU. It is too big for 4kB stacks.
Original patch by Donald Dutile <ddutile@redhat.com> backported from
upstream pv_ops work.
Signed-off-by: Keir Fraser <keir.fraser@citrix.com>

linux/x86: avoid casting hypercall return value

.., being a little more type correct and producing better code on
64-bits.

Signed-off-by: Jan Beulich <jbeulich@novell.com>

linux/x86-64: don't use explicit moves in 4- and 5-argument hypercall macros

.., allowing the compiler to schedule the moves earlier or even avoid
an extra move.

Signed-off-by: Jan Beulich <jbeulich@novell.com>

[gntdev] Fix multiple-grant mapping logic
Signed-off-by: Derek Murray <Derek.Murray@cl.cam.ac.uk>

xen, x86: Revert changes to use non-existent xen_halt() function.
Signed-off-by: Keir Fraser <keir.fraser@citrix.com>

xen/x86: Remove use of undefined __noreturn function attribute.
Signed-off-by: Keir Fraser <keir.fraser@citrix.com>

merge with linux-2.6.18-xen.hg (staging)

[IA64] Coding style fix

Mainly white spaces, // comments and * ops.

Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp>

xen/x86: make set_ldt() more consistent with native

Signed-off-by: Jan Beulich <jbeulich@novell.com>

xen: check hypercall return status

Not doing so is almost always a mistake, potentially resulting in
debugging of problems being much more difficult.

Signed-off-by: Jan Beulich <jbeulich@novell.com>

blkback/blktap: Check for kthread_should_stop() in inner loop,
mdelaay() should be msleep(), and these changes belong in blktap as
well as blkback.
Based on comments and patches from Jan Beulich and Steven Smith.
Signed-off-by: Keir Fraser <keir.fraser@citrix.com>

blkback: Request-processing loop is unbounded and hence requires a
yield point. Also, bad request type is a good cause to sleep for a
short while as the frontend has probably gone mad.

Patch by Steven Smith <steven.smith@eu.citrix.com>

Signed-off-by: Keir Fraser <keir.fraser@citrix.com>

linux/x86: make hypercall wrapper functions' arguments unsigned where possible
Signed-off-by: Jan Beulich <jbeulich@novell.com>

linux/x86: clean up hypercall headers

- don't define HYPERVISOR_hvm_op() for pv guests (requiring to not
  include
  include/xen/hvm.h in non-pv-driver builds)
- remove the custome __STR/STR macros
- remove stringification where not necessary
- reduce instruction size for pv-driver case on x86-64

Signed-off-by: Jan Beulich <jbeulich@novell.com>

linux/x86: convert 'cpu' (and a few other) variables to unsigned

Produces better code for x86-64.

Signed-off-by: Jan Beulich <jbeulich@novell.com>

linux: prevent non-Xen modules from loading in Xen kernel
Signed-off-by: Jan Beulich <jbeulich@novell.com>

xen: do_set_trap_table()'s argument can be const
Signed-off-by: Jan Beulich <jbeulich@novell.com>

xen: Constify pointer argument of HYPERVISOR_callback_op
Signed-off-by: Jan Beulich <jbeulich@novell.com>