From: Jens Axboe <qemu@kernel.dk>
To: Thiemo Seufer <ths@networkno.de>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] qemu/hw ide.c
Date: Mon, 19 Feb 2007 13:04:12 +0100 [thread overview]
Message-ID: <20070219120412.GH17728@kernel.dk> (raw)
In-Reply-To: <20070219115745.GB1857@networkno.de>
On Mon, Feb 19 2007, Thiemo Seufer wrote:
> Jens Axboe wrote:
> > On Mon, Feb 19 2007, Thiemo Seufer wrote:
> > > CVSROOT: /sources/qemu
> > > Module name: qemu
> > > Changes by: Thiemo Seufer <ths> 07/02/19 00:59:34
> > >
> > > Modified files:
> > > hw : ide.c
> > >
> > > Log message:
> > > Ignore special flags in nsector variable.
> > >
> > > CVSWeb URLs:
> > > http://cvs.savannah.gnu.org/viewcvs/qemu/hw/ide.c?cvsroot=qemu&r1=1.53&r2=1.54
> >
> > Why is nsector uint32_t to begin with?
>
> Because nobody sent a patch to fix it, I figure.
Actually I seem to recall it's because it's being overloaded for
requests that are > 256 sectors. It would be a good cleanup to get rid
of that an turn nsector into a proper uint8_t.
--
Jens Axboe
next prev parent reply other threads:[~2007-02-19 12:04 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-02-19 0:59 [Qemu-devel] qemu/hw ide.c Thiemo Seufer
2007-02-19 8:39 ` Jens Axboe
2007-02-19 11:57 ` Thiemo Seufer
2007-02-19 12:04 ` Jens Axboe [this message]
2007-02-19 12:23 ` Thiemo Seufer
2007-02-19 12:24 ` Thiemo Seufer
2007-02-19 12:24 ` Jens Axboe
-- strict thread matches above, loose matches on Subject: below --
2008-03-29 12:15 Aurelien Jarno
2008-02-03 3:37 Andrzej Zaborowski
2008-01-14 3:15 Andrzej Zaborowski
2008-01-14 3:05 Andrzej Zaborowski
2008-05-24 20:05 ` Alexander Graf
2007-12-11 22:15 Andrzej Zaborowski
2007-12-09 5:01 Thiemo Seufer
2007-11-08 16:38 Thiemo Seufer
2007-09-13 12:28 Thiemo Seufer
2007-09-13 2:20 Thiemo Seufer
2007-08-26 17:42 Thiemo Seufer
2007-08-19 21:46 Thiemo Seufer
2007-04-07 1:24 Paul Brook
2007-03-17 14:55 Thiemo Seufer
2007-03-17 14:54 Thiemo Seufer
2007-01-24 1:12 Thiemo Seufer
2007-01-20 1:12 Thiemo Seufer
2006-12-22 2:14 Thiemo Seufer
2006-08-19 11:44 Fabrice Bellard
2006-08-17 10:48 Fabrice Bellard
2006-08-07 21:36 Fabrice Bellard
2006-04-25 21:24 Fabrice Bellard
2006-04-23 22:21 Fabrice Bellard
2006-04-02 19:10 Fabrice Bellard
2006-02-01 22:20 Fabrice Bellard
2005-08-06 9:14 Fabrice Bellard
2005-07-07 21:46 Fabrice Bellard
2005-06-05 15:15 Fabrice Bellard
2005-04-28 19:26 Fabrice Bellard
2004-12-02 20:27 Fabrice Bellard
2004-11-18 7:02 ML
2004-11-17 22:41 Fabrice Bellard
2004-11-09 22:09 Fabrice Bellard
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20070219120412.GH17728@kernel.dk \
--to=qemu@kernel.dk \
--cc=qemu-devel@nongnu.org \
--cc=ths@networkno.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).