linux-toradex.git/drivers/firmware, branch v2.6.27.21

dell_rbu: use scnprintf() instead of less secure sprintf()

2009-01-25T00:36:25+00:00

commit 81156928f8fe31621e467490b9d441c0285998c3 upstream.

Reading 0 bytes from /sys/devices/platform/dell_rbu/image_type or
/sys/devices/platform/dell_rbu/packet_size by an ordinary user causes an
oops.

Signed-off-by: Pavel Roskin 
Signed-off-by: Linus Torvalds 
Signed-off-by: Greg Kroah-Hartman

ibft: fix target info parsing in ibft module

2008-09-03T02:21:40+00:00

I got this patch through Red Hat's bugzilla from the bug submitter and
patch creator.  I have just fixed it up so it applies without fuzz to
upstream kernels.

Original patch and description from Shyam kumar Iyer:

The issue [ibft module not displaying targets with short names] is because
of an offset calculatation error in the iscsi_ibft.c code.  Due to this
error directory structure for the target in /sys/firmware/ibft does not
get created and so the initiator is unable to connect to the target.

Note that this bug surfaced only with an name that had a short section at
the end.  eg: "iqn.1984-05.com.dell:dell".  It did not surface when the
iqn's had a longer section at the end.  eg:
"iqn.2001-04.com.example:storage.disk2.sys1.xyz"

So, the eot_offset was calculated such that an extra 48 bytes i.e.  the
size of the ibft_header which has already been accounted was subtracted
twice.

This was not evident with longer iqn names because they would overshoot
the total ibft length more than 48 bytes and thus would escape the bug.

Signed-off-by: Shyam Kumar Iyer 
Signed-off-by: Mike Christie 
Cc: Konrad Rzeszutek 
Cc: Peter Jones 
Cc: James Bottomley 
Cc: Greg Kroah-Hartman 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

firmware/memmap: cleanup

2008-08-12T23:07:31+00:00

Various cleanup the drivers/firmware/memmap (after review by AKPM):

    - fix kdoc to conform to the standard
    - move kdoc from header to implementation files
    - remove superfluous WARN_ON() after kmalloc()
    - WARN_ON(x); if (!x) -> if(!WARN_ON(x))
    - improve some comments

Signed-off-by: Bernhard Walle 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

iscsi_ibft_find: fix modpost warning

2008-07-30T16:41:44+00:00

Exporting __init functions is wrong.

Signed-off-by: Jan Beulich 
Acked-by: Konrad Rzeszutek 
Cc: Ingo Molnar 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

Merge branch 'linus' into cpus4096

2008-07-28T19:14:43+00:00

firmware: fix memmap printk format warnings

2008-07-27T03:16:48+00:00

Fix firmware/memmap printk format warnings:

  drivers/firmware/memmap.c:156: warning: format '%llx' expects type 'long long unsigned int', but argument 4 has type 'resource_size_t'
  drivers/firmware/memmap.c:161: warning: format '%llx' expects type 'long long unsigned int', but argument 4 has type 'resource_size_t'

Signed-off-by: Randy Dunlap 
Cc: Bernhard Walle 
Cc: Ingo Molnar 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

cpumask: change cpumask_of_cpu_ptr to use new cpumask_of_cpu

2008-07-26T14:40:33+00:00

  * Replace previous instances of the cpumask_of_cpu_ptr* macros
    with a the new (lvalue capable) generic cpumask_of_cpu().

Signed-off-by: Mike Travis 
Cc: Andrew Morton 
Cc: Jack Steiner 
Cc: Rusty Russell 
Signed-off-by: Ingo Molnar

dell_rbu: use memory_read_from_buffer()

2008-07-25T17:53:44+00:00

Signed-off-by: Akinobu Mita 
Cc: Abhay Salunke 
Cc: Zhang Rui 
Cc: Matt Domsch 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

dcdbas: use memory_read_from_buffer()

2008-07-25T17:53:44+00:00

Signed-off-by: Akinobu Mita 
Cc: Doug Warzecha 
Cc: Zhang Rui 
Cc: Matt Domsch 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

cpumask: Replace cpumask_of_cpu with cpumask_of_cpu_ptr

2008-07-18T20:02:57+00:00

  * This patch replaces the dangerous lvalue version of cpumask_of_cpu
    with new cpumask_of_cpu_ptr macros.  These are patterned after the
    node_to_cpumask_ptr macros.

    In general terms, if there is a cpumask_of_cpu_map[] then a pointer to
    the cpumask_of_cpu_map[cpu] entry is used.  The cpumask_of_cpu_map
    is provided when there is a large NR_CPUS count, reducing
    greatly the amount of code generated and stack space used for
    cpumask_of_cpu().  The pointer to the cpumask_t value is needed for
    calling set_cpus_allowed_ptr() to reduce the amount of stack space
    needed to pass the cpumask_t value.

    If there isn't a cpumask_of_cpu_map[], then a temporary variable is
    declared and filled in with value from cpumask_of_cpu(cpu) as well as
    a pointer variable pointing to this temporary variable.  Afterwards,
    the pointer is used to reference the cpumask value.  The compiler
    will optimize out the extra dereference through the pointer as well
    as the stack space used for the pointer, resulting in identical code.

    A good example of the orthogonal usages is in net/sunrpc/svc.c:

	case SVC_POOL_PERCPU:
	{
		unsigned int cpu = m->pool_to[pidx];
		cpumask_of_cpu_ptr(cpumask, cpu);

		*oldmask = current->cpus_allowed;
		set_cpus_allowed_ptr(current, cpumask);
		return 1;
	}
	case SVC_POOL_PERNODE:
	{
		unsigned int node = m->pool_to[pidx];
		node_to_cpumask_ptr(nodecpumask, node);

		*oldmask = current->cpus_allowed;
		set_cpus_allowed_ptr(current, nodecpumask);
		return 1;
	}

Signed-off-by: Mike Travis 
Signed-off-by: Ingo Molnar