* GNTCOPY_can_fail ???
@ 2013-06-07 9:23 Jan Beulich
2013-06-07 11:20 ` Keir Fraser
0 siblings, 1 reply; 2+ messages in thread
From: Jan Beulich @ 2013-06-07 9:23 UTC (permalink / raw)
To: xen-devel
Just noticed that apart from getting defined, this is completely
unused in both Xen and Linux. Does anyone know what the
intended purpose was? Unless we implement that intention,
shouldn't we at least mark it in the public header as unusable
at present, or even remove the visibility of the definitions for
new enough __XEN_INTERFACE_VERSION__?
Jan
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: GNTCOPY_can_fail ???
2013-06-07 9:23 GNTCOPY_can_fail ??? Jan Beulich
@ 2013-06-07 11:20 ` Keir Fraser
0 siblings, 0 replies; 2+ messages in thread
From: Keir Fraser @ 2013-06-07 11:20 UTC (permalink / raw)
To: Jan Beulich, xen-devel
On 07/06/2013 10:23, "Jan Beulich" <JBeulich@suse.com> wrote:
> Just noticed that apart from getting defined, this is completely
> unused in both Xen and Linux. Does anyone know what the
> intended purpose was? Unless we implement that intention,
> shouldn't we at least mark it in the public header as unusable
> at present, or even remove the visibility of the definitions for
> new enough __XEN_INTERFACE_VERSION__?
I think the definition leaked in from early work on xenpaging, on the guest
side, and never actually been used by a non-developer's guest kernel. I'd be
happy to see it removed completely and unconditionally.
-- Keir
> Jan
>
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-06-07 11:20 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-06-07 9:23 GNTCOPY_can_fail ??? Jan Beulich
2013-06-07 11:20 ` Keir Fraser
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).