kvm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Antoine Martin <antoine@nagafix.co.uk>
To: Avi Kivity <avi@redhat.com>
Cc: Michael Tokarev <mjt@tls.msk.ru>,
	Christoph Hellwig <hch@infradead.org>,
	kvm@vger.kernel.org
Subject: Re: raw disks no longer work in latest kvm (kvm-88 was fine)
Date: Sun, 23 May 2010 21:43:01 +0700	[thread overview]
Message-ID: <4BF93EF5.5090106@nagafix.co.uk> (raw)
In-Reply-To: <4BF9391A.1080302@redhat.com>

On 05/23/2010 09:18 PM, Avi Kivity wrote:
> On 05/23/2010 05:07 PM, Antoine Martin wrote:
>> On 05/23/2010 06:57 PM, Avi Kivity wrote:
>>> On 05/23/2010 11:53 AM, Antoine Martin wrote:
>>>> I'm not: 64-bit host and 64-bit guest.
>>>> Just to be sure, I've tested that patch and still no joy:
>>>>   /dev/vdc: read failed after 0 of 512 at 0: Input/output error
>>>>   /dev/vdc: read failed after 0 of 512 at 0: Input/output error
>>>>   /dev/vdc: read failed after 0 of 2048 at 0: Input/output error
>>>> pread is still broken on this big raw disk.
>>>>
>>>
>>> Can you summarize your environment and reproducer again?  The thread 
>>> is so long it is difficult to understand exactly what you are testing.
>>>
>> Sure thing:
>>
>> Description of the problem:
>> A guest tries to mount a large raw partition (1.5TB /dev/sda9 in my 
>> case), this fails with pread enabled, works with it disabled.
>
> Did you mean: preadv?
Yes, here's what makes it work ok (as suggested by Christoph earlier in 
the thread) in posix-aio-compat.c:
#undef CONFIG_PREADV
#ifdef CONFIG_PREADV
static int preadv_present = 1;
#else
static int preadv_present = 0;
#endif
>>
>> What has been tested:
>> Host has had 2.6.31.x and now 2.6.34 - with no improvement.
>> Guest has had numerous kernel versions and patches applied for 
>> testing (from 2.6.31.x to 2.6.34) - none made any difference. 
>> (although the recent patch stopped the large partition from getting 
>> corrupted by the merged requests bug.. yay!)
>> It was once thought that glibc needed to be rebuilt against newer 
>> kernel headers, done that too, and rebuilt qemu-kvm afterwards. (was 
>> 2.6.28 then 2.6.31 and now 2.6.33)
>>
>> What has been reported: straces, kernel error messages as above.
>> Here is the qemu command line (sanitized - removed the virtual disks 
>> which work fine and network config):
>> qemu -clock dynticks -m 256 -L ./ -kernel ./bzImage-2.6.34-aio 
>> -append "root=/dev/vda" -nographic -drive 
>> file=/dev/sda9,if=virtio,cache=none
>>
>> Let me know if you need any other details. I can also arrange ssh 
>> access to the system if needed.
>
> And: 64-bit host kernel, 64-bit host userspace, yes?
Yes. (and 64-bit guest/userspace too)
>
> Does aio=native help?
It does indeed!
>   How about if=ide?
Will test with another kernel and report back (this one doesn't have any 
non-virtio drivers)

Thanks
Antoine

  reply	other threads:[~2010-05-23 14:43 UTC|newest]

Thread overview: 51+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-03-06 20:48 raw disks no longer work in latest kvm (kvm-88 was fine) Antoine Martin
2010-03-06 21:28 ` Michael Tokarev
2010-03-07  4:14   ` Antoine Martin
2010-03-07  9:32     ` Michael Tokarev
2010-03-07 10:00       ` Christoph Hellwig
2010-03-07 13:12         ` Antoine Martin
2010-03-07 13:52           ` Antoine Martin
2010-03-07 15:15             ` Michael Tokarev
2010-03-07 17:11               ` Antoine Martin
2010-03-07 17:18                 ` Avi Kivity
2010-03-07 17:21                   ` Christoph Hellwig
2010-03-07 17:30                     ` Avi Kivity
2010-03-07 17:34                       ` Christoph Hellwig
2010-03-07 18:43                         ` raw disks no longer work in latest kvm (kvm-88 was fine) [SOLVED] Antoine Martin
2010-03-07 18:55                           ` Avi Kivity
2010-03-07 18:01                       ` raw disks no longer work in latest kvm (kvm-88 was fine) Antoine Martin
2010-03-07 18:47                         ` Avi Kivity
2010-03-07 19:07                           ` Antoine Martin
2010-03-07 19:10                             ` Avi Kivity
2010-03-07 19:13                               ` Antoine Martin
2010-03-07 19:17                                 ` Avi Kivity
2010-03-07 19:25                                   ` Antoine Martin
2010-03-07 19:35                                     ` Avi Kivity
2010-03-13  9:51                                       ` Antoine Martin
2010-03-14  7:34                                         ` Avi Kivity
2010-04-08 22:00                                         ` Antoine Martin
2010-05-22 10:44                                           ` Antoine Martin
2010-05-22 11:17                                             ` Michael Tokarev
2010-05-22 11:35                                               ` Antoine Martin
2010-05-23  8:53                                                 ` Antoine Martin
2010-05-23 11:57                                                   ` Avi Kivity
2010-05-23 14:07                                                     ` Antoine Martin
2010-05-23 14:18                                                       ` Avi Kivity
2010-05-23 14:43                                                         ` Antoine Martin [this message]
2010-05-23 14:53                                                           ` Antoine Martin
2010-05-23 14:56                                                             ` Avi Kivity
2010-05-23 15:06                                                           ` Antoine Martin
2010-05-23 15:12                                                           ` Avi Kivity
2010-05-23 16:18                                                             ` Antoine Martin
2010-05-23 17:47                                                               ` Stefan Hajnoczi
2010-05-29  9:42                                                                 ` Antoine Martin
2010-05-29  9:55                                                                   ` Stefan Hajnoczi
2010-05-29 10:34                                                                     ` Christoph Hellwig
2010-05-29 11:06                                                                       ` Stefan Hajnoczi
2010-05-29 10:34                                                                   ` Christoph Hellwig
2010-03-07 19:09                           ` Asdo
2010-03-07 19:11                             ` Antoine Martin
2010-03-07 19:12                             ` Avi Kivity
2010-03-07 16:21         ` Avi Kivity
2010-03-08 16:45           ` Anthony Liguori
2010-03-07  9:36 ` Gleb Natapov

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=4BF93EF5.5090106@nagafix.co.uk \
    --to=antoine@nagafix.co.uk \
    --cc=avi@redhat.com \
    --cc=hch@infradead.org \
    --cc=kvm@vger.kernel.org \
    --cc=mjt@tls.msk.ru \
    /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).