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 <[email protected]> > --- > Cc: Jan Beulich <[email protected]> > Cc: Andrew Cooper <[email protected]> > > @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 [email protected] https://lists.xenproject.org/mailman/listinfo/xen-devel
