22 Oct, 2005

18 commits

  • add || STI_CONSOLE to some of the basic FONTs. May need to get at
    least one of them to default to "Y" for parisc.

    Signed-off-by: Grant Grundler

    Signed-off-by: Kyle McMartin

    Grant Grundler
     
  • Fix card-mode Dino crashes on 725 (and probably other Snake) systems.
    Dino was coming up in fatal mode after a warm reboot. Resetting Dino
    brings it out of fatal mode, so do that if the status register indicates
    we're in fatal mode. Since this was never observed on any later systems,
    I presume firmware does this for us on those.

    Signed-off-by: Matthew Wilcox

    Add debug statements in the cfg_read and cfg_write functions
    Fix debug statements from the IRQ overhaul last winter
    Rename dino_driver_callback() to dino_probe()

    Signed-off-by: Matthew Wilcox

    Signed-off-by: Kyle McMartin

    Matthew Wilcox
     
  • revert use of %%sr0 in fdc asm.

    Thanks to Joel Soete for pointing out this oversight.

    Signed-off-by: Grant Grundler

    2.6.14-rc2-pa3 fdc/lci should be %r0 instead 0 for index (PA 1.1 compliance)
    From: Joel Soete
    Signed-off-by: Grant Grundler

    Explain why we need insert_resource() instead of request_resource().

    Fundementally, this is more convoluted for ccio driver because of
    o legacy (HP-PB) transperant bridges.
    o support for MMIO behind card-mode Dino (PCI)
    o support for above bridges without ccio in the box

    SBA driver doesn't have to worry about those issues.

    Signed-off-by: Grant Grundler

    Use insert_resource instead of request_resource now that the subdevices
    will already have their resources claimed

    Signed-off-by: Matthew Wilcox

    re-enable use of "inline" for perf critical functions.

    Signed-off-by: Grant Grundler

    2.6.12-rc4-pa5 fix sign extension of MMIO range

    Fixes the problem of claiming a range that is disabled on 64-bit kernel:
    ccio_init_resource() claimed CCIO bus address space (ffffffff00000000,
    ffffffffffffffff)
    also removes use of __FILE__.
    Tested on both 32 and 64-bit systems by Joel.

    From: Joel Soete
    Signed-off-by: Grant Grundler

    2.6.12-rc1-pa7 incorrect BUG_ON in ccio

    ccio-dma.c line 1317 was preventing K-class with 4GB RAM from booting.
    Any ccio machine with >=2GB of RAM would have (incorrectly) triggered this.

    Signed-off-by: Grant Grundler

    Convert to ioremap and __raw_read/write

    Signed-off-by: Matthew Wilcox

    Signed-off-by: Kyle McMartin

    Grant Grundler
     
  • revert use of %%sr0 in fdc asm.
    Thanks to Joel Soete for pointing out this oversight.

    Signed-off-by: Grant Grundler

    2.6.14-rc2-pa3 move "sync" outside the main loop that fills IO Pdir.

    Signed-off-by: Grant Grundler

    remove explicit use of sr0 in fdc ops.
    Thanks to Joel Soete for reminding me were I added those...

    Signed-off-by: Grant Grundler

    2.6.14-rc2-pa2 - make SBA more anal about invalidating pdir entries

    Previous code cleared the valid flag a pdir entry but it did NOT
    guarantee this change was visible to the PDIR before writing
    the PCOM register. Ie the SBA could pick up a stale entry if
    the write happened to hit the SBA before the cacheline was flushed
    from the cache.

    Long term, I think I want to make this a compile time flag.
    Developement tree should enable anal pdir checking by default
    and Debian can disable it with either a CONFIG option
    or one-line patch. fdc/sync options can only negatively affect
    performance though I haven't measure how much yet.
    If someone can run netperf TCP_RR across gige and compare
    -pa1 and -pa2, that would be sufficient.

    Cleaned up the use of "fdc" to make sure it's using "kernel"
    space id (specify sr0 but maps to sr4-7). It seems a bit fragile
    to assume "sr1" gets loaded with KERNEL_SPACE which is how the
    code works today.

    Tested on 32 and 64-bit SMP kernels on j6k.

    Signed-off-by: Grant Grundler

    remove PDC_NARROW from SBA and document history of PDC_NARROW a bit.
    It will still show up in an older kernel's .config file.

    Signed-off-by: Grant Grundler

    if/ifdef cleanups from Joel Soete.

    Signed-off-by: Grant Grundler

    2.6.12-rc4-pa2 fix 32-bit support for Astro platforms
    o Since my last SBA code change, SBA could allocate more than 1GB of IOVA
    space on Astro boxes with more than 1GB of RAM when running 32-bit kernel.
    This is bad since IOMMU can only talk to the first 1GB at most.
    Kudos to jejb for quickly spotting that bug.

    o jejb also noted SBA should *always* reject DMA masks > 32-bits since
    DMA-mapping.txt indicates caller should try again with 32-bits.

    o off-by-one error when comparing the mask to IOVA space size.

    Signed-off-by: Grant Grundler

    Signed-off-by: Kyle McMartin

    Grant Grundler
     
  • Convert pa_dev->hpa from an unsigned long to a struct resource.

    Signed-off-by: Matthew Wilcox

    Fix up users of ->hpa to use ->hpa.start instead.

    Signed-off-by: Matthew Wilcox

    Signed-off-by: Kyle McMartin

    Matthew Wilcox
     
  • Make /sys/bus/parisc/drivers look better by cleaning up parisc_driver
    names.

    Signed-off-by: Matthew Wilcox

    Signed-off-by: Kyle McMartin

    Matthew Wilcox
     
  • Fix parse_tree_node. much more needs to be done to fix this file.

    Signed-off-by: Matthew Wilcox

    Make drivers.c compile based on a patch from Pat Mochel.

    From: Patrick Mochel
    Signed-off-by: Kyle McMartin

    Fix drivers.c to create new device tree nodes when no match is found.

    Signed-off-by: Richard Hirst

    Do a proper depth-first search returning parents before children, using the
    new klist infrastructure.

    Signed-off-by: Richard Hirst

    Fixed parisc_device traversal so that pdc_stable works again
    Fixed check_dev so it doesn't dereference a parisc_device until it
    has verified the bus type

    Signed-off-by: Randolph Chung

    Convert pa_dev->hpa from an unsigned long to a struct resource.
    Use insert_resource() instead of request_mem_region().
    Request resources at bus walk time instead of driver probe time.
    Don't release the resources as we don't have any hotplug parisc_device
    support yet.
    Add parisc_pathname() to conveniently get the textual representation
    of the hwpath used in sysfs.
    Inline the remnants of claim_device() into its caller.

    Signed-off-by: Matthew Wilcox

    I noticed that some of the STI regions weren't showing up in iomem.
    Reading the STI spec indicated that all STI devices occupy at least 32MB.
    So check for STI HPAs and give them 32MB instead of 4kB.

    Signed-off-by: Matthew Wilcox

    Signed-off-by: Kyle McMartin

    Matthew Wilcox
     
  • Not sure how it slipped by, but here's a trivial typo fix for powernow.

    Signed-off-by: Chris Wright
    [ It's "nurter" backwards.. Maybe we have a hillbilly The Shining fan? ]
    Signed-off-by: Linus Torvalds

    Chris Wright
     
  • When I originally moved exit_itimers into __exit_signal, that was the only
    place where we could reliably know it was the last thread in the group
    dying, without races. Since then we've gotten the signal_struct.live
    counter, and do_exit can reliably do group-wide cleanup work.

    This patch moves the call to do_exit, where it's made without locks. This
    avoids the deadlock issues that the old __exit_signal code's comment talks
    about, and the one that Oleg found recently with process CPU timers.

    [ This replaces e03d13e985d48ac4885382c9e3b1510c78bd047f, which is why
    it was just reverted. ]

    Signed-off-by: Roland McGrath
    Signed-off-by: Linus Torvalds

    Roland McGrath
     
  • Revert commit e03d13e985d48ac4885382c9e3b1510c78bd047f, to be replaced
    by a much nicer fix from Roland.

    Linus Torvalds
     
  • AMD recently discovered that on some hardware, there is a race condition
    possible when a C-state change request goes onto the bus at the same
    time as a P-state change request.

    Both requests happen, but the southbridge hardware only acknowledges the
    C-state change. The PowerNow! driver is then stuck in a loop, waiting
    for the P-state change acknowledgement. The driver eventually times
    out, but can no longer perform P-state changes.

    It turns out the solution is to resend the P-state change, which the
    southbridge will acknowledge normally.

    Thanks to Johannes Winkelmann for reporting this and testing the fix.

    Signed-off-by: Mark Langsdorf
    Signed-off-by: Dave Jones
    Signed-off-by: Linus Torvalds

    Dave Jones
     
  • This fixes a stupid typo bug in the iSeries hash table code.

    When we place a hash PTE in the secondary bucket, instead of setting the
    SECONDARY flag bit, as we should, we (redundantly) set the VALID flag.

    This was introduced with the patch abolishing bitfields from the hash
    table code. Mea culpa, oops. It hasn't been noticed until now because
    in practice we don't hit the secondary bucket terribly often.

    Signed-off-by: David Gibson
    Signed-off-by: Linus Torvalds

    David Gibson
     
  • Linus Torvalds
     
  • Linus Torvalds
     
  • The wrong state emission routines were being called for G550, and
    consistent maps weren't correctly mapped...

    Signed-off-by: Dave Airlie
    Signed-off-by: Linus Torvalds

    Dave Airlie
     
  • While working on 64K pages, I found this little buglet in our
    update_mmu_cache() implementation.

    The code calls __hash_page() passing it an "access" parameter (the type
    of access that triggers the hash) containing the bits _PAGE_RW and
    _PAGE_USER of the linux PTE. The latter is useless in this case and the
    former is wrong. In fact, if we have a writeable PTE and we pass
    _PAGE_RW to hash_page(), it will set _PAGE_DIRTY (since we track dirty
    that way, by hash faulting !dirty) which is not what we want.

    In fact, the correct fix is to always pass 0. That means that only
    read-only or already dirty read write PTEs will be preloaded. The
    (hopefully rare) case of a non dirty read write PTE can't be preloaded
    this way, it will have to fault in hash_page on the actual access.

    Signed-off-by: Benjamin Herrenschmidt
    Signed-off-by: Linus Torvalds

    Benjamin Herrenschmidt
     
  • This fixes a typo in the div128_by_32 function used in the timekeeping
    calculations on ppc64. If you look at the code it's quite obvious
    that we need (rb + c) rather than (rb + b). The "b" is clearly just a
    typo.

    Signed-off-by: Paul Mackerras
    Signed-off-by: Linus Torvalds

    Paul Mackerras
     
  • This fixes handling of the phy identifiers in mptsas.

    Signed-off-by: Eric Moore
    [ split it a pre-2.6.14 portion from Eric's bigger patch ]
    Signed-off-by: Christoph Hellwig
    Signed-off-by: Linus Torvalds

    Eric Moore
     

21 Oct, 2005

10 commits

  • Signed-off-by: Russell King

    Russell King
     
  • Patch from Ben Dooks

    From: Guillaume Gourat

    Add MASK definitions for DCLK0 and DCLK1

    Signed-off-by: Guillaume Gourat
    Signed-off-by: Ben Dooks
    Signed-off-by: Russell King

    Ben Dooks
     
  • Patch from Ben Dooks

    The current Simtec BAST nand area timings are a little
    too slow to be obtained by a 2410 running at 266MHz,
    so reduce the timings slightly to bring them into the
    acceptable range.

    Signed-off-by: Ben Dooks
    Signed-off-by: Russell King

    Ben Dooks
     
  • Patch from Ben Dooks

    Avoid the possiblity that if the board is using
    a 16.9334 or higher crystal with a high PLL
    multiplier, then the pll value could overflow
    the capability of an int.

    Also fix the value types of the intermediate
    variables to unsigned int.

    Rewrite of patch from Guillaume Gourat

    Signed-off-by: Ben Dooks
    Signed-off-by: Russell King

    Ben Dooks
     
  • Patch from Matt Reimer

    Adds an I2S platform_device for PXA. I2S is used to interface
    with sound chips on systems like iPAQ h1910/h2200/hx4700 and
    Asus 716.

    Signed-off-by: mreimer@vpop.net
    Signed-off-by: Russell King

    Matt Reimer
     
  • It is legitimate to call tcp_fragment with len == skb->len since
    that is done for FIN packets and the FIN flag counts as one byte.
    So we should only check for the len > skb->len case.

    Signed-off-by: Herbert Xu
    Signed-off-by: Arnaldo Carvalho de Melo

    Herbert Xu
     
  • Turns out the problem has nothing to do with use-after-free or double-free.
    It's just that we're not clearing the CB area and DCCP unlike TCP uses a CB
    format that's incompatible with IP.

    Signed-off-by: Herbert Xu
    Signed-off-by: Ian McDonald
    Signed-off-by: Arnaldo Carvalho de Melo

    Herbert Xu
     
  • icmp_send doesn't use skb->sk at all so even if skb->sk has already
    been freed it can't cause crash there (it would've crashed somewhere
    else first, e.g., ip_queue_xmit).

    I found a double-free on an skb that could explain this though.
    dccp_sendmsg and dccp_write_xmit are a little confused as to what
    should free the packet when something goes wrong. Sometimes they
    both go for the ball and end up in each other's way.

    This patch makes dccp_write_xmit always free the packet no matter
    what. This makes sense since dccp_transmit_skb which in turn comes
    from the fact that ip_queue_xmit always frees the packet.

    Signed-off-by: Herbert Xu
    Signed-off-by: Arnaldo Carvalho de Melo

    Herbert Xu
     
  • David S. Miller wrote:
    > One thing you can probably do for this bug is to mark data packets
    > explicitly somehow, perhaps in the SKB control block DCCP already
    > uses for other data. Put some boolean in there, set it true for
    > data packets. Then change the test in dccp_transmit_skb() as
    > appropriate to test the boolean flag instead of "skb_cloned(skb)".

    I agree. In fact we already have that flag, it's called skb->sk.
    So here is patch to test that instead of skb_cloned().

    Signed-off-by: Herbert Xu
    Acked-by: Ian McDonald
    Signed-off-by: Arnaldo Carvalho de Melo

    Herbert Xu
     
  • This reverts commit 3359b54c8c07338f3a863d1109b42eebccdcf379 and
    replaces it with a cleaner version that is purely based on page table
    operations, so that the synchronization between inode size and hugetlb
    mappings becomes moot.

    Signed-off-by: Hugh Dickins
    Signed-off-by: Linus Torvalds

    Hugh Dickins
     

20 Oct, 2005

12 commits

  • The -rc4 release was supposed to be the last -rc, but here goes. The
    RCU fixes and the swiotlb changes need an -rc for final testing.

    Linus Torvalds
     
  • Missing half of the [PATCH] uml: Fix sysrq-r support for skas mode
    We need to remove these (UPT_[DEFG]S) from the read side as well as the
    write one - otherwise it simply won't build.

    Signed-off-by: Al Viro
    Acked-by: Paolo 'Blaisorblade' Giarrusso
    Acked-by: Jeff Dike
    Signed-off-by: Linus Torvalds

    Al Viro
     
  • Found in the -rt patch set. The scsi_error thread likely will be in the
    TASK_INTERRUPTIBLE state upon exit. This patch fixes this bug.

    Signed-off-by: Steven Rostedt
    Signed-off-by: Linus Torvalds

    Steven Rostedt
     
  • Signed-off-by: Paul Mackerras
    Signed-off-by: Linus Torvalds

    Paul Mackerras
     
  • Linus Torvalds
     
  • This introduces a limit parameter to the core bootmem allocator; The new
    parameter indicates that physical memory allocated by the bootmem
    allocator should be within the requested limit.

    We also introduce alloc_bootmem_low_pages_limit, alloc_bootmem_node_limit,
    alloc_bootmem_low_pages_node_limit apis, but alloc_bootmem_low_pages_limit
    is the only api used for swiotlb.

    The existing alloc_bootmem_low_pages() api could instead have been
    changed and made to pass right limit to the core allocator. But that
    would make the patch more intrusive for 2.6.14, as other arches use
    alloc_bootmem_low_pages(). We may be done that post 2.6.14 as a
    cleanup.

    With this, swiotlb gets memory within 4G for both x86_64 and ia64
    arches.

    Signed-off-by: Yasunori Goto
    Cc: Ravikiran G Thirumalai
    Signed-off-by: Linus Torvalds

    Yasunori Goto
     
  • In drivers/acpi/glue.c the address of an integer is cast to the address of
    an unsigned long. This breaks on systems where a long is larger than an
    int --- for a start the int can be misaligned; for a second the assignment
    through the pointer will overwrite part of the next variable.

    Signed-off-by: Peter Chubb
    Acked-by: "Brown, Len"
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Peter Chubb
     
  • I've gotten a report on lkml, of a possible regression in the MGA DRM in
    2.6.14-rc4 (since -rc1), I haven't been able to reproduce it here, but I've
    figured out some possible issues in the mga code that were definitely
    wrong, some of these are from DRM CVS, the main fix is the agp enable bit
    on the old code path still used by everyone.....

    Signed-off-by: Dave Airlie
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Dave Airlie
     
  • The PF_NOFREEZE process flag should not be inherited when a thread is
    forked. This patch (as585) removes the flag from the child.

    This problem is starting to show up more and more as drivers turn to the
    kthread API instead of using kernel_thread(). As a result, their kernel
    threads are now children of the kthread worker instead of modprobe, and
    they inherit the PF_NOFREEZE flag. This can cause problems during system
    suspend; the kernel threads are not getting frozen as they ought to be.

    Signed-off-by: Alan Stern
    Acked-by: Pavel Machek
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Alan Stern
     
  • The variable RCS_TAR_IGNORE is used in scripts/packaging/Makefile, but not
    exported from the main Makefile, so it's never used.

    This results in the rpm targets being very unhappy in quilted trees.

    Signed-off-by: Tom Rini
    Acked-by: Sam Ravnborg
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Tom Rini
     
  • The implementation of __kernel_gettimeofday() in the 32 bits vDSO has a
    small bug (a typo actually) that will cause it to lose 1 bit of precision.
    Not terribly bad but worth fixing.

    Signed-off-by: Benjamin Herrenschmidt
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Benjamin Herrenschmidt
     
  • The main problem fixes is that in certain situations stopping md arrays may
    take longer than you expect, or may require multiple attempts. This would
    only happen when resync/recovery is happening.

    This patch fixes three vaguely related bugs.

    1/ The recent change to use kthreads got the setting of the
    process name wrong. This fixes it.
    2/ The recent change to use kthreads lost the ability for
    md threads to be signalled with SIG_KILL. This restores that.
    3/ There is a long standing bug in that if:
    - An array needs recovery (onto a hot-spare) and
    - The recovery is being blocked because some other array being
    recovered shares a physical device and
    - The recovery thread is killed with SIG_KILL
    Then the recovery will appear to have completed with no IO being
    done, which can cause data corruption.
    This patch makes sure that incomplete recovery will be treated as
    incomplete.

    Note that any kernel affected by bug 2 will not suffer the problem of bug
    3, as the signal can never be delivered. Thus the current 2.6.14-rc
    kernels are not susceptible to data corruption. Note also that if arrays
    are shutdown (with "mdadm -S" or "raidstop") then the problem doesn't
    occur. It only happens if a SIGKILL is independently delivered as done by
    'init' when shutting down.

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

    NeilBrown