Skip to contentSkip to conversation

xen: remove Xencomm

PATCH In review Tim Deegan 3 participants 5 messages Nov 13, 2014 – Nov 14, 2014

lore
Share & tools
lore

Tim Deegan

author
#0 lore raw

PATCH v2 1/1 xen: remove Xencomm

Being a feature that has only been used by ia64 and/or ppc it doesn't seem like we need to keep it any longer in the tree.

So remove it.

Signed-off-by Boris Ostrovsky <[email protected]>
Signed-off-by Tim Deegan <[email protected]>

---
This was suggested by Boris a while ago and seems to have stalled.

v2: leave public header in place (suggested by Jan Beulich).

---
 xen/common/Makefile       |   2 -
 xen/common/xencomm.c      | 621 ----------------------------------------------
 xen/include/Makefile      |   1 -
 xen/include/xen/xencomm.h | 170 -------------
 4 files changed, 794 deletions(-)
 delete mode 100644 xen/common/xencomm.c
 delete mode 100644 xen/include/xen/xencomm.h
xen/common/Makefile +0 -2
index 8391246..1956091 100644
@@ -62,8 +62,6 @@ obj-$(perfc)       += perfc.o
 obj-$(crash_debug) += gdbstub.o
 obj-$(xenoprof)    += xenoprof.o
 
-obj-$(CONFIG_XENCOMM) += xencomm.o
-
 subdir-$(CONFIG_COMPAT) += compat
 
 subdir-$(x86_64) += hvm
xen/common/xencomm.c deleted +0 -621

621 lines not on this page · raw message

xen/include/Makefile +0 -1
index f7ccbc9..94112d1 100644
@@ -21,7 +21,6 @@ headers-y := \
     compat/vcpu.h \
     compat/version.h \
     compat/xen.h \
-    compat/xencomm.h \
     compat/xenoprof.h
 headers-$(CONFIG_X86)     += compat/arch-x86/xen-mca.h
 headers-$(CONFIG_X86)     += compat/arch-x86/xen.h
xen/include/xen/xencomm.h deleted +0 -170

170 lines not on this page · raw message

Signature · 1 line
2.0.1
#2 lore raw
Quoted 10 lines · Jan Beulich
Quoted 8 lines
Quoted 1 line
Quoted 1 line · On 13.11.14 at 17:03, <[email protected]> wrote:

Being a feature that has only been used by ia64 and/or ppc it doesn't seem like we need to keep it any longer in the tree.

So remove it.

Signed-off-by Boris Ostrovsky <[email protected]>
Signed-off-by Tim Deegan <[email protected]>
Acked-by Jan Beulich <[email protected]>

Are folks OK if this is deferred to Xen 4.6?

Thank you!

author
#3 lore raw
Quoted 5 lines · Konrad Rzeszutek Wilk
Quoted 4 lines · Jan Beulich
Quoted 4 lines
Quoted 1 line
Quoted 1 line · On 13.11.14 at 17:03, <[email protected]> wrote:

Being a feature that has only been used by ia64 and/or ppc it doesn't seem like we need to keep it any longer in the tree.

Quoted 8 lines
Quoted 6 lines
Quoted 4 lines

So remove it.

Signed-off-by Boris Ostrovsky <[email protected]>
Signed-off-by Tim Deegan <[email protected]>
Acked-by Jan Beulich <[email protected]>

Are folks OK if this is deferred to Xen 4.6?

Yes, sure. I'll queue it up for committing after the branch.

Tim.

#4 lore raw
In reply to Konrad Rzeszutek Wilk
Quoted 6 lines
Quoted 1 line
Quoted 1 line · On 13.11.14 at 20:18, <[email protected]> wrote:
Quoted 4 lines · Jan Beulich
Quoted 4 lines
Quoted 1 line
Quoted 1 line · On 13.11.14 at 17:03, <[email protected]> wrote:

Being a feature that has only been used by ia64 and/or ppc it doesn't seem like we need to keep it any longer in the tree.

Quoted 8 lines
Quoted 6 lines
Quoted 4 lines

So remove it.

Signed-off-by Boris Ostrovsky <[email protected]>
Signed-off-by Tim Deegan <[email protected]>
Acked-by Jan Beulich <[email protected]>

Are folks OK if this is deferred to Xen 4.6?

Sure, albeit I can't resist to say that the change is zero risk.

Jan