I was looking through the code for other ways to cut down memory usage when I noticed we only catch improper re-allocations, not adding another ref for metadata which is what I suspect your problem is. I added another patch and pushed it out, sorry for the churn. Josef Sent from my iPhone > On Sep 2, 2017, at 9:01 PM, Marc MERLIN <marc@xxxxxxxxxxx> wrote: > >> On Sun, Sep 03, 2017 at 12:30:07AM +0000, Josef Bacik wrote: >> My bad, I forgot I don't dynamically allocate the stack trace space so my patch did nothing, I blame the children for distracting me. I've dropped allocating the action altogether for the on disk stuff, that should dramatically reduce the memory usage. You can just do a git pull since I made a new commit. You are mounting with -o ref_verify on only the one fs right? Give this a try and if it still doesn't work we can try a stripped down version that doesn't build the initial tree and just hope that the problem exists in allocating a new block and not modifying the refs for an existing block. Thanks, > > Good news, this time it booted without crashing on OOM. > > I'll now get to see how it runs and hopefully it won't crash due to > other problems in 4.13 > > Marc > -- > "A mouse is a device used to point at the xterm you want to type in" - A.S.R. > Microsoft is to operating systems .... > .... what McDonalds is to gourmet cooking > Home page: https://urldefense.proofpoint.com/v2/url?u=http-3A__marc.merlins.org_&d=DwIBAg&c=5VD0RTtNlTh3ycd41b3MUw&r=sDzg6MvHymKOUgI8SFIm4Q&m=-zFzT4JPwAa-JY-PU1TRHuerYPlZf00HGKCTgtSRcxU&s=fyD-Ff-h7AsoFbRF2RqvzlURQJg38B1RTu7A_n0OLs8&e= | PGP 1024R/763BE901 -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
