21 Dec, 2010

2 commits

  • Silence warnings such as -

    drivers/video/s1d13xxxfb.c:421: warning: ‘bltbit_wait_bitset’ defined but not used

    Just drop the unused code.

    Signed-off-by: Andres Salomon
    Signed-off-by: Paul Mundt

    Andres Salomon
     
  • This patch adds support for runtime pm using the functions.
    - pm_runtime_get_sync()
    - pm_runtime_put_sync()

    pm_runtime_get_sync() and pm_runtime_put_sync() are called when
    open or release function of framebufer driver is called to inform
    the system if hardware is idle or not.

    Signed-off-by: Jingoo Han
    Signed-off-by: Paul Mundt

    Jingoo Han
     

14 Dec, 2010

1 commit


26 Nov, 2010

2 commits


19 Nov, 2010

6 commits


18 Nov, 2010

5 commits

  • Commit 451a3c24b013 ("BKL: remove extraneous #include ")
    removed the #include line that was the only thing that was surrounded by
    the #ifdef/#endif.

    So now that #ifdef is guarding nothing at all. Just remove it.

    Reported-by: Byeong-ryeol Kim
    Signed-off-by: Linus Torvalds

    Linus Torvalds
     
  • Arnd Bergmann did an automated scripting run to find left-over instances
    of , and had made it trigger it on the normal BKL use
    of lock_kernel and unlock_lernel (and apparently release_kernel_lock and
    reacquire_kernel_lock too, used by the scheduler).

    That resulted in commit 451a3c24b013 ("BKL: remove extraneous #include
    ").

    However, hardirq.h was the only remaining user of the old
    'kernel_locked()' interface, and Arnd's script hadn't checked for that.
    So depending on your configuration and what header files had been
    included, you would get errors like "implicit declaration of function
    'kernel_locked'" during the build.

    The right fix is not to just re-instate the smp_lock.h include - it is
    to just remove 'kernel_locked()' entirely, since the only use was this
    one special low-level detail. Just make hardirq.h do it directly.

    In fact this simplifies and clarifies the code, because some trivial
    analysis makes it clear that hardirq.h only ever used _one_ of the two
    definitions of kernel_locked(), so we can remove the other one entirely.

    Reported-by: Zimny Lech
    Reported-and-acked-by: Randy Dunlap
    Acked-by: Arnd Bergmann
    Signed-off-by: Linus Torvalds

    Linus Torvalds
     
  • Lock_kernel is gone from the code, so the comments should be updated,
    too. nfsd now uses lock_flocks instead of lock_kernel to protect
    against posix file locks.

    Signed-off-by: Arnd Bergmann
    Acked-by: J. Bruce Fields
    Cc: linux-nfs@vger.kernel.org
    Signed-off-by: Linus Torvalds

    Arnd Bergmann
     
  • The big kernel lock has been removed from all these files at some point,
    leaving only the #include.

    Remove this too as a cleanup.

    Signed-off-by: Arnd Bergmann
    Signed-off-by: Linus Torvalds

    Arnd Bergmann
     
  • The stradis driver is on its way out, but it should still be marked
    correctly as depending on the big kernel lock. It could easily be
    changed to not require it if someone decides to revive the driver and
    port it to v4l2 in the process.

    Signed-off-by: Arnd Bergmann
    Reported-by: Sedat Dilek
    Cc: Nathan Laredo
    Acked-by: Greg Kroah-Hartman
    Signed-off-by: Linus Torvalds

    Arnd Bergmann
     

17 Nov, 2010

12 commits

  • Making /proc/kallsyms readable only for root by default makes it
    slightly harder for attackers to write generic kernel exploits by
    removing one source of knowledge where things are in the kernel.

    This is the second submit, discussion happened on this on first submit
    and mostly concerned that this is just one hole of the sieve ... but
    one of the bigger ones.

    Changing the permissions of at least System.map and vmlinux is also
    required to fix the same set, but a packaging issue.

    Target of this starter patch and follow ups is removing any kind of
    kernel space address information leak from the kernel.

    [ Side note: the default of root-only reading is the "safe" value, and
    it's easy enough to then override at any time after boot. The /proc
    filesystem allows root to change the permissions with a regular
    chmod, so you can "revert" this at run-time by simply doing

    chmod og+r /proc/kallsyms

    as root if you really want regular users to see the kernel symbols.
    It does help some tools like "perf" figure them out without any
    setup, so it may well make sense in some situations. - Linus ]

    Signed-off-by: Marcus Meissner
    Acked-by: Tejun Heo
    Acked-by: Eugene Teo
    Reviewed-by: Jesper Juhl
    Signed-off-by: Linus Torvalds

    Marcus Meissner
     
  • * 'bugfixes' of git://git.linux-nfs.org/projects/trondmy/nfs-2.6:
    nfs: Ignore kmemleak false positive in nfs_readdir_make_qstr
    SUNRPC: Simplify rpc_alloc_iostats by removing pointless local variable
    nfs: trivial: remove unused nfs_wait_event macro
    NFS: readdir shouldn't read beyond the reply returned by the server
    NFS: Fix a couple of regressions in readdir.
    Revert "NFSv4: Fall back to ordinary lookup if nfs4_atomic_open() returns EISDIR"
    Regression: fix mounting NFS when NFSv3 support is not compiled
    NLM: Fix a regression in lockd

    Linus Torvalds
     
  • …l/git/tip/linux-2.6-tip

    * 'sched-fixes-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/linux-2.6-tip:
    sched: Fix cross-sched-class wakeup preemption
    sched: Fix runnable condition for stoptask
    sched: Use group weight, idle cpu metrics to fix imbalances during idle

    Linus Torvalds
     
  • * 'pm-fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/rafael/suspend-2.6:
    PM / PM QoS: Fix reversed min and max
    PM / OPP: Hide OPP configuration when SoCs do not provide an implementation
    PM: Allow devices to be removed during late suspend and early resume

    Linus Torvalds
     
  • * 'futexes-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/linux-2.6-tip:
    futex: Address compiler warnings in exit_robust_list

    Linus Torvalds
     
  • Move the mid-layer's ->queuecommand() invocation from being locked
    with the host lock to being unlocked to facilitate speeding up the
    critical path for drivers who don't need this lock taken anyway.

    The patch below presents a simple SCSI host lock push-down as an
    equivalent transformation. No locking or other behavior should change
    with this patch. All existing bugs and locking orders are preserved.

    Additionally, add one parameter to queuecommand,
    struct Scsi_Host *
    and remove one parameter from queuecommand,
    void (*done)(struct scsi_cmnd *)

    Scsi_Host* is a convenient pointer that most host drivers need anyway,
    and 'done' is redundant to struct scsi_cmnd->scsi_done.

    Minimal code disturbance was attempted with this change. Most drivers
    needed only two one-line modifications for their host lock push-down.

    Signed-off-by: Jeff Garzik
    Acked-by: James Bottomley
    Signed-off-by: Linus Torvalds

    Jeff Garzik
     
  • * 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/jbarnes/pci-2.6:
    PCI: fix offset check for sysfs mmapped files

    Linus Torvalds
     
  • * 'for-linus' of git://git390.marist.edu/pub/scm/linux-2.6:
    [S390] kprobes: Fix the return address of multiple kretprobes
    [S390] kprobes: disable interrupts throughout
    [S390] ftrace: build without frame pointers on s390
    [S390] mm: add devmem_is_allowed() for STRICT_DEVMEM checking
    [S390] vmlogrdr: purge after recording is switched off
    [S390] cio: fix incorrect ccw_device_init_count
    [S390] tape: add medium state notifications
    [S390] fix get_user_pages_fast

    Linus Torvalds
     
  • I just loaded 2.6.37-rc2 on my machines, and I noticed that X no longer starts.
    Running an strace of the X server shows that it's doing this:

    open("/sys/bus/pci/devices/0000:07:00.0/resource0", O_RDWR) = 10
    mmap(NULL, 16777216, PROT_READ|PROT_WRITE, MAP_SHARED, 10, 0) = -1 EINVAL (Invalid argument)

    This code seems to be asking for a shared read/write mapping of 16MB worth of
    BAR0 starting at file offset 0, and letting the kernel assign a starting
    address. Unfortunately, this -EINVAL causes X not to start. Looking into
    dmesg, there's a complaint like so:

    process "Xorg" tried to map 0x01000000 bytes at page 0x00000000 on 0000:07:00.0 BAR 0 (start 0x 96000000, size 0x 1000000)

    ...with the following code in pci_mmap_fits:

    pci_start = (mmap_api == PCI_MMAP_SYSFS) ?
    pci_resource_start(pdev, resno) >> PAGE_SHIFT : 0;
    if (start >= pci_start && start < pci_start + size &&
    start + nr vm_pgoff to be between the
    resource's start and end address, and the end of the vma to be no farther than
    the end. However, the sysfs PCI resource files always start at offset zero,
    which means that this test always fails for programs that mmap the sysfs files.
    Given the comment in the original commit
    3b519e4ea618b6943a82931630872907f9ac2c2b, I _think_ the old procfs files
    require that the file offset be equal to the resource's base address when
    mmapping.

    I think what we want here is for pci_start to be 0 when mmap_api ==
    PCI_MMAP_PROCFS. The following patch makes that change, after which the Matrox
    and Mach64 X drivers work again.

    Acked-by: Martin Wilck
    Signed-off-by: Darrick J. Wong
    Signed-off-by: Jesse Barnes

    Darrick J. Wong
     
  • Strings allocated via kmemdup() in nfs_readdir_make_qstr() are
    referenced from the nfs_cache_array which is stored in a page cache
    page. Kmemleak does not scan such pages and it reports several false
    positives. This patch annotates the string->name pointer so that
    kmemleak does not consider it a real leak.

    Signed-off-by: Catalin Marinas
    Cc: Bryan Schumaker
    Cc: Trond Myklebust
    Signed-off-by: Trond Myklebust

    Catalin Marinas
     
  • Hi,

    We can simplify net/sunrpc/stats.c::rpc_alloc_iostats() a bit by getting
    rid of the unneeded local variable 'new'.

    Please CC me on replies.

    Signed-off-by: Jesper Juhl
    Signed-off-by: Trond Myklebust

    Jesper Juhl
     
  • Nothing uses this macro anymore.

    Signed-off-by: Jeff Layton
    Signed-off-by: Trond Myklebust

    Jeff Layton
     

16 Nov, 2010

12 commits