On 07/22/2014 11:41 AM, Rob Herring wrote:
On Mon, Jul 21, 2014 at 11:39 AM, Murali Karicheri wrote:
On 07/20/2014 09:44 PM, Jingoo Han wrote:
On Saturday, July 19, 2014 5:29 AM, Murali Karicheri wrote:
On 07/18/2014 03:31 PM, Rob Herring wrote:
On Fri, Jul 18, 2014 at 10:14 AM, Murali Karic
Hello,
On Tue, Jul 22, 2014 at 12:50:02PM -0600, Bjorn Helgaas wrote:
> [+cc Tejun, LKML]
>
> On Fri, Jul 11, 2014 at 09:08:24AM +0200, Thierry Reding wrote:
> > From: Thierry Reding
> >
> > Provide device-managed implementations of the request_resource() and
> > release_resource() functions. U
From: Leonid Yegoshin
Provide hooks that allow architectures with aliasing cache to align
mapping address of high pages according to their color. Such architectures
may enforce similar coloring of low- and high-memory page mappings and
reuse existing cache management functions to support highmem.
Define ARCH_PKMAP_COLORING and provide corresponding macro definitions
on cores with aliasing data cache.
Instead of single last_pkmap_nr maintain an array last_pkmap_nr_arr of
pkmap counters for each page color. Make sure that kmap maps physical
page at virtual address with color matching its phy
On 07/22/2014 11:37 AM, Rob Herring wrote:
On Fri, Jul 18, 2014 at 2:50 PM, Arnd Bergmann wrote:
On Friday 18 July 2014 14:31:39 Rob Herring wrote:
+
+ Example:
+ pcie_msi_intc: msi-interrupt-controller {
+ interrupt-controller;
+ #interrupt-ce
Existing clear_user_page and copy_user_page cannot be used with highmem
because they calculate physical page address from its virtual address
and do it incorrectly in case of high memory page mapped with
kmap_atomic. Also kmap is not needed, as most likely userspace mapping
color would be different
Use __flush_invalidate_dcache_page_alias with alias set to color of the
page physical address instead of __flush_invalidate_dcache_page: this
works for high memory pages and mapping/unmapping to the TLBTEMP area is
virtually free.
Allow building configurations with aliasing cache and highmem enabl
Map high memory pages at virtual addresses with color that match color
of their physical address. Existing cache alias management mechanisms
may be used with such pages.
Signed-off-by: Max Filippov
---
arch/xtensa/include/asm/fixmap.h | 3 ++-
arch/xtensa/include/asm/page.h | 2 ++
arch/xten
On Tue, 22 Jul 2014 18:47:07 +0200
Oleg Nesterov wrote:
> On 07/03, Steven Rostedt wrote:
> >
> > [ NOT READY FOR INCLUSION! ]
> >
> > Note, this is based off of my remove ftrace_start/stop() patch set.
>
> So I simply pulled your tree. I can't really comment these changes simply
> because I do
To support aliasing cache both kmap region sizes are multiplied by the
number of data cache colors. After that expansion page tables that cover
kmap regions may become larger than one page. Correctly allocate and
initialize page tables in this case.
Signed-off-by: Max Filippov
---
arch/xtensa/mm
It's much easier to reason about alignment and coloring of regions
located in the fixmap when fixmap index is just a PFN within the fixmap
region. Change fixmap addressing so that index 0 corresponds to
FIXADDR_START instead of the FIXADDR_TOP.
Signed-off-by: Max Filippov
---
arch/xtensa/include
This is a resend, try two...
---
Hi,
While playing around compiling the kernel i noticed the following:
fs/direct-io.c: In function ‘do_blockdev_direct_IO’:
fs/direct-io.c:1022:29: warning: ‘from’ may be used uninitialized in
this function [-Wmaybe-uninitialized]
ret = submit_page_section(di
Current definition of TLBTEMP_BASE_2 is always 32K above the
TLBTEMP_BASE_1, whereas fast_second_level_miss handler for the TLBTEMP
region analyzes virtual address bit (PAGE_SHIFT + DCACHE_ALIAS_ORDER)
to determine TLBTEMP region where the fault happened. The size of the
TLBTEMP region is also chec
Hi,
this series implements highmem support on xtensa cores with aliasing cache.
It does so by making sure that high memory pages are always mapped at
virtual addresses with color that match color of their physical address.
This involves changing the generic kmap code to make it aware of cache
col
On Tue, Jul 22, 2014 at 04:17:22PM +0300, Adrian Hunter wrote:
SNIP
> +
> +static FILE *perf_pmu__open_file(struct perf_pmu *pmu, const char *name)
> +{
> + struct stat st;
> + char path[PATH_MAX];
> + const char *sysfs;
> +
> + sysfs = sysfs__mountpoint();
> + if (!sysfs)
> +
On Tue, Jul 22, 2014 at 9:03 PM, Ian Kumlien wrote:
> This is a resend, try two...
Please see "[PATCH v3] direct-io: fix uninitialized warning in do_direct_IO()".
> ---
> Hi,
>
> While playing around compiling the kernel i noticed the following:
> fs/direct-io.c: In function ‘do_blockdev_direct_
On Tue, 22 Jul 2014 17:48:18 +0200
Daniel Vetter wrote:
> On Tue, Jul 22, 2014 at 5:42 PM, Alex Deucher wrote:
> >> Fence-based syncing between userspace queues submitted stuff through
> >> doorbells and anything submitted by the general simply wont work.
> >> Which is why I think the doorbell i
On tis, 2014-07-22 at 21:12 +0200, Richard Weinberger wrote:
> On Tue, Jul 22, 2014 at 9:03 PM, Ian Kumlien wrote:
> > This is a resend, try two...
>
> Please see "[PATCH v3] direct-io: fix uninitialized warning in
> do_direct_IO()".
That looks like a better approach, couldn't find it before i
On 07/22/2014 11:25 AM, Nick Krause wrote:
> On Tue, Jul 22, 2014 at 2:14 PM, Levente Kurusa wrote:
>> 2014-07-22 19:40 GMT+02:00 Nick Krause :
>>> [...]
>>>
>>> Tony,
>>>
>>> What is the value of Page shift then ? If you can tell me it would be
>>> much easier for
>>> me to fix this.
>>> Cheers N
On 07/22/2014 12:03 PM, Ian Kumlien wrote:
> This is a resend, try two...
>
> ---
> Hi,
>
> While playing around compiling the kernel i noticed the following:
> fs/direct-io.c: In function ‘do_blockdev_direct_IO’:
> fs/direct-io.c:1022:29: warning: ‘from’ may be used uninitialized in
> this func
Handle error condition since pnpacpi_parse_allocated_resource() and
pnpacpi_parse_resource_option_data() could return -EPERM.
Signed-off-by: Arjun Sreedharan
---
drivers/pnp/pnpacpi/core.c | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/drivers/pnp/pnpacpi/core.c b/d
On Tue, Jul 22, 2014 at 04:17:26PM +0300, Adrian Hunter wrote:
SNIP
> }
> +
> +int perf_evlist__set_tracking_event(struct perf_evlist *evlist,
> + struct perf_evsel *tracking_evsel)
> +{
> + struct perf_evsel *evsel;
> +
> + if (tracking_evsel->idx == 0)
>
Hi Michael,
On Tue, Jul 22, 2014 at 12:09:54PM +0200, Michael Grzeschik wrote:
> I think your hints all make sense. I was poking around with the
> functions we have and need to asign. Unfortunetly I ran into no clear
> picture to fix it the correct way and came up with this hacky patch.
Your patc
On Tue, 2014-07-22 at 17:30 +, Hartley Sweeten wrote:
> Joe,
>
> I think this is a false positive, at least the check message appears
> incorrect:
>
> CHECK: No space is necessary after a cast
> #66: FILE: arch/arm/mach-ep93xx/include/mach/platform.h:66:
> +static inline int crunch_init(void
On tis, 2014-07-22 at 12:20 -0700, Randy Dunlap wrote:
> On 07/22/2014 12:03 PM, Ian Kumlien wrote:
> > This is a resend, try two...
> >
> > And while the fix is simple, something along the lines of:
> > diff --git a/fs/direct-io.c b/fs/direct-io.c
> > index 98040ba..64a8286 100644
> > --- a/fs/d
Following patchset add various improvements to gta04 devicetree.
Changes from v1:
- added description to all patches
Marek Belisko (7):
arm: dts: omap3-gta04: Add nand support
arm: dts: omap3-gta04: Fix magnetometer model
arm: dts: omap3-gta04: Add wifi reset node
arm: dts: omap3-gta04: M
Add the needed sections to enable nand support on
gta04 board.
Add nand partitions information.
Signed-off-by: Marek Belisko
---
arch/arm/boot/dts/omap3-gta04.dts | 54 +++
1 file changed, 54 insertions(+)
diff --git a/arch/arm/boot/dts/omap3-gta04.dts
b/ar
gta04 is using hmc5843l not hmc5843 so fix wrong compatible
entry.
Signed-off-by: Marek Belisko
---
arch/arm/boot/dts/omap3-gta04.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/omap3-gta04.dts
b/arch/arm/boot/dts/omap3-gta04.dts
index 7d7ddd7..6d1a8d8
Define alias for lcd display present on gta04 board.
Signed-off-by: Marek Belisko
---
arch/arm/boot/dts/omap3-gta04.dts | 4
1 file changed, 4 insertions(+)
diff --git a/arch/arm/boot/dts/omap3-gta04.dts
b/arch/arm/boot/dts/omap3-gta04.dts
index 370c08b..05fd4d2 100644
--- a/arch/arm/boot
Define USB Host port mode and the PHY device.
Also provide pin multiplexer information for USB host
pins.
Signed-off-by: Marek Belisko
---
arch/arm/boot/dts/omap3-gta04.dts | 45 +++
1 file changed, 45 insertions(+)
diff --git a/arch/arm/boot/dts/omap3-gta04
Define voltages and properties for various twl4030
regulators used on gta04 board.
Signed-off-by: Marek Belisko
---
arch/arm/boot/dts/omap3-gta04.dts | 26 ++
1 file changed, 26 insertions(+)
diff --git a/arch/arm/boot/dts/omap3-gta04.dts
b/arch/arm/boot/dts/omap3-gta04
Define gpio node in tca6507 which will be used as
wifi reset pin.
Signed-off-by: Marek Belisko
---
arch/arm/boot/dts/omap3-gta04.dts | 8
1 file changed, 8 insertions(+)
diff --git a/arch/arm/boot/dts/omap3-gta04.dts
b/arch/arm/boot/dts/omap3-gta04.dts
index 6d1a8d8..5b08d93 100644
--
Because of commit: 3d495383648a7cda3ea51a1e2fa5d288581479aa
spi_gpio_pins node isn't valid anymore. Move to pmx_core2 node.
Signed-off-by: Marek Belisko
---
arch/arm/boot/dts/omap3-gta04.dts | 11 +++
1 file changed, 7 insertions(+), 4 deletions(-)
diff --git a/arch/arm/boot/dts/omap3-g
On Tue, Jul 22, 2014 at 02:25:24PM -0400, Nick Krause wrote:
>
> I understand what your saying and I should have searched for Page shift.
> In addition I am already got commits in the kernel for fix mes so I feel
> that your comment on fix mes is incorrect.
After seeing your other solutions to "f
On 07/21/2014 04:17 PM, Jason Low wrote:
On Mon, 2014-07-21 at 11:24 -0400, Waiman Long wrote:
This patch adds code to do optimistic spinning for the FUTEX_SPIN_LOCK
primitive on the futex value when the lock owner is running. It is
the same optimistic spinning technique that is done in the mute
On 07/22/2014 12:01 PM, Max Filippov wrote:
From: Leonid Yegoshin
Provide hooks that allow architectures with aliasing cache to align
mapping address of high pages according to their color. Such architectures
may enforce similar coloring of low- and high-memory page mappings and
reuse existing
On 07/21/2014 05:18 PM, Ingo Molnar wrote:
* Waiman Long wrote:
Testing done on a 4-socket Westmere-EX boxes with 40 cores (HT off)
showed the following performance data (average kops/s) with various
load factor (number of pause instructions) used in the critical
section using an userspace mut
On Mon, Jul 21, 2014 at 6:49 PM, Andy Lutomirski wrote:
> [applies on jmorris's security-next tree]
>
> This is both a cleanup and a speedup. It reduces overhead due to
> installing a trivial seccomp filter by 87%. The speedup comes from
> avoiding the full syscall tracing mechanism for filters
On Tue, Jul 22, 2014 at 05:17:29PM +0100, Pawel Moll wrote:
> That approach has been also discussed, last time in the mentioned
> thread:
>
> http://thread.gmane.org/gmane.linux.kernel/1611683/focus=1612554
> http://thread.gmane.org/gmane.linux.kernel/1611683/focus=1612554
>
> With both Ingo and
On Mon, Jul 14, 2014 at 3:31 PM, Kees Cook wrote:
> On Mon, Jul 14, 2014 at 3:24 PM, Luis R. Rodriguez wrote:
>> On Mon, Jul 14, 2014 at 02:38:13PM -0700, Kees Cook wrote:
>>> +int security_kernel_fw_from_file(struct file *file, char *buf, size_t size)
>>> +{
>>> + return security_ops->kernel
Hi again,
Playing some more with the new kernel release i noticed this:
[ 9064.008821] WARNING: CPU: 3 PID: 22929 at
drivers/gpu/drm/i915/intel_pm.c:5997 intel_display_power_put+0x12d/0x160()
[ 9064.008822] Modules linked in: uas bnep b43 mac80211 cfg80211
snd_hda_codec_hdmi btusb bluetooth 6lo
On Tuesday, July 22, 2014 12:23 PM, Joe Perches wrote:
> On Tue, 2014-07-22 at 17:30 +, Hartley Sweeten wrote:
>> Joe,
>>
>> I think this is a false positive, at least the check message appears
>> incorrect:
>>
>> CHECK: No space is necessary after a cast
>> #66: FILE: arch/arm/mach-ep93xx/in
On Tue, Jul 22, 2014 at 11:35 PM, Leonid Yegoshin
wrote:
> On 07/22/2014 12:01 PM, Max Filippov wrote:
>>
>> From: Leonid Yegoshin
>>
>> Provide hooks that allow architectures with aliasing cache to align
>> mapping address of high pages according to their color. Such architectures
>> may enforce
> gta04 is using hmc5843l not hmc5843 so fix wrong compatible
> entry.
I guess you mean hmc5883l
> Signed-off-by: Marek Belisko
> ---
> arch/arm/boot/dts/omap3-gta04.dts | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/omap3-gta04.dts
> b/arch/arm/b
[ +cc Jesse Brandeburg - similar LKML report (but on x86_64) ]
On 07/22/2014 10:51 AM, Mike Qiu wrote:
> In ata_sas_port_alloc(), it haven't initialized scsi_host field in
> ata_port, although scsi_host is in parameters list and unused in this
> function.
>
> With commit 1871ee134b73 ("libata: su
[ +cc Tejun ]
On 07/22/2014 03:26 PM, Jesse Brandeburg wrote:
> This is a repeatable panic, happens every boot (and prevents starting my
> system)
>
> .config attached, built from commit 15ba2236f3556fc01b9ca91394465152b5ea74b6
>
>
>
> [8.732993] sd 6:0:1:0: [sdb] Attached SCSI disk
> [
Jan Kara pointed out that calling ext2_xip_verify_sb() in ext2_remount()
doesn't make sense, since changing the XIP option on remount isn't
allowed. It also doesn't make sense to re-check whether blocksize is
supported since it can't change between mounts.
Replace the call to ext2_xip_verify_sb()
It takes a get_block parameter just like nobh_truncate_page() and
block_truncate_page()
Signed-off-by: Matthew Wilcox
---
fs/dax.c | 44
fs/ext2/inode.c| 2 +-
include/linux/fs.h | 4 ++--
mm/filemap_xip.c | 40 --
This is practically generic code; other filesystems will want to call
it from other places, but there's nothing ext2-specific about it.
Make it a little more generic by allowing it to take a count of the number
of bytes to zero rather than fixing it to a single page. Thanks to Dave
Hansen for sug
From: Matthew Wilcox
Since this is relating to FS_XIP, not KERNEL_XIP, it should be called
DAX instead of XIP.
Signed-off-by: Matthew Wilcox
---
drivers/block/Kconfig | 13 +++--
drivers/block/brd.c | 14 +++---
fs/Kconfig| 4 ++--
3 files changed, 16 insertions(
One of the primary uses for NV-DIMMs is to expose them as a block device
and use a filesystem to store files on the NV-DIMM. While that works,
it currently wastes memory and CPU time buffering the files in the page
cache. We have support in ext2 for bypassing the page cache, but it
has some races
On 9 July 2014 08:52, Mathias Krause wrote:
> On 9 July 2014 00:48, Dave Airlie wrote:
>> On 2 July 2014 16:15, Mathias Krause wrote:
>>> On 15 June 2014 23:02, Mathias Krause wrote:
The VFS layer handles those in the very same way, if unset. No need for
additional stubs.
Si
From: Matthew Wilcox
For DAX, we want to be able to copy between iovecs and kernel addresses
that don't necessarily have a struct page. This is a fairly simple
rearrangement for bvec iters to kmap the pages outside and pass them in,
but for user iovecs it gets more complicated because we might t
Currently COW of an XIP file is done by first bringing in a read-only
mapping, then retrying the fault and copying the page. It is much more
efficient to tell the fault handler that a COW is being attempted (by
passing in the pre-allocated page in the vm_fault structure), and allow
the handler to
On Tue, Jul 22, 2014 at 9:46 PM, Peter Meerwald wrote:
>
>> gta04 is using hmc5843l not hmc5843 so fix wrong compatible
>> entry.
>
> I guess you mean hmc5883l
Ah right. Thanks. I'll post update version.
>
>> Signed-off-by: Marek Belisko
>> ---
>> arch/arm/boot/dts/omap3-gta04.dts | 2 +-
>> 1 f
Use an inode flag to tag inodes which should avoid using the page cache.
Convert ext2 to use it instead of mapping_is_xip().
Signed-off-by: Matthew Wilcox
Reviewed-by: Jan Kara
---
fs/ext2/inode.c| 9 ++---
fs/ext2/xip.h | 2 --
include/linux/fs.h | 6 ++
3 files changed, 12 in
The 'pfn' returned by axonram was completely bogus, and has been since
2008.
Signed-off-by: Matthew Wilcox
Reviewed-by: Jan Kara
---
arch/powerpc/sysdev/axonram.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/powerpc/sysdev/axonram.c b/arch/powerpc/sysdev/axonram.c
in
Replace ext2_use_xip() with test_opt(XIP) which expands to the same code
Signed-off-by: Matthew Wilcox
---
fs/ext2/ext2.h | 4
fs/ext2/inode.c | 2 +-
fs/ext2/namei.c | 4 ++--
3 files changed, 7 insertions(+), 3 deletions(-)
diff --git a/fs/ext2/ext2.h b/fs/ext2/ext2.h
index d9a17d0..5ec
Pagecache faults recheck i_size after taking the page lock to ensure that
the fault didn't race against a truncate. We don't have a page to lock
in the XIP case, so use the i_mmap_mutex instead. It is locked in the
truncate path in unmap_mapping_range() after updating i_size. So while
we hold it
All callers of get_xip_mem() are now gone. Remove checks for it,
initialisers of it, documentation of it and the only implementation of it.
Also remove mm/filemap_xip.c as it is now empty.
Signed-off-by: Matthew Wilcox
---
Documentation/filesystems/Locking | 3 ---
fs/exofs/inode.c
Use the generic AIO infrastructure instead of custom read and write
methods. In addition to giving us support for AIO, this adds the missing
locking between read() and truncate().
Signed-off-by: Matthew Wilcox
Reviewed-by: Ross Zwisler
Reviewed-by: Jan Kara
---
fs/Makefile| 1 +
fs/
We shouldn't need a special address_space_operations any more
Signed-off-by: Matthew Wilcox
---
fs/ext2/ext2.h | 1 -
fs/ext2/inode.c | 7 +--
fs/ext2/namei.c | 4 ++--
3 files changed, 3 insertions(+), 9 deletions(-)
diff --git a/fs/ext2/ext2.h b/fs/ext2/ext2.h
index b30c3bd..b8b1c11 1006
From: Matthew Wilcox
Based on the original XIP documentation, this documents the current
state of affairs, and includes instructions on how users can enable DAX
if their devices and kernel support it.
Signed-off-by: Matthew Wilcox
Reviewed-by: Randy Dunlap
---
Documentation/filesystems/dax.tx
The fewer Kconfig options we have the better. Use the generic
CONFIG_FS_DAX to enable XIP support in ext2 as well as in the core.
Signed-off-by: Matthew Wilcox
---
fs/Kconfig | 21 ++---
fs/Makefile| 2 +-
fs/ext2/Kconfig| 11 ---
fs/ext2/ext2.h
This new function allows us to support hole-punch for XIP files by zeroing
a partial page, as opposed to the xip_truncate_page() function which can
only truncate to the end of the page. Reimplement xip_truncate_page() as
a macro that calls xip_zero_page_range().
Signed-off-by: Matthew Wilcox
[po
Instead of calling aops->get_xip_mem from the fault handler, the
filesystem passes a get_block_t that is used to find the appropriate
blocks.
Signed-off-by: Matthew Wilcox
Reviewed-by: Jan Kara
---
fs/dax.c | 221 +
fs/ext2/file.c
To help people transition, accept the 'xip' mount option (and report it
in /proc/mounts), but print a message encouraging people to switch over
to the 'dax' option.
---
fs/ext2/ext2.h | 13 +++--
fs/ext2/file.c | 2 +-
fs/ext2/inode.c | 6 +++---
fs/ext2/namei.c | 8
fs/ext2/
From: Matthew Wilcox
vm_insert_mixed() will fail if there is already a valid PTE at that
location. The DAX code would rather replace the previous value with
the new PTE.
Signed-off-by: Matthew Wilcox
---
include/linux/mm.h | 8 ++--
mm/memory.c| 34 +--
From: Ross Zwisler
This is a port of the DAX functionality found in the current version of
ext2.
Signed-off-by: Ross Zwisler
Reviewed-by: Andreas Dilger
[heavily tweaked]
Signed-off-by: Matthew Wilcox
---
Documentation/filesystems/dax.txt | 1 +
Documentation/filesystems/ext4.txt | 2 ++
In order to support accesses to larger chunks of memory, pass in a
'size' parameter (counted in bytes), and return the amount available at
that address.
Support partitioning the underlying block device through a new helper
function, bdev_direct_access(), since partition handling should be done
in
These files are now empty, so delete them
Signed-off-by: Matthew Wilcox
---
fs/ext2/Makefile | 1 -
fs/ext2/inode.c | 1 -
fs/ext2/namei.c | 1 -
fs/ext2/super.c | 1 -
fs/ext2/xip.c| 15 ---
fs/ext2/xip.h| 16
6 files changed, 35 deletions(-)
delete
On Wed, Jul 16, 2014 at 01:14:08PM -0600, Alex Williamson wrote:
> There are numerous ATI/AMD GPUs available that report that they
> support a PM reset (NoSoftRst-) but for which such a reset has no
> apparent effect on the device. These devices continue to display the
> same framebuffer across PM
gta04 is using hmc5883l not hmc5843 so fix wrong compatible
entry.
Signed-off-by: Marek Belisko
---
arch/arm/boot/dts/omap3-gta04.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/omap3-gta04.dts
b/arch/arm/boot/dts/omap3-gta04.dts
index 7d7ddd7..4067495
On Tue, Jul 22, 2014 at 09:28:39AM +0200, Arnd Bergmann wrote:
> On Monday 21 July 2014 14:07:57 Brian Norris wrote:
> > diff --git a/arch/arm/mach-bcm/Kconfig b/arch/arm/mach-bcm/Kconfig
> > index 0073633e7699..9782e8d80647 100644
> > --- a/arch/arm/mach-bcm/Kconfig
> > +++ b/arch/arm/mach-bcm/Kco
Declare several functions and variables as static in order
to address the following sparse warnings
drivers/staging/lustre/lnet/lnet/acceptor.c:49:3: warning: symbol
'lnet_acceptor_state' was not declared. Should it be static?
drivers/staging/lustre/lnet/lnet/acceptor.c:79:1: warning: symbol
'ln
There is one function in aes_ccmp.c which is exported, but sparse sees
it unexported because it doesn't include the header that exports it.
Signed-off-by: Guillaume Clement
---
drivers/staging/vt6655/aes_ccmp.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/vt6655/aes_ccmp.c
On Fri, Jul 11, 2014 at 10:36 AM, Cyrill Gorcunov wrote:
> On Wed, Jul 09, 2014 at 07:06:04PM +0400, Cyrill Gorcunov wrote:
>>
>> Thanks a lot for comments, Kees! I tend to agre, leaving off the @prctl_map
>> variable out of macros should make code also shorter, I'll update that's
>> not the probl
Put extern declarations in cxt1e1_common.h to reduce sparse warnings for
linux.c:
drivers/staging/cxt1e1/linux.c:86:13: warning:
symbol 'error_flag' was not declared. Should it be static?
drivers/staging/cxt1e1/linux.c:91:13: warning:
symbol 'cxt1e1_max_mru' was not declared. Should it be stat
The IEEE11hbMgrRxAction is not exported and never used. Deleting it
allows to delete other functions that were only used by
IEEE11hbMgrRxAction.
This allows to fix several warnings reported by sparse.
Signed-off-by: Guillaume Clement
---
drivers/staging/vt6655/IEEE11h.c | 175 --
Sparse reports that MimeThread is not used. Actually, it can be used
if THREAD is defined. By enclosing the MimeThread function into the
same #ifdef as the caller of MimeThread, this fixes the sparse
warnings.
Signed-off-by: Guillaume Clement
---
drivers/staging/vt6655/device_main.c | 6 +-
Some variables are used only in the context of their .c file, which
gives warnings with sparse.
Signed-off-by: Guillaume Clement
---
drivers/staging/vt6655/baseband.c| 14 +++---
drivers/staging/vt6655/bssdb.c | 4 ++--
drivers/staging/vt6655/card.c| 6 +++---
drivers
PortOffset was an unsigned long, but used as an pointer to io
memory. Sometimes it was not properly cast before use, which caused
many warning by sparse.
By updating its type to void __iomem *, and reflecting the changes
where it is needed, this removes most of those warnings.
Signed-off-by: Guil
Hello,
Can you please test the following patch?
diff --git a/drivers/ata/libata-core.c b/drivers/ata/libata-core.c
index d19c37a7..773f4e6 100644
--- a/drivers/ata/libata-core.c
+++ b/drivers/ata/libata-core.c
@@ -4798,9 +4798,8 @@ void swap_buf_le16(u16 *buf, unsigned int buf_words)
static stru
Those patches fix warnings reported by sparse on the vt6655
driver. This will allow to more easily find real errors with sparse
later if one appears.
Note that checkpatch does return an error for patch 1 and 2 (line over
80 characters) but the line length was not increased by this patch. If
neede
On Tue, Jul 22, 2014 at 2:14 AM, AKASHI Takahiro
wrote:
> Arm64 holds a syscall number in w8(x8) register. Ptrace tracer may change
> its value either to:
> * any valid syscall number to alter a system call, or
> * -1 to skip a system call
>
> This patch implements this behavior by reloading t
On Tue, Jul 22, 2014 at 2:14 AM, AKASHI Takahiro
wrote:
> (Please apply this patch after my audit patch in order to avoid some
> conflict on arm64/Kconfig.)
>
> This patch enables secure computing (system call filtering) on arm64.
> System calls can be allowed or denied by loaded bpf-style rules.
On Tue, Jul 22, 2014 at 04:07:51PM -0400, Jeff Oczek wrote:
> Put extern declarations in cxt1e1_common.h to reduce sparse warnings for
> linux.c:
> drivers/staging/cxt1e1/linux.c:86:13: warning:
> symbol 'error_flag' was not declared. Should it be static?
> drivers/staging/cxt1e1/linux.c:91:13:
On Tue, Jul 22, 2014 at 04:17:41PM +0900, Alexandre Courbot wrote:
> As GPIO descriptors are not going to remain unique anymore, having this
> function public is not safe. Restrain its use to gpiolib since we have
> no user outside of it.
>
If I implement a gpio chip driver built as module, and I
On 07/21/2014 09:01 PM, Thomas Gleixner wrote:
On Mon, 21 Jul 2014, Darren Hart wrote:
On 7/21/14, 14:47, "Thomas Gleixner" wrote:
On Mon, 21 Jul 2014, Andy Lutomirski wrote:
On Mon, Jul 21, 2014 at 2:27 PM, Peter Zijlstra
wrote:
All this is predicated on the fact that syscalls are 'expensi
On Tue, 2014-07-22 at 13:55 -0600, Bjorn Helgaas wrote:
> On Wed, Jul 16, 2014 at 01:14:08PM -0600, Alex Williamson wrote:
> > There are numerous ATI/AMD GPUs available that report that they
> > support a PM reset (NoSoftRst-) but for which such a reset has no
> > apparent effect on the device. Th
On 07/22/2014 05:59 AM, Thomas Gleixner wrote:
On Tue, 22 Jul 2014, Peter Zijlstra wrote:
On Tue, Jul 22, 2014 at 10:39:17AM +0200, Thomas Gleixner wrote:
On Tue, 22 Jul 2014, Peter Zijlstra wrote:
Anyway, there is one big fail in the entire futex stack that we 'need'
to sort some day and that
From: Quentin Armitage
Date: Tue, 22 Jul 2014 09:10:10 +0100
> Currently, although IP_MULTICAST_ALL and IP_MSFILTER ioctl calls succeed on
> raw sockets, there is no code to implement the functionality on received
> packets; it is only implemented for UDP sockets. The raw(7) man page states:
> "I
On 07/03/2014 09:01 AM, nick.d...@itdev.co.uk wrote:
> Hi Dimitry-
>
> Here is another set of atmel_mxt_ts patches for upstream. There are some
> really useful new features, but I hope nothing too controversial.
Unfortunately, I still can't get these to work on my system.
Per your "Re: atmel_mxt
On Thu, Jul 17, 2014 at 11:22:17AM -0700, Andy Lutomirski wrote:
> Currently, init_std_data contains its own logic for using arch
> random sources. This logic is a bit strange: it reads one long of
> arch random data per byte of internal state.
This isn't true. Check out the init_std_data() a bi
On Tue, Jul 22, 2014 at 01:07:51PM -0700, Kees Cook wrote:
> On Fri, Jul 11, 2014 at 10:36 AM, Cyrill Gorcunov wrote:
> > On Wed, Jul 09, 2014 at 07:06:04PM +0400, Cyrill Gorcunov wrote:
> >>
> >> Thanks a lot for comments, Kees! I tend to agre, leaving off the @prctl_map
> >> variable out of macr
On 07/03/2014 09:01 AM, nick.d...@itdev.co.uk wrote:
> From: Stephen Warren
> diff --git a/Documentation/devicetree/bindings/input/atmel,maxtouch.txt
> b/Documentation/devicetree/bindings/input/atmel,maxtouch.txt
> + touch@4b {
> + compatible = "atmel,maxtouch";
> +
On Tue, 22 Jul 2014 09:17:45 +0200 Bart Van Assche wrote:
> Evaluating a macro argument only if certain configuration options
> have been selected is confusing and error-prone. Hence always
> evaluate the second argument of spin_lock_nested() and
> spin_lock_nest_lock().
>
> An intentional side
+ Sebastian
On Tue, Jul 22, 2014 at 09:35:45AM +0200, Arnd Bergmann wrote:
> On Monday 21 July 2014 14:07:55 Brian Norris wrote:
> > I'm taking over the latest resubmission of this patch series.
> > There are a few moderate changes for v8 (noted below), but we
> > are waiting mostly for an Ack for
On Tue, Jul 22, 2014 at 6:59 AM, Theodore Ts'o wrote:
> On Thu, Jul 17, 2014 at 11:22:17AM -0700, Andy Lutomirski wrote:
>> Currently, init_std_data contains its own logic for using arch
>> random sources. This logic is a bit strange: it reads one long of
>> arch random data per byte of internal
On Tue, 22 Jul 2014, Waiman Long wrote:
> On 07/22/2014 05:59 AM, Thomas Gleixner wrote:
> > On Tue, 22 Jul 2014, Peter Zijlstra wrote:
> > > On Tue, Jul 22, 2014 at 10:39:17AM +0200, Thomas Gleixner wrote:
> > > > On Tue, 22 Jul 2014, Peter Zijlstra wrote:
> > > > > Anyway, there is one big fail
601 - 700 of 1126 matches
Mail list logo