[CRIU] [PATCHv4 0/8] Large pages support for aarch64/ppc64

Adrian Reber areber at redhat.com
Fri May 11 16:29:15 MSK 2018


On Fri, May 11, 2018 at 01:52:08PM +0100, Dmitry Safonov wrote:
> 2018-05-11 9:47 GMT+01:00 Adrian Reber <areber at redhat.com>:
> > Testing on ppc64le looks good:
> >
> > ==== ALARM ====
> > === Run 351/355 ===============- zdtm/transition/pid_reuse
> > === Run 352/355 ===============- zdtm/transition/file_read
> > === Run 353/355 ===============- zdtm/transition/unix_sock
> > Skipping zdtm/transition/fifo_dyn (manual run only)
> > === Run 355/355 ================ zdtm/transition/fifo_loop
> > ==== ALARM ====
> > ################### 1 TEST(S) FAILED (TOTAL 355/SKIPPED 34) ####################
> >  * zdtm/static/maps06(h)
> > ##################################### FAIL #####################################
> >
> > You said some time ago that maps06 has hardcoded page size and therefore it fails.
> >
> > On aarch64:
> >
> > ==== ALARM ====
> > === Run 353/355 ===============- zdtm/transition/unix_sock
> > Skipping zdtm/transition/fifo_dyn (manual run only)
> > ==== ALARM ====
> > === Run 355/355 ================ zdtm/transition/fifo_loop
> > ==== ALARM ====
> > ################### 11 TEST(S) FAILED (TOTAL 355/SKIPPED 35) ###################
> >  * zdtm/static/fd(h)
> >  * zdtm/static/userns00(uns)
> >  * zdtm/static/userns01(uns)
> >  * zdtm/static/userns02(uns)
> >  * zdtm/static/userns-leaked-sock(uns)
> >  * zdtm/static/netns_sub_veth(uns)
> >  * zdtm/static/pidns01(uns)
> >  * zdtm/static/maps06(h)
> >  * zdtm/static/cow01(h)
> >  * zdtm/transition/maps007(h)
> >  * zdtm/transition/socket_loop00(ns)
> > ##################################### FAIL #####################################
> 
> Thanks, Adrian!
> 
> AFAIU, those are not regressions of the patches set, right?
> So, just not to enlarge the number of patches, the fails will
> be fixed on top of the set. I prefer having 10 patches in a set
> or less, rather than having 30 patches those fix everything.

I agree. Without your patches it is not working at all. So I am also in
favor of merging your patches.

		Adrian


More information about the CRIU mailing list