1. 13 Oct, 2009 3 commits
  2. 09 Nov, 2009 1 commit
    • Joe Perches's avatar
      UUID/GUIDs are somewhat common in kernel source. · c988f0ff
      Joe Perches authored
      Standardize the printed style of UUID/GUIDs by using
      another extension to %p.
      
      %pUb:   01020304-0506-0708-090a-0b0c0d0e0f10
      %pUB:   01020304-0506-0708-090A-0B0C0D0E0F10 (upper case)
      %pUl:   04030201-0605-0807-090a-0b0c0d0e0f10
      %pUL:   04030201-0605-0807-090A-0B0C0D0E0F10 (upper case)
      
      %pU defaults to %pUb
      Signed-off-by: default avatarJoe Perches <joe@perches.com>
      Cc: Jeff Garzik <jgarzik@redhat.com>
      Cc: Tejun Heo <tj@kernel.org>
      Cc: Alex Elder <aelder@sgi.com>
      Cc: Christoph Hellwig <hch@lst.de>
      Cc: Artem Bityutskiy <dedekind@infradead.org>
      Cc: Adrian Hunter <adrian.hunter@nokia.com>
      Cc: Steven Whitehouse <swhiteho@redhat.com>
      Cc: Mauro Carvalho Chehab <mchehab@infradead.org>
      Cc: Matt Mackall <mpm@selenic.com>
      Cc: Neil Brown <neilb@suse.de>
      Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
      c988f0ff
  3. 30 Oct, 2009 1 commit
  4. 28 Oct, 2009 2 commits
    • Joe Perches's avatar
      Fix email matching without name --n and --git-blame · b67d94a0
      Joe Perches authored
         Using --non and --git-blame caused maintainer signature
         matching to fail.  Fixed that by adding 3rd argument to
         sub format_email to control show/hide name portion of address
      Slurp -f file instead of reading line-by-line for K: pattern matching.
         Suggested by Wolfram Sang as more efficient
      Refactor git command execution
         Break into 2 functions, execute/analyze
         Share code between --git and --git-blame
         Don't warn multiple times when git isn't installed
      Improve stats reporting
         --git-min-percent and -- rolestats now count the total number of commits
         for either the period of --git-since or if using --git-blame the commits
         used by the current file and calculate commit % as
            # of commits signed / total commits * 100
      Code style cleaning
         Use consistent sub foo { my (args...) = @_;
      Signed-off-by: default avatarJoe Perches <joe@perches.com>
      Cc: Ben Hutchings <ben@decadent.org.uk>
      Cc: Greg KH <greg@kroah.com>
      Cc: Pavel Machek <pavel@ucw.cz>
      Cc: Wolfram Sang <w.sang@pengutronix.de>
      Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
      b67d94a0
    • Joe Perches's avatar
      --roles shows the role of each email address, i.e. why it was selected. · e13e757d
      Joe Perches authored
      --rolestats selects --roles and adds git log/blame signers #'s and %
      
      Multiple roles are possible (supporter, maintainer, git-signer...)
      
      --roles or --rolestats is meant to help identify appropriate maintainers
      to notify and should not be used with "git send-email --cc-cmd"
      
      Example output:
      
      Existing:
      
      $ ./scripts/get_maintainer.pl -f arch/x86/kernel/acpi/boot.c
      Corentin Chary <corentincj@iksaif.net>
      Karol Kozimor <sziwan@users.sourceforge.net>
      Len Brown <len.brown@intel.com>
      Pavel Machek <pavel@ucw.cz>
      Rafael J. Wysocki <rjw@sisk.pl>
      Thomas Gleixner <tglx@linutronix.de>
      Ingo Molnar <mingo@redhat.com>
      H. Peter Anvin <hpa@zytor.com>
      x86@kernel.org
      Yinghai Lu <yhlu.kernel@gmail.com>
      Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com>
      acpi4asus-user@lists.sourceforge.net
      linux-pm@lists.linux-foundation.org
      linux-kernel@vger.kernel.org
      
      With --roles
      
      $ ./scripts/get_maintainer.pl --roles -f arch/x86/kernel/acpi/boot.c
      Corentin Chary <corentincj@iksaif.net> (maintainer:ASUS ACPI EXTRAS...)
      Karol Kozimor <sziwan@users.sourceforge.net> (maintainer:ASUS ACPI EXTRAS...)
      Len Brown <len.brown@intel.com> (supporter:SUSPEND TO RAM,git-signer)
      Pavel Machek <pavel@ucw.cz> (supporter:SUSPEND TO RAM)
      Rafael J. Wysocki <rjw@sisk.pl> (supporter:SUSPEND TO RAM)
      Thomas Gleixner <tglx@linutronix.de> (maintainer:X86 ARCHITECTURE...)
      Ingo Molnar <mingo@redhat.com> (maintainer:X86 ARCHITECTURE...,git-signer)
      H. Peter Anvin <hpa@zytor.com> (maintainer:X86 ARCHITECTURE...)
      x86@kernel.org (maintainer:X86 ARCHITECTURE...)
      Yinghai Lu <yhlu.kernel@gmail.com> (git-signer)
      Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com> (git-signer)
      acpi4asus-user@lists.sourceforge.net (open list:ASUS ACPI EXTRAS...)
      linux-pm@lists.linux-foundation.org (open list:SUSPEND TO RAM)
      linux-kernel@vger.kernel.org (open list)
      
      With --rolestats
      
      $ ./scripts/get_maintainer.pl --rolestats -f arch/x86/kernel/acpi/boot.c
      Corentin Chary <corentincj@iksaif.net> (maintainer:ASUS ACPI EXTRAS...)
      Karol Kozimor <sziwan@users.sourceforge.net> (maintainer:ASUS ACPI EXTRAS...)
      Len Brown <len.brown@intel.com> (supporter:SUSPEND TO RAM,git-signer:16/79=20%)
      Pavel Machek <pavel@ucw.cz> (supporter:SUSPEND TO RAM)
      Rafael J. Wysocki <rjw@sisk.pl> (supporter:SUSPEND TO RAM)
      Thomas Gleixner <tglx@linutronix.de> (maintainer:X86 ARCHITECTURE...)
      Ingo Molnar <mingo@redhat.com> (maintainer:X86 ARCHITECTURE...,git-signer:29/79=37%)
      H. Peter Anvin <hpa@zytor.com> (maintainer:X86 ARCHITECTURE...)
      x86@kernel.org (maintainer:X86 ARCHITECTURE...)
      Yinghai Lu <yhlu.kernel@gmail.com> (git-signer:12/79=15%)
      Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com> (git-signer:6/79=8%)
      acpi4asus-user@lists.sourceforge.net (open list:ASUS ACPI EXTRAS...)
      linux-pm@lists.linux-foundation.org (open list:SUSPEND TO RAM)
      linux-kernel@vger.kernel.org (open list)
      
      With --rolestats and --git-blame
      
      $ ./scripts/get_maintainer.pl --rolestats --git-blame -f arch/x86/kernel/acpi/boot.c
      Corentin Chary <corentincj@iksaif.net> (maintainer:ASUS ACPI EXTRAS...)
      Karol Kozimor <sziwan@users.sourceforge.net> (maintainer:ASUS ACPI EXTRAS...)
      Len Brown <len.brown@intel.com> (supporter:SUSPEND TO RAM,git-signer:16/79=20%,commits:22/154=14%)
      Pavel Machek <pavel@ucw.cz> (supporter:SUSPEND TO RAM)
      Rafael J. Wysocki <rjw@sisk.pl> (supporter:SUSPEND TO RAM)
      Thomas Gleixner <tglx@linutronix.de> (maintainer:X86 ARCHITECTURE...)
      Ingo Molnar <mingo@redhat.com> (maintainer:X86 ARCHITECTURE...,git-signer:29/79=37%,commits:36/154=23%)
      H. Peter Anvin <hpa@zytor.com> (maintainer:X86 ARCHITECTURE...)
      x86@kernel.org (maintainer:X86 ARCHITECTURE...)
      Yinghai Lu <yhlu.kernel@gmail.com> (git-signer:12/79=15%,commits:9/154=6%)
      Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com> (git-signer:6/79=8%)
      Andi Kleen <ak@suse.de> (commits:11/154=7%)
      Andrew Morton <akpm@osdl.org> (commits:10/154=6%)
      acpi4asus-user@lists.sourceforge.net (open list:ASUS ACPI EXTRAS...)
      linux-pm@lists.linux-foundation.org (open list:SUSPEND TO RAM)
      linux-kernel@vger.kernel.org (open list)
      
      Other changes:
      
      Format git-signers email addresses a bit to reduce bad signatures
      Command line bad arguments emitted a verbose usage(), just show --help
      Version number bumped to .22
      
      Ben Hutchings had the idea and created a good deal of this implementation.
      Signed-off-by: default avatarJoe Perches <joe@perches.com>
      Signed-off-by: default avatarBen Hutchings <ben@decadent.org.uk>
      Cc: Greg KH <greg@kroah.com>
      Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
      e13e757d
  5. 12 Oct, 2009 1 commit
  6. 09 Nov, 2009 1 commit
    • Bernhard Walle's avatar
      The kernel offers with TIOCL_GETKMSGREDIRECT ioctl() the possibility to · ac6fe062
      Bernhard Walle authored
      redirect the kernel messages to a specific console.
      
      However, since it's not possible to switch to the kernel message console
      after a panic(), it would be nice if the kernel would print the panic
      message on the current console.
      
      This patch series adds a new interface to access the global kmsg_redirect
      variable by a function to be able to use it in code where
      CONFIG_VT_CONSOLE is not set (kernel/panic.c).
      
      
      
      This patch:
      
      Instead of using and exporting a global value kmsg_redirect, introduce a
      function vt_kmsg_redirect() that both can set and return the console where
      messages are printed.
      
      Change all users of kmsg_redirect (the VT code itself and kernel/power.c)
      to the new interface.
      
      The main advantage is that vt_kmsg_redirect() can also be used when
      CONFIG_VT_CONSOLE is not set.
      Signed-off-by: default avatarBernhard Walle <bernhard@bwalle.de>
      Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>
      Cc: Ingo Molnar <mingo@elte.hu>
      Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
      ac6fe062
  7. 11 Nov, 2009 6 commits
  8. 09 Nov, 2009 1 commit
  9. 19 Sep, 2009 1 commit
  10. 23 Sep, 2009 1 commit
  11. 12 Nov, 2009 3 commits
  12. 09 Oct, 2009 1 commit
  13. 29 Sep, 2009 1 commit
  14. 10 Nov, 2009 6 commits
    • Alexey Dobriyan's avatar
      It's easy to lose useful DEBUG_BUGVERBOSE by switching EMBEDDED left and right. · b317fcae
      Alexey Dobriyan authored
      Signed-off-by: default avatarAlexey Dobriyan <adobriyan@gmail.com>
      Cc: Sam Ravnborg <sam@ravnborg.org>
      Cc: Ingo Molnar <mingo@elte.hu>
      Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
      b317fcae
    • Andrew Morton's avatar
      s/_rl/_ratelimited/g · f88c2488
      Andrew Morton authored
      Cc: Hiroshi Shimamoto <h-shimamoto@ct.jp.nec.com>
      Cc: Ingo Molnar <mingo@elte.hu>
      Cc: Joe Perches <joe@perches.com>
      Cc: Naohiro Ooiwa <nooiwa@miraclelinux.com>
      Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
      Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
      f88c2488
    • Andrew Morton's avatar
      ERROR: code indent should use tabs where possible · 81118b38
      Andrew Morton authored
      #42: FILE: include/linux/kernel.h:427:
      +        printk_ratelimited(KERN_EMERG pr_fmt(fmt), ##__VA_ARGS__)$
      
      ERROR: code indent should use tabs where possible
      #44: FILE: include/linux/kernel.h:429:
      +        printk_ratelimited(KERN_ALERT pr_fmt(fmt), ##__VA_ARGS__)$
      
      ERROR: code indent should use tabs where possible
      #46: FILE: include/linux/kernel.h:431:
      +        printk_ratelimited(KERN_CRIT pr_fmt(fmt), ##__VA_ARGS__)$
      
      ERROR: code indent should use tabs where possible
      #48: FILE: include/linux/kernel.h:433:
      +        printk_ratelimited(KERN_ERR pr_fmt(fmt), ##__VA_ARGS__)$
      
      ERROR: code indent should use tabs where possible
      #50: FILE: include/linux/kernel.h:435:
      +        printk_ratelimited(KERN_WARNING pr_fmt(fmt), ##__VA_ARGS__)$
      
      ERROR: code indent should use tabs where possible
      #52: FILE: include/linux/kernel.h:437:
      +        printk_ratelimited(KERN_NOTICE pr_fmt(fmt), ##__VA_ARGS__)$
      
      ERROR: code indent should use tabs where possible
      #54: FILE: include/linux/kernel.h:439:
      +        printk_ratelimited(KERN_INFO pr_fmt(fmt), ##__VA_ARGS__)$
      
      total: 7 errors, 0 warnings, 50 lines checked
      
      ./patches/kernelh-add-printk_ratelimited-and-pr_level_rl.patch has style problems, please review.  If any of these errors
      are false positives report them to the maintainer, see
      CHECKPATCH in MAINTAINERS.
      
      Please run checkpatch prior to sending patches
      
      Cc: Hiroshi Shimamoto <h-shimamoto@ct.jp.nec.com>
      Cc: Ingo Molnar <mingo@elte.hu>
      Cc: Joe Perches <joe@perches.com>
      Cc: Naohiro Ooiwa <nooiwa@miraclelinux.com>
      Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
      Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
      81118b38
    • Joe Perches's avatar
      Add a printk_ratelimited statement expression macro that uses a per-call · f736107b
      Joe Perches authored
      ratelimit_state so that multiple subsystems output messages are not
      suppressed by a global __ratelimit state.
      Signed-off-by: default avatarJoe Perches <joe@perches.com>
      Cc: Naohiro Ooiwa <nooiwa@miraclelinux.com>
      Cc: Ingo Molnar <mingo@elte.hu>
      Cc: Hiroshi Shimamoto <h-shimamoto@ct.jp.nec.com>
      Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
      Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
      f736107b
    • Helge Deller's avatar
      The PR_GET_ENDIAN and PR_SET_ENDIAN prctl() calls have been implemented to · c21a779d
      Helge Deller authored
      allow to switch processes at runtime from big-endian to little-endian mode
      (and vice versa) on PowerPC processors.  Since the other architectures
      don't support this feature, they currently will just fail and return
      -EINVAL.
      
      This patch adds just minimal overhead and implements the PR_GET_ENDIAN
      call for all other architectures by returning the native endianess of the
      architecture.  Furthermore, calling prctl(PR_SET_ENDIAN) with the native
      endianess of the architecture will succeed, while trying to set another
      (not-supported) endianess, will fail.
      
      The patch can be tested with the following program:
      
      #include <stdio.h>
      #include <linux/prctl.h>
      
      int main(int argc, char **argv)
      {
      	int endian, ret;
      
      	ret = prctl(PR_GET_ENDIAN, &endian);
      	if (ret)
      		perror("prctl(PR_GET_ENDIAN) not implemented");
      	printf("current process/machine is running in %s endian mode (%d)\n",
      		endian == PR_ENDIAN_LITTLE ? "little":"big", endian);
      
      	/* setting native endianess should succeed */
      	ret = prctl(PR_SET_ENDIAN, endian);
      	printf("prctl(PR_SET_ENDIAN,%d) should succeed: %s\n",
      		endian, ret == 0 ? "OK":"FAIL");
      
      	/* setting foreign endianess should fail */
      	endian = (endian == PR_ENDIAN_LITTLE) ?
      		PR_ENDIAN_BIG : PR_ENDIAN_LITTLE;
      	ret = prctl(PR_SET_ENDIAN, endian);
      	printf("prctl(PR_SET_ENDIAN,%d) should fail: %s\n",
      		endian, ret == 0 ? "OK":"FAIL");
      }
      Signed-off-by: default avatarHelge Deller <deller@gmx.de>
      Cc: Anton Blanchard <anton@samba.org>
      Cc: Paul Mackerras <paulus@samba.org>
      Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
      c21a779d
    • Thadeu Lima de Souza Cascardo's avatar
      Commit 8c870933 has removed the · 2257d081
      Thadeu Lima de Souza Cascardo authored
      pmu_device_init call from misc_init, but unlike other similar commits,
      has not removed its declaration.
      Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
      2257d081
  15. 03 Nov, 2009 1 commit
  16. 15 Oct, 2009 1 commit
  17. 16 Oct, 2009 2 commits
    • Andrew Morton's avatar
      simplify · 28a5ae65
      Andrew Morton authored
      Cc: Amerigo Wang <amwang@redhat.com>
      Cc: Ben Woodard <bwoodard@llnl.gov>
      Cc: Brian Behlendorf <behlendorf1@llnl.gov>
      Cc: David Howells <dhowells@redhat.com>
      Cc: WANG Cong <amwang@redhat.com>
      Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
      28a5ae65
    • Amerigo Wang's avatar
      rwsem_is_locked() tests ->activity without locks, so we should always keep · d2ce5671
      Amerigo Wang authored
      ->activity consistent.  However, the code in __rwsem_do_wake() breaks this
      rule, it updates ->activity after _all_ readers waken up, this may give
      some reader a wrong ->activity value, thus cause rwsem_is_locked() behaves
      wrong.
      
      Quote from Andrew:
      
      "
      - we have one or more processes sleeping in down_read(), waiting for access.
      
      - we wake one or more processes up without altering ->activity
      
      - they start to run and they do rwsem_is_locked().  This incorrectly
        returns "false", because the waker process is still crunching away in
        __rwsem_do_wake().
      
      - the waker now alters ->activity, but it was too late.
      "
      
      So we need get a spinlock to protect this.  And rwsem_is_locked() should
      not block, thus we use spin_trylock_irqsave().
      Reported-by: default avatarBrian Behlendorf <behlendorf1@llnl.gov>
      Cc: Ben Woodard <bwoodard@llnl.gov>
      Cc: David Howells <dhowells@redhat.com>
      Signed-off-by: default avatarWANG Cong <amwang@redhat.com>
      Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
      d2ce5671
  18. 14 Oct, 2009 1 commit
  19. 09 Nov, 2009 1 commit
  20. 13 Oct, 2009 2 commits
    • H Hartley Sweeten's avatar
      The symbol 'call' is a static symbol used for initcall_debug. This same · 506fbd3e
      H Hartley Sweeten authored
      symbol name is used locally by a couple functions and produces the
      following sparse warnings:
      
      	warning: symbol 'call' shadows an earlier one
      
      Fix this noise by renaming the local symbols.
      Signed-off-by: default avatarH Hartley Sweeten <hsweeten@visionengravers.com>
      Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
      506fbd3e
    • Daniel Mack's avatar
      On Mon, Oct 12, 2009 at 12:31:46PM -0400, H Hartley Sweeten wrote: · 97876a82
      Daniel Mack authored
      > On Wednesday, October 07, 2009 1:01 PM, Daniel Mack wrote:
      > > This is actually too trivial to publish, but to export the function of
      > > that chip to the userspace, a module like this is needed.
      > >
      > > Signed-off-by: Daniel Mack <daniel@caiaq.de>
      > > Cc: Andrew Morton <akpm@linux-foundation.org>
      > > Cc: David Brownell <dbrownell@users.sourceforge.net>
      > > Cc: spi-devel-general@lists.sourceforge.net
      > > ---
      >
      > [snip]
      >
      > > +static ssize_t dac7512_store_val(struct device *dev,
      > > +				 struct device_attribute *attr,
      > > +				 const char *buf, size_t count)
      > > +{
      > > +	struct spi_device *spi = to_spi_device(dev);
      > > +	unsigned char tmp[2];
      > > +	unsigned long val;
      > > +
      > > +	if (strict_strtoul(buf, 10, &val) < 0)
      > > +		return -EINVAL;
      > > +
      > > +	tmp[0] = val >> 8;
      > > +	tmp[1] = val & 0xff;
      > > +	spi_write(spi, tmp, sizeof(tmp));
      > > +	return count;
      > > +}
      > > +
      > > +static DEVICE_ATTR(value, S_IWUSR | S_IRUGO,
      > > +		   NULL, dac7512_store_val);
      >
      > You have declared the "value" device attribute with mode S_IWUSR | S_IRUGO
      > but have not provided a show callback.
      
      Sorry, forget my last mail, I got you wrong. You're of course right,
      S_IRUGO shouldn't be set for write-only attributes. Updates patch below.
      
      Thanks,
      Daniel
      
      >From ab18a967e55d2bb1d39559333bca81a01c2838f0 Mon Sep 17 00:00:00 2001
      Date: Thu, 8 Oct 2009 03:55:46 +0800
      Subject: [PATCH] drivers/misc: add driver for Texas Instruments DAC7512
      This is actually too trivial to publish, but to export the function of
      that chip to the userspace, a module like this is needed.
      Signed-off-by: default avatarDaniel Mack <daniel@caiaq.de>
      Cc: David Brownell <dbrownell@users.sourceforge.net>
      Cc: "H Hartley Sweeten" <hartleys@visionengravers.com>
      Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
      97876a82
  21. 26 Oct, 2009 1 commit
  22. 24 Sep, 2009 1 commit
  23. 09 Oct, 2009 1 commit