Re: [Xen-devel] [PATCH v2 2/2] libvchan: replace munmap with correct xc_gntshr_munmap

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

I can't seem to find the 1/2 patch from this series anywhere?

On Sat, 2013-05-04 at 23:10 +0100, Marek Marczykowski wrote:
> On linux it will end up in munmap anyway, but do not assume any
> particular xc_gntshr_munmap implementation details.
> 
> Signed-off-by: Marek Marczykowski <marmarek@xxxxxxxxxxxxxxxxxxxxxx>
> ---
>  tools/libvchan/io.c | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/tools/libvchan/io.c b/tools/libvchan/io.c
> index 5ec5fb9..3040099 100644
> --- a/tools/libvchan/io.c
> +++ b/tools/libvchan/io.c
> @@ -324,9 +324,11 @@ void libxenvchan_close(struct libxenvchan *ctrl)
>       if (!ctrl)
>               return;
>       if (ctrl->read.order >= PAGE_SHIFT)
> -             munmap(ctrl->read.buffer, 1 << ctrl->read.order);
> +             xc_gntshr_munmap(ctrl->gntshr, ctrl->read.buffer,
> +                             1 << (ctrl->read.order - PAGE_SHIFT));
>       if (ctrl->write.order >= PAGE_SHIFT)
> -             munmap(ctrl->write.buffer, 1 << ctrl->write.order);
> +             xc_gntshr_munmap(ctrl->gntshr, ctrl->write.buffer,
> +                             1 << (ctrl->write.order - PAGE_SHIFT));
>       if (ctrl->ring) {
>               if (ctrl->is_server) {
>                       ctrl->ring->srv_live = 0;



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel