From nobody Mon Feb 9 21:21:31 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) client-ip=208.118.235.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com Return-Path: Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) by mx.zohomail.com with SMTPS id 1527020123634387.5391127070326; Tue, 22 May 2018 13:15:23 -0700 (PDT) Received: from localhost ([::1]:57727 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fLDgk-0004tE-S3 for importer@patchew.org; Tue, 22 May 2018 16:15:18 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:46312) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fLDci-0001gw-Rc for qemu-devel@nongnu.org; Tue, 22 May 2018 16:11:09 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1fLDch-0002y0-QM for qemu-devel@nongnu.org; Tue, 22 May 2018 16:11:08 -0400 Received: from mail-pl0-x243.google.com ([2607:f8b0:400e:c01::243]:39442) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1fLDch-0002wq-KM; Tue, 22 May 2018 16:11:07 -0400 Received: by mail-pl0-x243.google.com with SMTP id c19-v6so11547438pls.6; Tue, 22 May 2018 13:11:07 -0700 (PDT) Received: from donizetti.event.priv ([72.28.93.232]) by smtp.gmail.com with ESMTPSA id r90-v6sm43926803pfg.122.2018.05.22.13.11.04 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Tue, 22 May 2018 13:11:05 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:from:to:cc:subject:date:message-id:in-reply-to:references; bh=ydnM9u+p1O2JYBmclDANVa19zTRw/1cpjuzYTFqtjkc=; b=ognHCtGXO/ZcPYCHHfq5R7z+JAPmM7IgBlE2oa+w/9yba+PdSR7UybtywCI9i2neO4 ORESH4rHKuwWQjCMCXruFtgUcTvPAtvWEyDG/fZEJFIFgJsh8eiqAgdwwErqeV0KGYaV PawHGlnqa6z/TBItwEd9D/WhZSOEs76gMPc5sILKV0fJS/Ljt1T0pe2h54Gl7wVPUZqU Sex5LagqCJciTFoBAtWh/rY7tx7hRCPjji+LYNmNgVf3UqLa/RxQMAIyD9MAY0MNwCZ1 AQNXCOfUFl5pyWi82CEC+4iLXmjzObYJVPQ1lpkHwcNNhyjuh11JT2MSwD2b76+kpxR9 crfQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:from:to:cc:subject:date:message-id :in-reply-to:references; bh=ydnM9u+p1O2JYBmclDANVa19zTRw/1cpjuzYTFqtjkc=; b=iSCmrQU0I3dfbWseobhzwbftPDxXuHEAFran+Eb7QwgDSGvf2YAmON4cxEhmQ4t9TF 1FVElRL94SnNS93o2f6ExRQPR4XquJra/WVz6QgqfsmvkICej/iGcSQ+g1z49nM5uuew cLy8VznhN/FN86lK/cdhiWgsYdry5bUBF3QbmVnE3fFXNnrmJipIUDo8UKnsmIXDN8u2 bj+Ne9+rBtWZFLIsmH+/IdzrObdJHVajUpx2R+Zmnl7OFYITV2x7cwQ9JkwKWO+l8mR7 btvwFLhfoGndrsV0wKUIwSsCgh4iNsuBRuNzfZIPfwJHNcgBmzI0aMxhi96aCOeePElR vsJw== X-Gm-Message-State: ALKqPwd98xrkMEYh1GwWPi6kJT82ch44DJqNmL9zZRGSbsJFdHWbehoQ RKBDDxfYTMIq69dcSmLC9Lq6vSPd X-Google-Smtp-Source: AB8JxZoO4fyRdXkQr1k5TCSMgCaP8Ib8Trt4l1kuPaf65s2N6e2dWECCG4qUqjvaKvaD7pPMjs11dw== X-Received: by 2002:a17:902:7402:: with SMTP id g2-v6mr26786987pll.246.1527019866456; Tue, 22 May 2018 13:11:06 -0700 (PDT) From: Paolo Bonzini To: qemu-devel@nongnu.org Date: Tue, 22 May 2018 22:10:56 +0200 Message-Id: <20180522201056.1442-3-pbonzini@redhat.com> X-Mailer: git-send-email 2.17.0 In-Reply-To: <20180522201056.1442-1-pbonzini@redhat.com> References: <20180522201056.1442-1-pbonzini@redhat.com> X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 2607:f8b0:400e:c01::243 Subject: [Qemu-devel] [PATCH 2/2] sheepdog: remove huge BSS object X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: jcody@redhat.com, qemu-block@nongnu.org Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) X-ZohoMail: RDKM_2 RSF_0 Z_629925259 SPT_0 Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" block/sheepdog.o has a 4M static variable that is 90% of QEMU's whole .bss section. Replace it with a heap-allocated block, and make it smaller too since only the inode header is actually being used. bss size goes down from 4464280 to 269976. Signed-off-by: Paolo Bonzini Reviewed-by: Philippe Mathieu-Daud=C3=A9 --- block/sheepdog.c | 21 +++++++++++---------- 1 file changed, 11 insertions(+), 10 deletions(-) diff --git a/block/sheepdog.c b/block/sheepdog.c index 23cf5a8430..2068166e3e 100644 --- a/block/sheepdog.c +++ b/block/sheepdog.c @@ -2940,13 +2940,14 @@ static int sd_snapshot_list(BlockDriverState *bs, Q= EMUSnapshotInfo **psn_tab) QEMUSnapshotInfo *sn_tab =3D NULL; unsigned wlen, rlen; int found =3D 0; - static SheepdogInode inode; + SheepdogInode *inode; unsigned long *vdi_inuse; unsigned int start_nr; uint64_t hval; uint32_t vid; =20 vdi_inuse =3D g_malloc(max); + inode =3D g_malloc(SD_INODE_HEADER_SIZE); =20 fd =3D connect_to_sdog(s, &local_err); if (fd < 0) { @@ -2989,7 +2990,7 @@ static int sd_snapshot_list(BlockDriverState *bs, QEM= USnapshotInfo **psn_tab) } =20 /* we don't need to read entire object */ - ret =3D read_object(fd, s->bs, (char *)&inode, + ret =3D read_object(fd, s->bs, (char *)inode, vid_to_vdi_oid(vid), 0, SD_INODE_HEADER_SIZE, 0, s->cache_flags); @@ -2998,17 +2999,17 @@ static int sd_snapshot_list(BlockDriverState *bs, Q= EMUSnapshotInfo **psn_tab) continue; } =20 - if (!strcmp(inode.name, s->name) && is_snapshot(&inode)) { - sn_tab[found].date_sec =3D inode.snap_ctime >> 32; - sn_tab[found].date_nsec =3D inode.snap_ctime & 0xffffffff; - sn_tab[found].vm_state_size =3D inode.vm_state_size; - sn_tab[found].vm_clock_nsec =3D inode.vm_clock_nsec; + if (!strcmp(inode->name, s->name) && is_snapshot(inode)) { + sn_tab[found].date_sec =3D inode->snap_ctime >> 32; + sn_tab[found].date_nsec =3D inode->snap_ctime & 0xffffffff; + sn_tab[found].vm_state_size =3D inode->vm_state_size; + sn_tab[found].vm_clock_nsec =3D inode->vm_clock_nsec; =20 snprintf(sn_tab[found].id_str, sizeof(sn_tab[found].id_str), - "%" PRIu32, inode.snap_id); + "%" PRIu32, inode->snap_id); pstrcpy(sn_tab[found].name, - MIN(sizeof(sn_tab[found].name), sizeof(inode.tag)), - inode.tag); + MIN(sizeof(sn_tab[found].name), sizeof(inode->tag)), + inode->tag); found++; } } --=20 2.17.0