qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Daniel P. Berrangé" <berrange@redhat.com>
To: Fiona Ebner <f.ebner@proxmox.com>
Cc: qemu-devel@nongnu.org, pbonzini@redhat.com, sw@weilnetz.de,
	t.lamprecht@proxmox.com
Subject: Re: [PATCH v2] qemu_init: increase NOFILE soft limit on POSIX
Date: Mon, 5 Feb 2024 10:27:04 +0000	[thread overview]
Message-ID: <ZcC3-G_COqbk8SxX@redhat.com> (raw)
In-Reply-To: <a912519d-e83c-460b-ab89-931e83e2ee44@proxmox.com>

On Fri, Feb 02, 2024 at 09:35:19AM +0100, Fiona Ebner wrote:
> Am 18.12.23 um 11:13 schrieb Fiona Ebner:
> > In many configurations, e.g. multiple vNICs with multiple queues or
> > with many Ceph OSDs, the default soft limit of 1024 is not enough.
> > QEMU is supposed to work fine with file descriptors >= 1024 and does
> > not use select() on POSIX. Bump the soft limit to the allowed hard
> > limit to avoid issues with the aforementioned configurations.
> > 
> > Of course the limit could be raised from the outside, but the man page
> > of systemd.exec states about 'LimitNOFILE=':
> > 
> >> Don't use.
> >> [...]
> >> Typically applications should increase their soft limit to the hard
> >> limit on their own, if they are OK with working with file
> >> descriptors above 1023,
> > 
> > If the soft limit is already the same as the hard limit, avoid the
> > superfluous setrlimit call. This can avoid a warning with a strict
> > seccomp filter blocking setrlimit if NOFILE was already raised before
> > executing QEMU.
> > 
> > Buglink: https://bugzilla.proxmox.com/show_bug.cgi?id=4507
> > Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
> > ---
> > 
> 
> Ping

I have this queued already for my next pull request

With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|



      reply	other threads:[~2024-02-05 10:27 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-12-18 10:13 [PATCH v2] qemu_init: increase NOFILE soft limit on POSIX Fiona Ebner
2024-01-03 12:19 ` Daniel P. Berrangé
2024-02-02  8:35 ` Fiona Ebner
2024-02-05 10:27   ` Daniel P. Berrangé [this message]

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=ZcC3-G_COqbk8SxX@redhat.com \
    --to=berrange@redhat.com \
    --cc=f.ebner@proxmox.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=sw@weilnetz.de \
    --cc=t.lamprecht@proxmox.com \
    /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).