[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 01/13] libxl/x86: check return value of SHADOW_OP_SET_ALLOCATION domctl
- To: Jan Beulich <jbeulich@xxxxxxxx>
- From: Anthony PERARD <anthony.perard@xxxxxxxxxx>
- Date: Fri, 16 Jul 2021 14:03:49 +0100
- Authentication-results: esa6.hc3370-68.iphmx.com; dkim=none (message not signed) header.i=none
- Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, "Andrew Cooper" <andrew.cooper3@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, Juergen Gross <jgross@xxxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>, Ian Jackson <iwj@xxxxxxxxxxxxxx>
- Delivery-date: Fri, 16 Jul 2021 13:03:56 +0000
- Ironport-hdrordr: A9a23:ROKLYaC4aPXyfVjlHelW55DYdb4zR+YMi2TDt3oddfWaSKylfq GV7ZAmPHrP4gr5N0tOpTntAse9qBDnhPtICOsqTNSftWDd0QPFEGgL1+DfKlbbak/DH4BmtJ uJc8JFeaDN5VoRt7eH3OFveexQv+Vu88qT9JnjJ28Gd3AMV0n5hT0JcTpyFCdNNW97LKt8Lr WwzOxdqQGtfHwGB/7LfEXsD4D41qT2fIuNW29/OyIa
- Ironport-sdr: K1DqCOEy5/NlMJ9XKUN/sy+j2xrU2L6umDSH7WUAFEN+u4DzPemx7iEjnh32FLYtalDn5uRZGQ Ly9LICzu0f1+KC9B1pqhvpoZ3mOKUyRDGir+0bFEOdDqvkjyuC2cd3o/Sjqa0wVbfeiMZFSUkh GvChFKg2oFaihzfKMdQq7syVPtC3o3BPh3XEDeVoiLc+gZFiHVhj0XPSX4OgHOHdizDejDPKja ++A1Pa9y/ve5K/8fFdS9mgAMtmgWre486IGZ4r+Q/ir3ObD8gNfQdw2L4VibsMpgWl7GiHAtvk 5Bg=
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On Mon, Jul 05, 2021 at 05:11:31PM +0200, Jan Beulich wrote:
> The hypervisor may not have enough memory to satisfy the request. While
> there, make the unit of the value clear by renaming the local variable.
>
> Requested-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
> Reviewed-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
> ---
> v2: Rename local variables. Also log requested value. Don't log errno
> explicitly in LOGED() (nor function return value). Integrate into
> series.
Reviewed-by: Anthony PERARD <anthony.perard@xxxxxxxxxx>
Thanks,
--
Anthony PERARD
|
