[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v19 3/7] xbitmap: add more operations
From: |
Tetsuo Handa |
Subject: |
Re: [Qemu-devel] [PATCH v19 3/7] xbitmap: add more operations |
Date: |
Sun, 17 Dec 2017 19:21:35 +0900 |
Wei Wang wrote:
> > But passing GFP_NOWAIT means that we can handle allocation failure. There is
> > no need to use preload approach when we can handle allocation failure.
>
> I think the reason we need xb_preload is because radix tree insertion
> needs the memory being preallocated already (it couldn't suffer from
> memory failure during the process of inserting, probably because
> handling the failure there isn't easy, Matthew may know the backstory of
> this)
According to https://lwn.net/Articles/175432/ , I think that preloading is
needed
only when failure to insert an item into a radix tree is a significant problem.
That is, when failure to insert an item into a radix tree is not a problem,
I think that we don't need to use preloading.
>
> So, I think we can handle the memory failure with xb_preload, which
> stops going into the radix tree APIs, but shouldn't call radix tree APIs
> without the related memory preallocated.
It seems to me that virtio-ballon case has no problem without using preloading.
Re: [Qemu-devel] [PATCH v19 3/7] xbitmap: add more operations, Matthew Wilcox, 2017/12/15
- Re: [Qemu-devel] [PATCH v19 3/7] xbitmap: add more operations, Wei Wang, 2017/12/16
- Re: [Qemu-devel] [PATCH v19 3/7] xbitmap: add more operations, Tetsuo Handa, 2017/12/16
- Re: [Qemu-devel] [PATCH v19 3/7] xbitmap: add more operations, Wei Wang, 2017/12/17
- Re: [Qemu-devel] [PATCH v19 3/7] xbitmap: add more operations,
Tetsuo Handa <=
- Re: [Qemu-devel] [PATCH v19 3/7] xbitmap: add more operations, Wang, Wei W, 2017/12/17
- Re: [Qemu-devel] [PATCH v19 3/7] xbitmap: add more operations, Tetsuo Handa, 2017/12/17
- Re: [Qemu-devel] [PATCH v19 3/7] xbitmap: add more operations, Wei Wang, 2017/12/18
[Qemu-devel] [PATCH v19 4/7] virtio-balloon: VIRTIO_BALLOON_F_SG, Wei Wang, 2017/12/12
[Qemu-devel] [PATCH v19 5/7] mm: support reporting free page blocks, Wei Wang, 2017/12/12
[Qemu-devel] [PATCH v19 6/7] virtio-balloon: VIRTIO_BALLOON_F_FREE_PAGE_VQ, Wei Wang, 2017/12/12
[Qemu-devel] [PATCH v19 7/7] virtio-balloon: don't report free pages when page poisoning is enabled, Wei Wang, 2017/12/12