public inbox for developer@lists.illumos.org (since 2011-08)
 help / color / mirror / Atom feed
From: "Toomas Soome via illumos-developer" <developer@lists.illumos.org>
To: illumos-developer <developer@lists.illumos.org>
Subject: [developer] REVIEW: 13897 & 16822
Date: Thu, 3 Oct 2024 10:50:48 +0300	[thread overview]
Message-ID: <4A483101-A294-4E24-96E1-899F37B7F69A@me.com> (raw)

[-- Attachment #1: Type: text/plain, Size: 1108 bytes --]

hi!

Please review:
13897 loader: implement mount/unmount rootfs
https://code.illumos.org/c/illumos-gate/+/1564

and

16822 EFI loader: Don't free bcache for DEVT_DISK devs
https://code.illumos.org/c/illumos-gate/+/3726

Those are logically related updates addressing the same problem, which is losing cached data (in loader block cache) as we repeatedly open and close files from the same rootfs and with every open, we need to re-read the file system metadata. The solution there is to keep rootfs “mounted”, that is, keep its metadata till we change  value of “currdev” variable.

Since we do not have integrated the metrics mechanism to make it possible to measure the speedup (referred in the issue #16822), we only can do limited measurements and need to rely on performance data posted with FreeBSD commits.

Thanks,
Toomas
------------------------------------------
illumos: illumos-developer
Permalink: https://illumos.topicbox.com/groups/developer/T6e90cf598f97f7fb-Mff6316b884fefce90ad17e80
Delivery options: https://illumos.topicbox.com/groups/developer/subscription

[-- Attachment #2: Type: text/html, Size: 8355 bytes --]

                 reply	other threads:[~2024-10-03  7:53 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4A483101-A294-4E24-96E1-899F37B7F69A@me.com \
    --to=developer@lists.illumos.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).