All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/4] fix preemption handling for XENMEM_add_to_physmap_range{, _range}
@ 2013-12-18 14:29 Jan Beulich
  2013-12-18 14:34 ` [PATCH 1/4] move XENMEM_add_to_physmap handling framework to common code Jan Beulich
                   ` (6 more replies)
  0 siblings, 7 replies; 47+ messages in thread
From: Jan Beulich @ 2013-12-18 14:29 UTC (permalink / raw)
  To: xen-devel; +Cc: Tim Deegan, Keir Fraser, Ian Campbell, Stefano Stabellini

The hypervisor isn't supposed to use the input structures for
storing continuation information - only fields explicitly used as
hypercall outputs should ever be updated.

Obviously this implies an ABI change, but since the previous
behavior was not intended to be that way I don't think we should
stick to the old behavior.

There's one caveat though - with the previous model, the caller
could - upon failure - use the updated structure to find out how
by progress was made. However, that wasn't intended afaict,
largely supported by this information depending on hypervisor
internals (i.e. the caller would have to know the order of request 
processing and the meaning of the respective size fields, which
aren't simply saying "this much was processed").

Consequently, as a follow-up we may want to consider making
explicit (and straight forward) this progress indication on error.

1: move XENMEM_add_to_physmap handling framework to common code
2: fix XENMEM_add_to_physmap preemption handling
3: move XENMEM_add_to_physmap_range handling framework to common code
4: fix XENMEM_add_to_physmap_range preemption handling

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

^ permalink raw reply	[flat|nested] 47+ messages in thread

end of thread, other threads:[~2013-12-20 12:45 UTC | newest]

Thread overview: 47+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-12-18 14:29 [PATCH 0/4] fix preemption handling for XENMEM_add_to_physmap_range{, _range} Jan Beulich
2013-12-18 14:34 ` [PATCH 1/4] move XENMEM_add_to_physmap handling framework to common code Jan Beulich
2013-12-18 17:52   ` Ian Campbell
2013-12-20  7:48     ` Jan Beulich
2013-12-20  8:58       ` Ian Campbell
2013-12-20  9:12         ` Jan Beulich
2013-12-20  9:55           ` Ian Campbell
2013-12-18 14:35 ` [PATCH 2/4] fix XENMEM_add_to_physmap preemption handling Jan Beulich
2013-12-18 15:48   ` Tim Deegan
2013-12-18 16:03     ` Jan Beulich
2013-12-18 16:04       ` Tim Deegan
2013-12-18 17:59   ` Ian Campbell
2013-12-20  7:52     ` Jan Beulich
2013-12-20  9:02       ` Ian Campbell
2013-12-20  9:16         ` Jan Beulich
2013-12-18 14:35 ` [PATCH 3/4] move XENMEM_add_to_physmap_range handling framework to common code Jan Beulich
2013-12-18 15:59   ` Tim Deegan
2013-12-18 16:08     ` Jan Beulich
2013-12-18 16:14       ` Tim Deegan
2013-12-19 10:48   ` Ian Campbell
2013-12-20  7:58     ` Jan Beulich
2013-12-20  9:09       ` Ian Campbell
2013-12-20  9:27         ` Jan Beulich
2013-12-18 14:36 ` [PATCH 4/4] fix XENMEM_add_to_physmap_range preemption handling Jan Beulich
2013-12-18 14:43 ` [PATCH 0/4] fix preemption handling for XENMEM_add_to_physmap_range{, _range} Ian Campbell
2013-12-18 15:04   ` Jan Beulich
2013-12-18 17:15 ` Keir Fraser
2013-12-20  8:22 ` [PATCH v2 " Jan Beulich
2013-12-20  8:28   ` [PATCH v2 1/4] move XENMEM_add_to_physmap handling framework to common code Jan Beulich
2013-12-20  8:29   ` [PATCH v2 2/4] fix XENMEM_add_to_physmap preemption handling Jan Beulich
2013-12-20  8:29   ` [PATCH v2 3/4] move XENMEM_add_to_physmap_range handling framework to common code Jan Beulich
2013-12-20  8:30   ` [PATCH v2 4/4] fix XENMEM_add_to_physmap_range preemption handling Jan Beulich
2013-12-20 10:35   ` [PATCH v3 0/5] fix preemption handling for XENMEM_add_to_physmap_range{, _range} Jan Beulich
2013-12-20 10:41     ` [PATCH v3 1/5] move XENMEM_add_to_physmap handling framework to common code Jan Beulich
2013-12-20 10:45       ` Ian Campbell
2013-12-20 12:26       ` Andrew Cooper
2013-12-20 12:45         ` Jan Beulich
2013-12-20 10:42     ` [PATCH v3 2/5] fix XENMEM_add_to_physmap preemption handling Jan Beulich
2013-12-20 10:47       ` Ian Campbell
2013-12-20 10:42     ` [PATCH v3 3/5] move XENMEM_add_to_physmap_range handling framework to common code Jan Beulich
2013-12-20 10:51       ` Ian Campbell
2013-12-20 10:43     ` [PATCH v3 4/5] fix XENMEM_add_to_physmap_range preemption handling Jan Beulich
2013-12-20 10:53       ` Ian Campbell
2013-12-20 10:44     ` [PATCH v3 5/5] rename XENMEM_add_to_physmap_{range => batch} Jan Beulich
2013-12-20 10:55       ` Ian Campbell
2013-12-20 12:05         ` Jan Beulich
2013-12-20 12:09           ` Ian Campbell

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.