Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 24 May 2018 16:06:36 +0200
From:      Roger Pau =?utf-8?B?TW9ubsOp?= <roger.pau@citrix.com>
To:        Pratyush Yadav <pratyush@freebsd.org>
Cc:        FreeBSD-Xen <freebsd-xen@freebsd.org>
Subject:   Re: Leftover code in gnttab.h?
Message-ID:  <20180524140636.7vz2gjshqow6efwe@MBP-de-Roger.citrite.net>
In-Reply-To: <CA%2BX=3TJ0OubDVenkjC0CQnv9Cqc64cdXzwbO=qY1ZSq3dLXG6Q@mail.gmail.com>
References:  <CA%2BX=3TJ0OubDVenkjC0CQnv9Cqc64cdXzwbO=qY1ZSq3dLXG6Q@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, May 24, 2018 at 05:53:02PM +0530, Pratyush Yadav wrote:
> Hi everyone,
> 
> I was reading the Xen grant table code and I noticed that some code is
> wrapped in an #if 0. You can see it in sys/xen/gnttab.h:138. I have
> also attached the "commented out" part down below.
> 
> Is it useful? Is it all right if I submit a patch that removes it?

This are leftovers from when the code was imported from Linux AFAICT.
I guess the original committer thought that those functions would be
implemented, so the prototypes where left in place.

If you give me a patch to remove them I'm happy to commit it.

Roger.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20180524140636.7vz2gjshqow6efwe>