qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Thomas Huth <304636@bugs.launchpad.net>
To: qemu-devel@nongnu.org
Subject: [Bug 304636] Re: -hda FAT:. limited to 504MBytes
Date: Tue, 09 Feb 2021 14:10:50 -0000	[thread overview]
Message-ID: <161287985051.32649.84804766016914672.malone@gac.canonical.com> (raw)
In-Reply-To: 20081203025459.23099.79789.malonedeb@gangotri.canonical.com

Pedro,
please also note that vvfat driver is general in a bad state and more or less completely unmaintaind. I can only strongly recommend to *not* use it in production. If you have to share files between guest and host, please use something more modern like virtio-fs (or maybe virtio-9p) instead.

-- 
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/304636

Title:
  -hda FAT:. limited to 504MBytes

Status in QEMU:
  Confirmed
Status in qemu package in Ubuntu:
  Won't Fix

Bug description:
  Binary package hint: qemu

  The size of the virtual FAT file system (for sharing a particular directory with Guest OS) is hard-coded to be limited to 504MBytes, in block-vvfat.c
  --
  /* 504MB disk*/
  bs->cyls=1024; bs->heads=16; bs->secs=63;
  --

  If the directory contents exceeds this is stops with an assert
  --
  qemu: block-vvfat.c:97: array_get: Assertion `index < array->next' failed.
  Aborted
  --

  Also the FAT16 mode (default) only uses 8KByte cluster sizes which prevents the above being increased. 16KByte and 32KByte sectors can be selected with the following patch
  --
  --- block-vvfat.c_orig  2008-12-02 12:37:28.000000000 -0700
  +++ block-vvfat.c       2008-12-02 19:50:35.000000000 -0700
  @@ -1042,6 +1042,12 @@
          s->fat_type = 32;
       } else if (strstr(dirname, ":16:")) {
          s->fat_type = 16;
  +    } else if (strstr(dirname, ":16-16K:")) {
  +       s->fat_type = 16;
  +       s->sectors_per_cluster=0x20;
  +    } else if (strstr(dirname, ":16-32K:")) {
  +       s->fat_type = 16;
  +       s->sectors_per_cluster=0x40;
       } else if (strstr(dirname, ":12:")) {
          s->fat_type = 12;
          s->sector_count=2880;
  --

  Cheers,
  Mungewell

To manage notifications about this bug go to:
https://bugs.launchpad.net/qemu/+bug/304636/+subscriptions


  parent reply	other threads:[~2021-02-09 14:26 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20081203025459.23099.79789.malonedeb@gangotri.canonical.com>
2021-02-07  0:33 ` [Bug 304636] Re: -hda FAT:. limited to 504MBytes Pedro Pablo Lopez Rodriguez
2021-02-08 22:20 ` Bryce Harrington
2021-02-09 14:10 ` Thomas Huth [this message]
2021-02-09 14:29 ` Daniel Berrange
2021-04-20 16:00 ` Thomas Huth
2021-05-01  8:25 ` Thomas Huth

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=161287985051.32649.84804766016914672.malone@gac.canonical.com \
    --to=304636@bugs.launchpad.net \
    --cc=qemu-devel@nongnu.org \
    /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).