On Thu, 2017-12-07 at 18:09 +0800, Haozhong Zhang wrote: > cleanup_frame_table() initializes the entire newly added frame table > to all -1's. If it's called after extend_frame_table() failed to map > the entire frame table, the initialization will hit a page fault. > > Move the cleanup of partially mapped frametable to > extend_frame_table(), > which has enough knowledge of the mapping status. > > Signed-off-by: Haozhong Zhang <haozhong.zh...@intel.com> > --- > Cc: Jan Beulich <jbeul...@suse.com> > Cc: Andrew Cooper <andrew.coop...@citrix.com> > > @Chao: I don't modify this patch per your comment, because I feel it's > better to handle the errors locally in each function (rather than > handle > all of them in the top-level), which will make each function easier to > use.
I don't insist on this, to me this is kind of flavor choice. Chao _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
