Project

General

Profile

Actions
Copy link

Bug #14493

closed

Three small memory leaks in bhyve

Added by Andy Fiddaman 6 months ago. Updated 3 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Andy Fiddaman
Category:
bhyve
Start date:
Due date:
% Done:

100%

Estimated time:
Difficulty:
Bite-size
Tags:
Gerrit CR:
2072

Description

umem_alloc_16 leak: 1 buffer, 16 bytes
            ADDR          BUFADDR        TIMESTAMP           THREAD
                            CACHE          LASTLOG         CONTENTS
          81e460           817fd0     570a4304d986                1
                           7d6668           79aa80                0
                 libumem.so.1`umem_cache_alloc_debug+0x213
                 libumem.so.1`umem_cache_alloc+0xe3
                 libumem.so.1`umem_alloc+0x9f
                 libumem.so.1`umem_malloc+0x46
                 libc.so.1`strdup+0x25
                 topology_parse+0x2b
                 main+0x1c5
                 _start_crt+0x87
                 _start+0x18

umem_alloc_32 leak: 1 buffer, 32 bytes
            ADDR          BUFADDR        TIMESTAMP           THREAD
                            CACHE          LASTLOG         CONTENTS
          8339a0           82dfc0     570a601fcddd                1
                           7d8028           7a4a80                0
                 libumem.so.1`umem_cache_alloc_debug+0x213
                 libumem.so.1`umem_cache_alloc+0xe3
                 libumem.so.1`umem_alloc+0x9f
                 libumem.so.1`umem_malloc+0x46
                 libc.so.1`strdup+0x25
                 uart_sock_backend+0x46
                 uart_set_backend+0x60
                 lpc_init+0xcb
                 pci_lpc_init+0x30
                 pci_emul_init+0xd3
                 init_pci+0x32f
                 main+0x4ed
                 _start_crt+0x87
                 _start+0x18

umem_alloc_64 leak: 1 buffer, 64 bytes
            ADDR          BUFADDR        TIMESTAMP           THREAD
                            CACHE          LASTLOG         CONTENTS
          828b60           826f80     570a60076310                1
                           7db028           7a4480                0
                 libumem.so.1`umem_cache_alloc_debug+0x213
                 libumem.so.1`umem_cache_alloc+0xe3
                 libumem.so.1`umem_alloc+0x9f
                 libumem.so.1`umem_malloc+0x46
                 libc.so.1`strdup+0x25
                 bootrom_loadrom+0x2e
                 lpc_init+0x48
                 pci_lpc_init+0x30
                 pci_emul_init+0xd3
                 init_pci+0x32f
                 main+0x4ed
                 _start_crt+0x87
                 _start+0x18

Related issues

Related to illumos gate - Feature #14578: bhyve upstream sync 2022 MarchClosedAndy Fiddaman

Actions
Actions
Copy link
 #1

Updated by Andy Fiddaman 5 months ago

Actions
Copy link
 #2

Updated by Andy Fiddaman 5 months ago

Two of these are now fixed in upstream bhyve and will be incorporated in #14578
The third, in the UART code, is illumos-specific.

Actions
Copy link
 #3

Updated by Electric Monk 5 months ago

  • Gerrit CR set to 2072
Actions
Copy link
 #4

Updated by Electric Monk 3 months ago

  • Status changed from In Progress to Closed
  • % Done changed from 0 to 100

git commit d7b72f7b52f902da47cc7210a9121f4caabbcb9c

commit  d7b72f7b52f902da47cc7210a9121f4caabbcb9c
Author: Andy Fiddaman <omnios@citrus-it.co.uk>
Date:   2022-05-12T18:30:12.000Z

    14578 bhyve upstream sync 2022 March
    14493 Three small memory leaks in bhyve
    Reviewed by: Patrick Mooney <pmooney@pfmooney.com>
    Approved by: Dan McDonald <danmcd@mnx.io>

Actions
Copy link

Also available in: Atom PDF