From patchwork Fri Apr 12 14:38:21 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stefan Hajnoczi X-Patchwork-Id: 236125 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id 4B3872C00B6 for ; Sat, 13 Apr 2013 00:41:53 +1000 (EST) Received: from localhost ([::1]:39438 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UQfAl-0007Fc-J8 for incoming@patchwork.ozlabs.org; Fri, 12 Apr 2013 10:41:51 -0400 Received: from eggs.gnu.org ([208.118.235.92]:38708) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UQf7c-0003xa-8E for qemu-devel@nongnu.org; Fri, 12 Apr 2013 10:38:40 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UQf7X-0004NF-O6 for qemu-devel@nongnu.org; Fri, 12 Apr 2013 10:38:36 -0400 Received: from mx1.redhat.com ([209.132.183.28]:50223) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UQf7X-0004Ml-7W for qemu-devel@nongnu.org; Fri, 12 Apr 2013 10:38:31 -0400 Received: from int-mx11.intmail.prod.int.phx2.redhat.com (int-mx11.intmail.prod.int.phx2.redhat.com [10.5.11.24]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id r3CEcUjF011009 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK); Fri, 12 Apr 2013 10:38:30 -0400 Received: from localhost (ovpn-112-42.ams2.redhat.com [10.36.112.42]) by int-mx11.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id r3CEcT5e024601; Fri, 12 Apr 2013 10:38:29 -0400 From: Stefan Hajnoczi To: Date: Fri, 12 Apr 2013 16:38:21 +0200 Message-Id: <1365777502-7035-4-git-send-email-stefanha@redhat.com> In-Reply-To: <1365777502-7035-1-git-send-email-stefanha@redhat.com> References: <1365777502-7035-1-git-send-email-stefanha@redhat.com> X-Scanned-By: MIMEDefang 2.68 on 10.5.11.24 X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x X-Received-From: 209.132.183.28 Cc: Anthony Liguori Subject: [Qemu-devel] [PATCH 3/4] Typo, spelling and grammatical fixes X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Sender: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org From: Peter Maydell Minor fixes to documentation and code comments. Signed-off-by: Peter Maydell Signed-off-by: Stefan Hajnoczi --- docs/bootindex.txt | 10 +++++----- docs/ccid.txt | 2 +- hw/block/virtio-blk.c | 4 ++-- hw/intc/armv7m_nvic.c | 2 +- hw/s390x/s390-virtio-ccw.c | 2 +- hw/s390x/s390-virtio.c | 2 +- hw/sd/sd.c | 4 ++-- target-microblaze/mmu.c | 2 +- target-s390x/mem_helper.c | 8 ++++---- 9 files changed, 18 insertions(+), 18 deletions(-) diff --git a/docs/bootindex.txt b/docs/bootindex.txt index 803ebfc..f84fac7 100644 --- a/docs/bootindex.txt +++ b/docs/bootindex.txt @@ -9,7 +9,7 @@ still be bootable. == Example == -Lets assume we have QEMU machine with two NICs (virtio, e1000) and two +Let's assume we have a QEMU machine with two NICs (virtio, e1000) and two disks (IDE, virtio): qemu -drive file=disk1.img,if=none,id=disk1 @@ -20,7 +20,7 @@ qemu -drive file=disk1.img,if=none,id=disk1 -netdev type=user,id=net1 -device e1000,netdev=net1,bootindex=1 Given the command above, firmware should try to boot from the e1000 NIC -first. If this fails, it should try the virtio NIC next, if this fails +first. If this fails, it should try the virtio NIC next; if this fails too, it should try the virtio disk, and then the IDE disk. == Limitations == @@ -28,7 +28,7 @@ too, it should try the virtio disk, and then the IDE disk. 1. Some firmware has limitations on which devices can be considered for booting. For instance, the PC BIOS boot specification allows only one disk to be bootable. If boot from disk fails for some reason, the BIOS -won't retry booting from other disk. It still can try to boot from +won't retry booting from other disk. It can still try to boot from floppy or net, though. 2. Sometimes, firmware cannot map the device path QEMU wants firmware to @@ -36,8 +36,8 @@ boot from to a boot method. It doesn't happen for devices the firmware can natively boot from, but if firmware relies on an option ROM for booting, and the same option ROM is used for booting from more then one device, the firmware may not be able to ask the option ROM to boot from -a particular device reliably. For instance with PC BIOS, if a SCSI HBA +a particular device reliably. For instance with the PC BIOS, if a SCSI HBA has three bootable devices target1, target3, target5 connected to it, the option ROM will have a boot method for each of them, but it is not possible to map from boot method back to a specific target. This is a -shortcoming of PC BIOS boot specification. +shortcoming of the PC BIOS boot specification. diff --git a/docs/ccid.txt b/docs/ccid.txt index 450a66a..8bbaa94 100644 --- a/docs/ccid.txt +++ b/docs/ccid.txt @@ -24,7 +24,7 @@ information see the specification: Revision 1.1 April 22rd, 2005 -Smartcard are used for authentication, single sign on, decryption in +Smartcards are used for authentication, single sign on, decryption in public/private schemes and digital signatures. A smartcard reader on the client cannot be used on a guest with simple usb passthrough since it will then not be available on the client, possibly locking the computer when it is "removed". On diff --git a/hw/block/virtio-blk.c b/hw/block/virtio-blk.c index 6efb2f0..9fdf009 100644 --- a/hw/block/virtio-blk.c +++ b/hw/block/virtio-blk.c @@ -494,10 +494,10 @@ static void virtio_blk_update_config(VirtIODevice *vdev, uint8_t *config) blkcfg.heads = s->conf->heads; /* * We must ensure that the block device capacity is a multiple of - * the logical block size. If that is not the case, lets use + * the logical block size. If that is not the case, let's use * sector_mask to adopt the geometry to have a correct picture. * For those devices where the capacity is ok for the given geometry - * we dont touch the sector value of the geometry, since some devices + * we don't touch the sector value of the geometry, since some devices * (like s390 dasd) need a specific value. Here the capacity is already * cyls*heads*secs*blk_size and the sector value is not block size * divided by 512 - instead it is the amount of blk_size blocks diff --git a/hw/intc/armv7m_nvic.c b/hw/intc/armv7m_nvic.c index 566b4bf..67dbe68 100644 --- a/hw/intc/armv7m_nvic.c +++ b/hw/intc/armv7m_nvic.c @@ -172,7 +172,7 @@ static uint32_t nvic_readl(nvic_state *s, uint32_t offset) return 10000; case 0xd00: /* CPUID Base. */ return cpu_single_env->cp15.c0_cpuid; - case 0xd04: /* Interrypt Control State. */ + case 0xd04: /* Interrupt Control State. */ /* VECTACTIVE */ val = s->gic.running_irq[0]; if (val == 1023) { diff --git a/hw/s390x/s390-virtio-ccw.c b/hw/s390x/s390-virtio-ccw.c index 76b63e2..a49e440 100644 --- a/hw/s390x/s390-virtio-ccw.c +++ b/hw/s390x/s390-virtio-ccw.c @@ -76,7 +76,7 @@ static void ccw_init(QEMUMachineInitArgs *args) } my_ram_size = my_ram_size >> (20 + shift) << (20 + shift); - /* lets propagate the changed ram size into the global variable. */ + /* let's propagate the changed ram size into the global variable. */ ram_size = my_ram_size; /* get a BUS */ diff --git a/hw/s390x/s390-virtio.c b/hw/s390x/s390-virtio.c index f82c0e1..46aec99 100644 --- a/hw/s390x/s390-virtio.c +++ b/hw/s390x/s390-virtio.c @@ -240,7 +240,7 @@ static void s390_init(QEMUMachineInitArgs *args) } my_ram_size = my_ram_size >> (20 + shift) << (20 + shift); - /* lets propagate the changed ram size into the global variable. */ + /* let's propagate the changed ram size into the global variable. */ ram_size = my_ram_size; /* get a BUS */ diff --git a/hw/sd/sd.c b/hw/sd/sd.c index 66c4014..2e0ef3e 100644 --- a/hw/sd/sd.c +++ b/hw/sd/sd.c @@ -308,7 +308,7 @@ static void sd_set_csd(SDState *sd, uint64_t size) sd->csd[13] = 0x40; sd->csd[14] = 0x00; sd->csd[15] = 0x00; - sd->ocr |= 1 << 30; /* High Capacity SD Memort Card */ + sd->ocr |= 1 << 30; /* High Capacity SD Memory Card */ } } @@ -1541,7 +1541,7 @@ void sd_write_data(SDState *sd, uint8_t value) case 25: /* CMD25: WRITE_MULTIPLE_BLOCK */ if (sd->data_offset == 0) { - /* Start of the block - lets check the address is valid */ + /* Start of the block - let's check the address is valid */ if (sd->data_start + sd->blk_len > sd->size) { sd->card_status |= ADDRESS_ERROR; break; diff --git a/target-microblaze/mmu.c b/target-microblaze/mmu.c index 53ad263..73bf805 100644 --- a/target-microblaze/mmu.c +++ b/target-microblaze/mmu.c @@ -114,7 +114,7 @@ unsigned int mmu_translate(struct microblaze_mmu *mmu, tlb_ex = d & TLB_EX; tlb_wr = d & TLB_WR; - /* Now lets see if there is a zone that overrides the protbits. */ + /* Now let's see if there is a zone that overrides the protbits. */ tlb_zsel = (d >> 4) & 0xf; t0 = mmu->regs[MMU_R_ZPR] >> (30 - (tlb_zsel * 2)); t0 &= 0x3; diff --git a/target-s390x/mem_helper.c b/target-s390x/mem_helper.c index 372334b..02bc432 100644 --- a/target-s390x/mem_helper.c +++ b/target-s390x/mem_helper.c @@ -344,7 +344,7 @@ uint64_t HELPER(srst)(CPUS390XState *env, uint64_t r0, uint64_t end, env->retxl = str; /* Lest we fail to service interrupts in a timely manner, limit the - amount of work we're willing to do. For now, lets cap at 8k. */ + amount of work we're willing to do. For now, let's cap at 8k. */ for (len = 0; len < 0x2000; ++len) { if (str + len == end) { /* Character not found. R1 & R2 are unmodified. */ @@ -375,7 +375,7 @@ uint64_t HELPER(clst)(CPUS390XState *env, uint64_t c, uint64_t s1, uint64_t s2) s2 = fix_address(env, s2); /* Lest we fail to service interrupts in a timely manner, limit the - amount of work we're willing to do. For now, lets cap at 8k. */ + amount of work we're willing to do. For now, let's cap at 8k. */ for (len = 0; len < 0x2000; ++len) { uint8_t v1 = cpu_ldub_data(env, s1 + len); uint8_t v2 = cpu_ldub_data(env, s2 + len); @@ -424,7 +424,7 @@ uint64_t HELPER(mvst)(CPUS390XState *env, uint64_t c, uint64_t d, uint64_t s) s = fix_address(env, s); /* Lest we fail to service interrupts in a timely manner, limit the - amount of work we're willing to do. For now, lets cap at 8k. */ + amount of work we're willing to do. For now, let's cap at 8k. */ for (len = 0; len < 0x2000; ++len) { uint8_t v = cpu_ldub_data(env, s + len); cpu_stb_data(env, d + len, v); @@ -708,7 +708,7 @@ uint64_t HELPER(cksm)(CPUS390XState *env, uint64_t r1, uint64_t cksm = (uint32_t)r1; /* Lest we fail to service interrupts in a timely manner, limit the - amount of work we're willing to do. For now, lets cap at 8k. */ + amount of work we're willing to do. For now, let's cap at 8k. */ max_len = (src_len > 0x2000 ? 0x2000 : src_len); /* Process full words as available. */