28 Aug, 2006

18 commits

  • Unlike the other tty comment patch this one has code changes. Specifically
    it limits the queue size for a tty to 64K characters (128Kbytes) worst case
    even if the tty is ignoring tty->throttle. This is because certain drivers
    don't honour the throttle value correctly, although it is a useful
    safeguard anyway.

    Signed-off-by: Alan Cox
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Alan Cox
     
  • Doesn't fix them but does show up some interesting areas that need review
    and fixing.

    Signed-off-by: Alan Cox
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Alan Cox
     
  • Fix printk format warning:
    drivers/cdrom/gscd.c:269: warning: format ‘%lu’ expects type ‘long unsigned int’, but argument 2 has type ‘unsigned int’

    Signed-off-by: Randy Dunlap
    Cc: Jens Axboe
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Randy Dunlap
     
  • None of the other /proc/meminfo lines have a space in the identifier. This
    post-2.6.17 addition has the potential to break existing parsers, so use an
    underscore instead (like Committed_AS).

    Cc: Christoph Lameter
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Andrew Morton
     
  • idescsi_pc_intr() uses local_irq_enable() in IRQ context: annotate it.

    (this has no effect on kernels with lockdep disabled. On kernels with lockdep
    enabled this means that we wont actually disable interrupts, and the warning
    message will go away as well.)

    Signed-off-by: Ingo Molnar
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Ingo Molnar
     
  • A recent patch broke the ability to do a user-request check of a raid1.
    This patch fixes the breakage and also moves a comment that was dislocated
    by the same patch.

    Signed-off-by: Neil Brown
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    NeilBrown
     
  • If we
    - shut down a clean array,
    - restart with one (or more) drive(s) missing
    - make some changes
    - pause, so that they array gets marked 'clean',
    the event count on the superblock of included drives
    will be the same as that of the removed drives.
    So adding the removed drive back in will cause it
    to be included with no resync.

    To avoid this, we only update the eventcount backwards when the array
    is not degraded. In this case there can (should) be no non-connected
    drives that we can get confused with, and this is the particular case
    where updating-backwards is valuable.

    Signed-off-by: Neil Brown
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    NeilBrown
     
  • The recent hwctrl core conversion for MTD NAND devices broke the Amstrad
    Delta driver. This fixes it up and uses the existing control line defines
    rather than unclear magic numbers.

    Signed-off-by: Jonathan McDowell
    Acked-by: David Woodhouse
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Jonathan McDowell
     
  • When reading /dev/vcsa while a font with more than 256 characters is
    loaded, one of the attribute bits records the 9th bit of the character.
    But depending on the console driver (vgacon or fbcon for instance), that's
    bit 3 or bit 0. And there is no way for userland to know that, thus no way
    for userland to safely grab the screen content. So here is a (tested)
    patch:

    Add a VT_GETHIFONTMASK ioctl for knowing which bit is the 9th bit for VC
    text (vc_hi_font_mask field of the vc_data structure).

    Signed-off-by: Samuel Thibault
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Samuel Thibault
     
  • I wish I was happier about this patch. It'll serve as a placeholder for
    the moment. I'm still trying to get a G550 working in order to even
    reproduce the problem this patch introduces. I find that the G450 has
    jitter even without this patch, so it won't show me what the patch changed.
    At this point, I'll continue trying to get the G550 to work, and in
    parallel work with the G450 to work out the kinks.

    The patch is below.

    Set XDVICLKCTRL only on PPC, as doing this apparently introduces jitter on
    the G550, at least on x86 architectures.

    Signed-off-by: Paul A. Clarke
    Signed-off-by: Petr Vandrovec
    Cc: "Antonino A. Daplas"
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Paul A. Clarke
     
  • While testing Moxa C218T/PCI on PowerPC 405EP I found that loading firmware
    using the linux kernel driver fails because calculation of the checksum is
    not endianess independent in the original code.

    After I fixed this I found that uploading firmware in a system with
    multiple cards causes a kernel oops. I had a look in the recent moxa
    sources and found that they do some kind of locking there. Applying this
    lock fixed the problem.

    Alan sayeth:

    Checksum changes are clearly correct. Other changes is an improvement but
    not I think enough to handle malicious firmware attacks. That said such an
    attacker has CAP_SYS_RAWIO anyway so that part is irrelevant except for
    neatness.

    [akpm@osdl.org: cleanups]
    Signed-off-by: Dirk Eibach
    Acked-by: Alan Cox
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Dirk Eibach
     
  • Commit b64ef8afa58f397e1eaba2bd9ecaa6812064d464 ("[PATCH] add imacfb
    documentation and detection") contained a wrong DMI_MATCH.

    Signed-off-by: Thomas Meyer
    Cc: Greg KH
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Thomas Meyer
     
  • Read the return value before we release the nand device otherwise the
    value can become corrupted by another user of chip->ops, ultimately
    resulting in filesystem corruption.

    Signed-off-by: Richard Purdie
    Cc: David Woodhouse
    Acked-by: Josh Boyer
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Richard Purdie
     
  • Recently a patch was added for preliminary suspend/resume handling on
    !PPC_PMAC. However, this broke both suspend and firewire on powerpc
    because it saves the pci state after the device has already been disabled.

    This moves the save state to before the pmac specific code.

    Signed-off-by: Danny Tholen
    Cc: Stefan Richter
    Acked-by: Benjamin Herrenschmidt
    Cc: Ben Collins
    Cc: Jody McIntyre
    Cc:
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Danny Tholen
     
  • When cdev_add() failed there is no reason to call cdev_del().

    Signed-off-by: Rolf Eike Beer
    Cc: Alan Cox
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Rolf Eike Beer
     
  • Fix the year check on setting the time with the S3C24XX RTC driver. Also
    move the debug to before the set to see what is going on if it does fail.

    Signed-off-by: Ben Dooks
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Ben Dooks
     
  • On an nForce4-equipped machine with two SATA disk in raid1 setup using dmraid,
    we experienced frequent deadlock of the system under high i/o load. 'cat
    /dev/zero > ~/zero' was the most reliable way to reproduce them: Randomly
    after a few GB, 'cp' would be left in 'D' state along with kjournald and
    kmirrord. The functions cp and kjournald were blocked in did vary, but
    kmirrord's wchan always pointed to 'mempool_alloc()'. We've seen this pattern
    on 2.6.15 and 2.6.17 kernels. http://lkml.org/lkml/2005/4/20/142 indicates
    that this problem has been around even before.

    So much for the facts, here's my interpretation: mempool_alloc() first tries
    to atomically allocate the requested memory, or falls back to hand out
    preallocated chunks from the mempool. If both fail, it puts the calling
    process (kmirrord in this case) on a private waitqueue until somebody refills
    the pool. Where the only 'somebody' is kmirrord itself, so we have a
    deadlock.

    I worked around this problem by falling back to a (blocking) kmalloc when
    before kmirrord would have ended up on the waitqueue. This defeats part of
    the benefits of using the mempool, but at least keeps the system running. And
    it could be done with a two-line change. Note that mempool_alloc() clears the
    GFP_NOIO flag internally, and only uses it to decide whether to wait or return
    an error if immediate allocation fails, so the attached patch doesn't change
    behaviour in the non-deadlocking case. Path is against current git
    (2.6.18-rc4), but should apply to earlier versions as well. I've tested on
    2.6.15, where this patch makes the difference between random lockup and a
    stable system.

    Signed-off-by: Daniel Kobras
    Acked-by: Alasdair G Kergon
    Cc:
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Daniel Kobras
     
  • In the cleanups of drivers/rtc/s3c-rtc.c, the base address for the
    registers got broken. This patch fixes that by ensuring the readb/writeb
    are all prefixed with the base returned from ioremap()ing the registers.

    Also fix check for valid year range, which was the wrong way around.

    Signed-off-by: Ben Dooks
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Ben Dooks
     

27 Aug, 2006

9 commits

  • The existing unusual_devs entry for the UCR-61S2B appears to have too
    wide a revision range. It matches at least one device that doesn't
    respond to the initialization sequence. Perhaps the sequence needs to
    be updated, or perhaps something else can be done. For now, this patch
    (as764) restricts the range to include only the revision mentioned in
    the original comment.

    This resolves (for now!) Bugzilla entry #6950.

    Signed-off-by: Alan Stern
    Signed-off-by: Greg Kroah-Hartman

    Alan Stern
     
  • This patch removes support for a clone of Nokia DKU-5 cable made by Ours
    Technology Inc, as it turned out that the cable does not use the pl2303
    chip, but OTI-6858 chip which is not compatible with the pl2303.

    Signed-off-by: Tomasz Kazmierczak
    Signed-off-by: Greg Kroah-Hartman

    Tomasz Kazmierczak
     
  • This was pointed out by Adrian Bunk , as found by the Coverity Checker.

    Cc: Adrian Bunk
    Cc: Oliver Bock
    Signed-off-by: Greg Kroah-Hartman

    Greg Kroah-Hartman
     
  • Removes an unused kerneldoc entry from pci_match_device and
    put the others into correct order.

    Signed-off-by: Henrik Kretzschmar
    Signed-off-by: Greg Kroah-Hartman

    Henrik Kretzschmar
     
  • Here is a patch against the CPCI hotplug core to fix up PCI resource
    assignment such that things will actually work when a hot inserted
    device is enabled. I mentioned this patch to you way back in April at
    ELC, but am only now out from under things enough to clean it up and
    submit it. I've basically cribbed the corresponding code from
    shpchp_pci.c, so there are no big surprises. If it's still possible, I
    wouldn't mind this going into 2.6.18, but it wouldn't be the end of the
    world if it went into 2.6.19.

    Signed-off-by: Scott Murray
    Signed-off-by: Greg Kroah-Hartman

    Scott Murray
     
  • - add the ICH6(R) LPC to the ICH6 ACPI quirks. currently only the ICH6-M
    is handled. [ PCI_DEVICE_ID_INTEL_ICH6_1 is the ICH6-M LPC, ICH6_0 is
    the ICH6(R) ]

    - remove the wrong quirk calling asus_hides_smbus_lpc() for ICH6. the
    register modified in asus_hides_smbus_lpc() has a different meaning in
    ICH6.

    Signed-off-by: Daniel Ritz
    Acked-by: Jean Delvare
    Signed-off-by: Andrew Morton
    Signed-off-by: Greg Kroah-Hartman

    Daniel Ritz
     
  • This patch contains 2 sets of fixes for the abituguru:
    1) Much improved timeout handling, drasticly reducing the amount of
    timeout errors on some motherboards
    2) Fix the exit paths in the bank1 sensor type detect code to always
    restore the original settings even on an error. Without this our
    special test settings could remain seriously confusing the system
    BIOS's setup menu.

    Both are very much related and are must haves, to avoid messing up the
    uguru CMOS settings.

    Detailed changes:
    - Much improved timeout / wait for status handling. Many thanks to Sunil
    Kumar, for all his testing, ideas and patches! The code now first busy
    waits, polling the uguru for the expected status as this usually
    succeeds pretty quickly (within 90 reads). To avoid unnecessary CPU burn
    in timeout conditions, the amount of busy waiting has been halved from
    previous versions (120 tries instead of 250). This is not a problem,
    because this version goes to sleep after 120 attemps for 1 jiffy and
    then tries again, it does this sleep and try again 5 times before
    finally giving up. This (almost?) completly removes the timeout errors
    some people have seen regulary. Apparently some older uguru versions
    sometimes are distracted for a (relatively) long time. This solves this.
    - These timeout errors not only occur in the sending address part of
    reading the uguru but also in the wait for read state, so errors in
    this state are now handled as retryable just like send address state
    errors and are only logged and reported to userspace if 3 executive
    tries fail.
    - Fix a very nasty bug in the bank1 sensor type detection code, where it
    would not restore the original settings in any of the error paths!
    - Since not successfully restoring the original settings can seriously
    confuse the system BIOS (hang when entering the relevant setup menu),
    we now try restoring them 3 times before giving up.

    Signed-off-by: Hans de Goede
    Signed-off-by: Jean Delvare
    Signed-off-by: Greg Kroah-Hartman

    Hans de Goede
     
  • The tps65010.c driver in the main tree never got updated with
    build fixes since the last batch of I2C driver changes; and the
    genirq trigger flags were updated wierdly too.

    This also includes a minor tweak to reduce the frequency used to
    poll for unplug-the-AC-power on the TPS chips that don't provide
    relevant IRQs. It _would_ be nice to sense whether there's even
    a battery, but that'd normally be an HDQ/1-wire interface to a
    smart battery, and such APIs aren't standardized.

    Signed-off-by: David Brownell
    Signed-off-by: Jean Delvare
    Signed-off-by: Greg Kroah-Hartman

    David Brownell
     
  • Greg Kroah-Hartman
     

