# HG changeset patch # User john.levon@sun.com # Date 1196969481 28800 # Node ID 2c28ac4db1f7542220b1472821e8631b79b6a23a # Parent 4a98d4b6e4daccd610175e3c0f8cab49b056c649 Fix ioemu compile serial.c was including another Linux-specific header. Signed-off-by: John Levon <john.levon@sun.com> diff --git a/tools/ioemu/hw/serial.c b/tools/ioemu/hw/serial.c --- a/tools/ioemu/hw/serial.c +++ b/tools/ioemu/hw/serial.c @@ -26,9 +26,15 @@ #include <sys/time.h> #include <time.h> #include <assert.h> -#include <asm/termios.h> //#define DEBUG_SERIAL + +#define TIOCM_DTR 0x002 +#define TIOCM_RTS 0x004 +#define TIOCM_CTS 0x020 +#define TIOCM_CAR 0x040 +#define TIOCM_RI 0x080 +#define TIOCM_DSR 0x100 #define UART_LCR_DLAB 0x80 /* Divisor latch access bit */ _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
On Thu, Dec 06, 2007 at 11:31:35AM -0800, john.levon@sun.com wrote:> serial.c was including another Linux-specific header. > > Signed-off-by: John Levon <john.levon@sun.com> > > diff --git a/tools/ioemu/hw/serial.c b/tools/ioemu/hw/serial.c > --- a/tools/ioemu/hw/serial.c > +++ b/tools/ioemu/hw/serial.c > @@ -26,9 +26,15 @@ > #include <sys/time.h> > #include <time.h> > #include <assert.h> > -#include <asm/termios.h>Erm, or it looks like just making this be <sys/termios.h> will work too regards john _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
On Thu, Dec 06, 2007 at 07:35:20PM +0000, John Levon wrote:> On Thu, Dec 06, 2007 at 11:31:35AM -0800, john.levon@sun.com wrote: > > > serial.c was including another Linux-specific header. > > > > Signed-off-by: John Levon <john.levon@sun.com> > > > > diff --git a/tools/ioemu/hw/serial.c b/tools/ioemu/hw/serial.c > > --- a/tools/ioemu/hw/serial.c > > +++ b/tools/ioemu/hw/serial.c > > @@ -26,9 +26,15 @@ > > #include <sys/time.h> > > #include <time.h> > > #include <assert.h> > > -#include <asm/termios.h> > > Erm, or it looks like just making this be <sys/termios.h> will work tooDefinitely. Including asm/ headers directly on Linux is frowned upon. Dan. -- |=- Red Hat, Engineering, Emerging Technologies, Boston. +1 978 392 2496 -=| |=- Perl modules: http://search.cpan.org/~danberr/ -=| |=- Projects: http://freshmeat.net/~danielpb/ -=| |=- GnuPG: 7D3B9505 F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 -=| _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Unfortunately, sys/termios.h doesn''t work on linux, at least on RHEL/Fedora. I knew it wasn''t quite kosher to include asm/termios.h when I did it, but it was the only way I could get the termbit defines included. I was going to look for a clean solution, but forgot. It would seem bits/termios.h needs to be extended with the necessary stuff from asm/termios.h, but I''m not sure where/who to suggest that change. On Dec 6, 2007 8:41 PM, Daniel P. Berrange <berrange@redhat.com> wrote:> On Thu, Dec 06, 2007 at 07:35:20PM +0000, John Levon wrote: > > On Thu, Dec 06, 2007 at 11:31:35AM -0800, john.levon@sun.com wrote: > > > > > serial.c was including another Linux-specific header. > > > > > > Signed-off-by: John Levon <john.levon@sun.com> > > > > > > diff --git a/tools/ioemu/hw/serial.c b/tools/ioemu/hw/serial.c > > > --- a/tools/ioemu/hw/serial.c > > > +++ b/tools/ioemu/hw/serial.c > > > @@ -26,9 +26,15 @@ > > > #include <sys/time.h> > > > #include <time.h> > > > #include <assert.h> > > > -#include <asm/termios.h> > > > > Erm, or it looks like just making this be <sys/termios.h> will work too > > Definitely. Including asm/ headers directly on Linux is frowned upon. > > Dan. > -- > |=- Red Hat, Engineering, Emerging Technologies, Boston. +1 978 392 2496 > -=| > |=- Perl modules: http://search.cpan.org/~danberr/<http://search.cpan.org/%7Edanberr/> -=| > |=- Projects: http://freshmeat.net/~danielpb/<http://freshmeat.net/%7Edanielpb/> -=| > |=- GnuPG: 7D3B9505 F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 > -=| > > _______________________________________________ > Xen-devel mailing list > Xen-devel@lists.xensource.com > http://lists.xensource.com/xen-devel >_______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
I checked in John Levon¹s original patch for now, but perhaps <sys/ioctl.h> would work? -- Keir On 7/12/07 09:27, "Trolle Selander" <trolle.selander@gmail.com> wrote:> Unfortunately, sys/termios.h doesn''t work on linux, at least on RHEL/Fedora. I > knew it wasn''t quite kosher to include asm/termios.h when I did it, but it was > the only way I could get the termbit defines included. I was going to look for > a clean solution, but forgot. It would seem bits/termios.h needs to be > extended with the necessary stuff from asm/termios.h, but I''m not sure > where/who to suggest that change. > > On Dec 6, 2007 8:41 PM, Daniel P. Berrange <berrange@redhat.com> wrote: >> On Thu, Dec 06, 2007 at 07:35:20PM +0000, John Levon wrote: >>> > On Thu, Dec 06, 2007 at 11:31:35AM -0800, john.levon@sun.com wrote: >>> > >>>> > > serial.c was including another Linux-specific header. >>>> > > >>>> > > Signed-off-by: John Levon <john.levon@sun.com> >>>> > > >>>> > > diff --git a/tools/ioemu/hw/serial.c b/tools/ioemu/hw/serial.c >>>> > > --- a/tools/ioemu/hw/serial.c >>>> > > +++ b/tools/ioemu/hw/serial.c >>>> > > @@ -26,9 +26,15 @@ >>>> > > #include <sys/time.h> >>>> > > #include <time.h> >>>> > > #include < assert.h> >>>> > > -#include <asm/termios.h> >>> > >>> > Erm, or it looks like just making this be <sys/termios.h> will work too >> >> Definitely. Including asm/ headers directly on Linux is frowned upon. >> >> Dan. >> -- >> |=- Red Hat, Engineering, Emerging Technologies, Boston. +1 978 392 2496 -=| >> |=- Perl modules: http://search.cpan.org/~danberr/ >> <http://search.cpan.org/%7Edanberr/> -=| >> |=- Projects: http://freshmeat.net/~danielpb/ >> <http://freshmeat.net/%7Edanielpb/> -=| >> |=- GnuPG: 7D3B9505 F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 -=| >> >> _______________________________________________ >> Xen-devel mailing list >> Xen-devel@lists.xensource.com >> http://lists.xensource.com/xen-devel >> > > > _______________________________________________ > Xen-devel mailing list > Xen-devel@lists.xensource.com > http://lists.xensource.com/xen-devel_______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
So I propose: #include <termios.h> #include <sys/ioctl.h> I think that will cover all cases. Is everyone happy with that? -- Keir On 7/12/07 11:07, "Keir Fraser" <Keir.Fraser@cl.cam.ac.uk> wrote:> I checked in John Levon¹s original patch for now, but perhaps <sys/ioctl.h> > would work? > > -- Keir > > On 7/12/07 09:27, "Trolle Selander" <trolle.selander@gmail.com> wrote: > >> Unfortunately, sys/termios.h doesn''t work on linux, at least on RHEL/Fedora. >> I knew it wasn''t quite kosher to include asm/termios.h when I did it, but it >> was the only way I could get the termbit defines included. I was going to >> look for a clean solution, but forgot. It would seem bits/termios.h needs to >> be extended with the necessary stuff from asm/termios.h, but I''m not sure >> where/who to suggest that change. >> >> On Dec 6, 2007 8:41 PM, Daniel P. Berrange <berrange@redhat.com> wrote: >>> On Thu, Dec 06, 2007 at 07:35:20PM +0000, John Levon wrote: >>>> > On Thu, Dec 06, 2007 at 11:31:35AM -0800, john.levon@sun.com wrote: >>>> > >>>>> > > serial.c was including another Linux-specific header. >>>>> > > >>>>> > > Signed-off-by: John Levon <john.levon@sun.com> >>>>> > > >>>>> > > diff --git a/tools/ioemu/hw/serial.c b/tools/ioemu/hw/serial.c >>>>> > > --- a/tools/ioemu/hw/serial.c >>>>> > > +++ b/tools/ioemu/hw/serial.c >>>>> > > @@ -26,9 +26,15 @@ >>>>> > > #include <sys/time.h> >>>>> > > #include <time.h> >>>>> > > #include < assert.h> >>>>> > > -#include <asm/termios.h> >>>> > >>>> > Erm, or it looks like just making this be <sys/termios.h> will work too >>> >>> Definitely. Including asm/ headers directly on Linux is frowned upon. >>> >>> Dan. >>> -- >>> |=- Red Hat, Engineering, Emerging Technologies, Boston. +1 978 392 2496 >>> -=| >>> |=- Perl modules: http://search.cpan.org/~danberr/ >>> <http://search.cpan.org/%7Edanberr/> -=| >>> |=- Projects: http://freshmeat.net/~danielpb/ >>> <http://freshmeat.net/%7Edanielpb/> -=| >>> |=- GnuPG: 7D3B9505 F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 >>> -=| >>> >>> _______________________________________________ >>> Xen-devel mailing list >>> Xen-devel@lists.xensource.com >>> http://lists.xensource.com/xen-devel >>> >> >> >> _______________________________________________ >> Xen-devel mailing list >> Xen-devel@lists.xensource.com >> http://lists.xensource.com/xen-devel > > > > _______________________________________________ > Xen-devel mailing list > Xen-devel@lists.xensource.com > http://lists.xensource.com/xen-devel_______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
On Fri, Dec 07, 2007 at 11:17:30AM +0000, Keir Fraser wrote:> So I propose: > #include <termios.h> > #include <sys/ioctl.h> > > I think that will cover all cases. Is everyone happy with that?Good for me regards john _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
On Fri, Dec 07, 2007 at 11:17:30AM +0000, Keir Fraser wrote:> So I propose: > #include <termios.h> > #include <sys/ioctl.h> > > I think that will cover all cases. Is everyone happy with that?Yes, that looks good. Regards, Dan. -- |=- Red Hat, Engineering, Emerging Technologies, Boston. +1 978 392 2496 -=| |=- Perl modules: http://search.cpan.org/~danberr/ -=| |=- Projects: http://freshmeat.net/~danielpb/ -=| |=- GnuPG: 7D3B9505 F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 -=| _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Apparently Analagous Threads
- [PATCH] ioemu-remote: ACPI S3 state wake up
- Re: [Xen-staging] [xen-unstable] [QEMU-DM] Upgrade emulated UART to 16550A.
- Writing a tool for Shared Persistent Windows Boot Image
- Writing a tool for Shared Persistent Windows Boot Image
- [PATCH] [HVM] Fix lmsw handling