[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] bdrv_flush error handling
From: |
Daniel P. Berrange |
Subject: |
Re: [Qemu-devel] [PATCH] bdrv_flush error handling |
Date: |
Thu, 21 Feb 2008 18:24:14 +0000 |
User-agent: |
Mutt/1.4.1i |
On Thu, Feb 21, 2008 at 11:28:23AM -0600, address@hidden wrote:
> On Thu, Feb 21, 2008 at 05:24:10PM +0000, Daniel P. Berrange wrote:
> > On Thu, Feb 21, 2008 at 12:19:22PM -0500, Ben Taylor wrote:
> > > > > Also, in block-raw-posix.c, raw_pwrite et al seem to return -1 on
> > > > > error (the return value from write) whereas the other block read/write
> > > > > methods return errno values. This is a mistake, surely ? -1 would be
> > > > > -EPERM. If any of the callers did anything with these return values
> > > > > you'd get incorrect error indications.
> > > > >
> > > > >
> > > > > Finally, it would perhaps be best if the block device emulators wrote
> > > > > to the qemu console to complain if they give write errors. Otherwise
> > > > > the errno value and other important information will be lost, which
> > > > > makes debugging hard.
> > > >
> > > > If by 'qemu console' you mean stderr, then fine, but please don't
> > > > spew log messages to the monitor console, because that'll make it
> > > > very hard to interact with reliably from management tools.
> > >
> > > Would it make sense to have a log messages screen associated with
> > > the monitor (like Ctrl-Alt-7) to deal with those sorts of things?
> >
> > Why invent a new special QEMU log screen, when stderr works just fine. If an
> > app wants to capture log messages they just capture stderr and persist it.
> > We already capture stderr for exactly this reason in libvirt when managing
> > QEMU instances.
>
> Please excuse my intrusion in this thread, but I'm a user of the new
> ncurses user interface. when ssh'd in, running qemu, I don't believe
> having messages pop out of stderr and over the current screen contents is
> the appropriate behavior, as it sounds to me like it would cause redraw
> defects in the normal text console (via ncurses)
So just redirect stderr to a logfile, or /dev/null, etc, etc
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 -=|
Re: [Qemu-devel] [PATCH] bdrv_flush error handling, Ben Taylor, 2008/02/21