From nobody Sat Dec 25 17:56:10 2021 X-Original-To: dev-commits-src-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id EC3F8191771A; Sat, 25 Dec 2021 17:56:18 +0000 (UTC) (envelope-from unkadoug@gmail.com) Received: from mail-oi1-x22a.google.com (mail-oi1-x22a.google.com [IPv6:2607:f8b0:4864:20::22a]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (2048 bits) client-digest SHA256) (Client CN "smtp.gmail.com", Issuer "GTS CA 1D4" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4JLs765ylcz4cNv; Sat, 25 Dec 2021 17:56:18 +0000 (UTC) (envelope-from unkadoug@gmail.com) Received: by mail-oi1-x22a.google.com with SMTP id i9so10251343oih.4; Sat, 25 Dec 2021 09:56:18 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=from:subject:to:cc:references:message-id:date:user-agent :mime-version:in-reply-to:content-transfer-encoding:content-language; bh=RfP17LXJZhml8vjYscjXXEsUW5DHqKQ/4dgaoE++Fsc=; b=HVYYJreQ+dPADPRZq4Y4nd9RAeXuFxN0BySxgphz+Wmpna9IsaaQHkHyBAWb77XYgK Ef4S0GSRNhn66T7vkvabLYX9TyuprJekwxGA+hRjcmNQkpJ3ES1FwFocKX1xHn/mSDyz Y7dLwOlI4Zk1zdaH+tY9hfDSHYtyfCCXg1TbdcLKXNXJlZuKwnffRZTufUDLiguZnMiN 3EFtsdjnE87GXfuSBd0J8ZWdi7tlBof6+2wqZYqdMQ6v8Qffs3byGV8CDA/sohjZPzqH s3vjTWWasdYbIx1obHdAADgrekEWbn+LbqE8dXOCh6dJgleZ1oo8Qs9LybNYfsRJiqb2 uMLA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:subject:to:cc:references:message-id:date :user-agent:mime-version:in-reply-to:content-transfer-encoding :content-language; bh=RfP17LXJZhml8vjYscjXXEsUW5DHqKQ/4dgaoE++Fsc=; b=DPUc/mdPLCLEjMqVNw5kJXdcV+vnDa0wsMRk6ZX2OtwL4K6/B7xrikfgtAUNX7vi8c ZA13ylpO4K6gG6yh1QUIcxhTtE79MREoP2AGRa1UYRRNGHwRKeWgwdBrlH9mSOUog/aS j4uFfAsKCdjfAuixVR10RnMKKfsDr2sAtnnmEX1CNN2LV4mugTZyloniO+48Sz0LFGR8 WvSia8UpV48S0lFIYBo3kUBa/4vAt1JdCFrxj1Oio5ZGpKDZuivjEL9W2ExHe0tZ0Yy9 AurkqwFbbgzPbx5B/wGGCHC1S/jiiSsAW76yQPPhH4a9TGqYTnnM21A3a94Kzoos72uA 68LA== X-Gm-Message-State: AOAM531O9fYyKgEqLL6TO0uWJf/ZjzCz9+vtsOAgFsDWPDEhOV2FkvQC qKT0kyI6nnCk8MmfqM76LlnuO13SK6I= X-Google-Smtp-Source: ABdhPJzqGD/DadAEDqzprUJ0W+j9SdjS9A6/JeHnwENuTLY82FQ1Pex5XCVgbctbgzTVCJiBwUDzdA== X-Received: by 2002:a05:6808:22a6:: with SMTP id bo38mr8473816oib.78.1640454971848; Sat, 25 Dec 2021 09:56:11 -0800 (PST) Received: from 108-254-203-202.lightspeed.hstntx.sbcglobal.net (108-254-203-202.lightspeed.hstntx.sbcglobal.net. [108.254.203.202]) by smtp.gmail.com with ESMTPSA id y22sm2020288ots.40.2021.12.25.09.56.10 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sat, 25 Dec 2021 09:56:11 -0800 (PST) From: Doug Moore X-Google-Original-From: Doug Moore Subject: Re: git: 184c63db3c94 - main - Fix clerical error in page alloc To: Guido Falsi , FreeBSD User Cc: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org References: <202112240851.1BO8pdnH043305@gitrepo.freebsd.org> <20211225122334.651e9acd@jelly.fritz.box> <675f75ea-b5ca-a47c-f1a9-9621b0c8b36e@FreeBSD.org> Message-ID: Date: Sat, 25 Dec 2021 11:56:10 -0600 User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:78.0) Gecko/20100101 Thunderbird/78.14.0 List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-src-all@freebsd.org X-BeenThere: dev-commits-src-all@freebsd.org MIME-Version: 1.0 In-Reply-To: <675f75ea-b5ca-a47c-f1a9-9621b0c8b36e@FreeBSD.org> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Content-Language: en-US X-Rspamd-Queue-Id: 4JLs765ylcz4cNv X-Spamd-Bar: ---- Authentication-Results: mx1.freebsd.org; none X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[] X-ThisMailContainsUnwantedMimeParts: N Guido - I've made three vm changes this month, though it's taken me 6 commits to do it. The first change was trivial. The last commit before the second change was commit 02732f945ed2ec2b4fd03421923720608b28a615 and that change was in place after commit f7aa44763d20d06c9ea5caf330aca02a8b107a70 The last commit before the third change was commit b7ec0d268b73ce20c4f785d21cde9b174c91a553 and it was done after commit 0d5fac287294490ac488d74e598e019334610bdb (except for moving a comment). I suggest you test before and after the second change to see if it introduced a problem, then before and after the third change. Doug Moore (dougm@freebsd.org) On 12/25/21 11:11 AM, Guido Falsi wrote: > On 25/12/21 12:23, FreeBSD User wrote: >> Am Fri, 24 Dec 2021 08:51:39 GMT >> schrieb Doug Moore : >> >>> The branch main has been updated by dougm: >>> >>> URL: >>> https://cgit.FreeBSD.org/src/commit/?id=184c63db3c949d8ba766dc7b2bd2f082404e169d >>> >>> >>> commit 184c63db3c949d8ba766dc7b2bd2f082404e169d >>> Author:     Doug Moore >>> AuthorDate: 2021-12-24 08:47:21 +0000 >>> Commit:     Doug Moore >>> CommitDate: 2021-12-24 08:47:21 +0000 >>> >>>      Fix clerical error in page alloc >>>           Fix a very recent change that introduced a page accounting >>> error >>> in case of a reserveration being broken. >>>      Reviewed by:    alc >>>      Fixes:  fb38b29b5609 (page_alloc_br) vm_page: Remove extra test, >>> dup code from page alloc Differential Revision: >>> https://reviews.freebsd.org/D33645 --- >>>   sys/vm/vm_page.c | 4 ++-- >>>   1 file changed, 2 insertions(+), 2 deletions(-) >>> >>> diff --git a/sys/vm/vm_page.c b/sys/vm/vm_page.c >>> index c24da96f4312..03351b0ad3dd 100644 >>> --- a/sys/vm/vm_page.c >>> +++ b/sys/vm/vm_page.c >>> @@ -2186,11 +2186,11 @@ vm_page_find_contig_domain(int domain, int >>> req, u_long npages, vm_paddr_t low, vm_page_t m_ret; >>>         vmd = VM_DOMAIN(domain); >>> -    if (!vm_domain_allocate(vmd, req, npages)) >>> -        return (NULL); >>>   #if VM_NRESERVLEVEL > 0 >>>   again: >>>   #endif >>> +    if (!vm_domain_allocate(vmd, req, npages)) >>> +        return (NULL); >>>       /* >>>        * Try to allocate the pages from the free page queues. >>>        */ >>> >> >> It seems that our hosts running with this patch are "dead" after a >> while while under load (poudriere): ssh on both IPv4 and IPv6 are dead >> as well as http/https on IPv4/IPv6 (remote site, no connection via ssh >> anymore, but hosts respond to ping, nmap show several other services on >> local network as reachable, but no ssh(22)/apache24(http/https). >> Other hosts at OS level before this patch seem to be allright so far >> (i.e. FreeBSD 14.0-CURRENT #39 main-n251899-fa255ab1b895: Thu Dec 23 >> 13:48:41 CET 2021 amd64). >> >> I have to admit its a wild guess that this patch is the culprit, but it >> is strange that two out of four hosts with this patch applied are now >> both unreachable on both ssh and http (lates www/apache24) while two >> other hosts stuck with the version showed above seem to operate on >> ssh/http. >> >> Can investigate earliest after 26th of December. > > > I'm also seeing a strange behaviour related to memory and the VM > subsystem with recent (24th December) head . It was not happening with > head from mid November. > > I'm seeing strange issues with virtualbox on recent head too. It fails > to launch VMs or VMs pause due to memory exhaustion, while the machine > has lots of free memory. Maybe the real issue is memory fragmentation > though. > > I'm now testing updating to newer head including commit > 0d5fac287294490ac488d74e598e019334610bdb (vm: alloc pages from reserv > before breaking it) which is definitely related and maybe fix this. > > Anyway there is definitely something going on with recent changes to > VM subsystem that requires investigation. > > I can reproduce this easily, just install virtualbox-ose, create a VM > with a non tiny memory footprint and run it, or run more than one. It > is easier to reproduce if some other software is already running on > the machine (this is why I suspect some memory fragmentation issue). >