26 Aug, 2006

5 commits

  • The callers of scsi_send_eh_cmnd are setting the cmnd buffer,
    and then scsi_send_eh_cmnd is copying that updated buffer to
    the old_cmnd variable. Then after the command runs, we end up
    copying that old_cmnd var which has the new cmnd to the scsi
    command buffer. When this command gets recent, all types of fun
    things happen like getting TUR or START_STOP commands with
    data and scatterlists.

    This patch made against scsi-rc-fixes, has the callers of
    scsi_send_eh_cmnd pass in the command so scsi_send_eh_cmnd
    can do the right thing. This should go into 2.6.18 since this
    fixes a regression added when we removed some of the scsi_cmnd
    fields and replaced them with local variables.

    Signed-off-by: Mike Christie
    Signed-off-by: James Bottomley

    Mike Christie
     
  • Signed-off-by: Andrew Vasquez
    Signed-off-by: James Bottomley

    Andrew Vasquez
     
  • Software must explicitely re-enable extended firmware tracing
    after any ISP abort condition.

    Signed-off-by: Andrew Vasquez
    Signed-off-by: James Bottomley

    Andrew Vasquez
     
  • Original code attempts to retry PLOGIs to fcports that are
    FCP_TARGETs only. If the driver never performed a successful
    PLOGI/PRLI, the port-type would never be assigned, and the
    relogin logic would silently drop the request (and thus the port
    would not be recognized and registered).

    The fix is relatively straightforward, drop the FCP_TARGET-only
    check.

    Signed-off-by: Andrew Vasquez
    Signed-off-by: James Bottomley

    Andrew Vasquez
     
  • There's a problem where sg is executing a ->nopage operation on a
    compound page, it actually calls get_page() on the first page in the
    compound rather than the page which is being mapped. The fix is to
    select the correct page by indexing into the compound.

    Signed-off-by: James Bottomley

    Douglas Gilbert
     

25 Aug, 2006

1 commit


24 Aug, 2006

7 commits