[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [win-pv-devel] [PATCH] Dont log CacheCreateSlab failures due to the cap


  • To: Owen Smith <owen.smith@xxxxxxxxxx>, "win-pv-devel@xxxxxxxxxxxxxxxxxxxx" <win-pv-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Paul Durrant <Paul.Durrant@xxxxxxxxxx>
  • Date: Wed, 10 Jul 2019 19:00:18 +0000
  • Accept-language: en-GB, en-US
  • Authentication-results: esa3.hc3370-68.iphmx.com; dkim=none (message not signed) header.i=none; spf=None smtp.pra=Paul.Durrant@xxxxxxxxxx; spf=Pass smtp.mailfrom=Paul.Durrant@xxxxxxxxxx; spf=None smtp.helo=postmaster@xxxxxxxxxxxxxxx
  • Cc: Owen Smith <owen.smith@xxxxxxxxxx>
  • Delivery-date: Wed, 10 Jul 2019 19:00:27 +0000
  • Ironport-sdr: Ut3Z8aiV6wIxfg5oQ5SCRft2xzTCdswSCw5W4uf7bNJjyAArSXaB6TYzFl/pgsrNe2B+1NyNrE mZrcMR6uaKiOoPY1SM09UDFkD8limwaBXeBxXKJQShk70qj6zdsNK+HX3LtJus5LGzO7MOvHwU W6ukSvS5HKbY4l3A8QdrXNFTh8Zd5a43AtqD/ew9Wc0NNPVJ1Xwkb+h30KMTPbGGXqzgZH9D9R z4YP+xwxhhuChXVUP9BaliRjnxL/uu8lOjkojOuidij1XvPiYaoOR4+HHniRwog87pnFJL1YoE Zko=
  • List-id: Developer list for the Windows PV Drivers subproject <win-pv-devel.lists.xenproject.org>
  • Thread-index: AQHVNZrMLx4U9YfDx0uPCFvIlyjqc6bEN5cg
  • Thread-topic: [win-pv-devel] [PATCH] Dont log CacheCreateSlab failures due to the cap

> -----Original Message-----
> From: win-pv-devel <win-pv-devel-bounces@xxxxxxxxxxxxxxxxxxxx> On Behalf Of 
> Owen Smith
> Sent: 08 July 2019 09:38
> To: win-pv-devel@xxxxxxxxxxxxxxxxxxxx
> Cc: Owen Smith <owen.smith@xxxxxxxxxx>
> Subject: [win-pv-devel] [PATCH] Dont log CacheCreateSlab failures due to the 
> cap
> 
> Signed-off-by: Owen Smith <owen.smith@xxxxxxxxxx>
> ---
>  src/xenbus/cache.c | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/src/xenbus/cache.c b/src/xenbus/cache.c
> index 7daf8da..4687241 100644
> --- a/src/xenbus/cache.c
> +++ b/src/xenbus/cache.c
> @@ -313,11 +313,9 @@ fail3:
>      MmFreeContiguousMemory(Slab);
> 
>  fail2:
> -    Error("fail2\n");
> +    Error("fail2 (%08x)\n", status);
> 
>  fail1:
> -    Error("fail1 (%08x)\n", status);
> -

Actually I'd prefer to log squash on the basis of the status, which is 
STATUS_INSUFFICIENT_RESOURCES in the case of hitting the cap vs. 
STATUS_NO_MEMORY in the case of an actual out-of-memory condition.

  Paul

>      return NULL;
>  }
> 
> --
> 2.16.2.windows.1
> 
> 
> _______________________________________________
> win-pv-devel mailing list
> win-pv-devel@xxxxxxxxxxxxxxxxxxxx
> https://lists.xenproject.org/mailman/listinfo/win-pv-develSTATUS_NO_MEMORY
_______________________________________________
win-pv-devel mailing list
win-pv-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/win-pv-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.