From a1476ce4f5ee04d40bdff4fad516022862a7d9c5 Mon Sep 17 00:00:00 2001 From: Sergey Madaminov Date: Tue, 8 Nov 2022 14:00:59 -0800 Subject: [PATCH] io_uring_setup: fixing cqentries setup bug. We should compare against maximum number of allowed cq entries instead of comparing against the flag that allows to set that number. PiperOrigin-RevId: 487045476 --- pkg/sentry/fsimpl/iouringfs/iouringfs.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/sentry/fsimpl/iouringfs/iouringfs.go b/pkg/sentry/fsimpl/iouringfs/iouringfs.go index 13a12d764..a29e23732 100644 --- a/pkg/sentry/fsimpl/iouringfs/iouringfs.go +++ b/pkg/sentry/fsimpl/iouringfs/iouringfs.go @@ -94,7 +94,7 @@ func New(ctx context.Context, vfsObj *vfs.VirtualFilesystem, entries uint32, par if params.Flags&linux.IORING_SETUP_CQSIZE != 0 { var ok bool numCqEntries, ok = roundUpPowerOfTwo(params.CqEntries) - if !ok || numCqEntries < numSqEntries || numCqEntries > linux.IORING_SETUP_CQSIZE { + if !ok || numCqEntries < numSqEntries || numCqEntries > linux.IORING_MAX_CQ_ENTRIES { return nil, linuxerr.EINVAL } } else {