Subject: Re: CVS commit: src/sys/arch/xen
To: None <bouyer@netbsd.org>
From: YAMAMOTO Takashi <yamt@mwd.biglobe.ne.jp>
List: source-changes
Date: 08/22/2005 07:50:28
> - implement _xen_bus_dmamem_alloc_range(), which will request a contigous
>   set of pages to the hypervisor if the pages returned by uvm_pglistalloc()
>   don't fit the constraints.

- it seems to leak memory in the case of EINVAL.

- there is no recovery in the case of failure of MEMOP_increase_reservation.
  is this intended?  i think it's better to do MEMOP_decrease_reservation
  only when MEMOP_increase_reservation succeeded.
  MEMOP_decrease_reservation unlikely helps MEMOP_increase_reservation
  anyway.

YAMAMOTO Takashi