Re: [PATCH] arm64: add kc_offset_to_vaddr and kc_vaddr_to_offset macro

2015-09-05 Thread yalin wang
ok i will send a V2 patch for this change. > On Sep 2, 2015, at 18:24, Will Deacon <will.dea...@arm.com> wrote: > > On Sun, Aug 30, 2015 at 07:19:59AM +0100, yalin wang wrote: >> This patch add kc_offset_to_vaddr() and kc_vaddr_to_offset(), >> the default version doe

[PATCH V2] arm64: add kc_offset_to_vaddr and kc_vaddr_to_offset macro

2015-09-05 Thread yalin wang
This patch add kc_offset_to_vaddr() and kc_vaddr_to_offset(), the default version doesn't work on arm64, because arm64 kernel address is below the PAGE_OFFSET, like module address and vmemmap address are all below PAGE_OFFSET address. Signed-off-by: yalin wang <yalin.wang2...@gmail.com> ---

Re: [PATCH] arm64: add kc_offset_to_vaddr and kc_vaddr_to_offset macro

2015-08-31 Thread yalin wang
Ping ? > On Aug 30, 2015, at 14:19, yalin wang wrote: > > This patch add kc_offset_to_vaddr() and kc_vaddr_to_offset(), > the default version doesn't work on arm64, because arm64 kernel address > is below the PAGE_OFFSET, like module address and vmemmap address are > all

Re: [PATCH 1/2] x86/bitops: implement __test_bit

2015-08-31 Thread yalin wang
> On Aug 31, 2015, at 15:59, Ingo Molnar wrote: > > > * Michael S. Tsirkin wrote: > >> On Sun, Aug 30, 2015 at 11:13:20PM -0700, H. Peter Anvin wrote: >>> Presumably because gcc can't generate bt... whether or not it is worth it >>> is another matter. >>> >>> On August 30, 2015 11:05:49 PM

Re: [PATCH] arm64: add kc_offset_to_vaddr and kc_vaddr_to_offset macro

2015-08-31 Thread yalin wang
Ping ? > On Aug 30, 2015, at 14:19, yalin wang <yalin.wang2...@gmail.com> wrote: > > This patch add kc_offset_to_vaddr() and kc_vaddr_to_offset(), > the default version doesn't work on arm64, because arm64 kernel address > is below the PAGE_OFFSET, like module addres

Re: [PATCH 1/2] x86/bitops: implement __test_bit

2015-08-31 Thread yalin wang
> On Aug 31, 2015, at 15:59, Ingo Molnar wrote: > > > * Michael S. Tsirkin wrote: > >> On Sun, Aug 30, 2015 at 11:13:20PM -0700, H. Peter Anvin wrote: >>> Presumably because gcc can't generate bt... whether or not it is worth it >>> is another matter. >>>

Re: [PATCH] task_work: remove fifo ordering guarantee

2015-08-30 Thread yalin wang
> On Aug 30, 2015, at 01:08, Linus Torvalds > wrote: > > On Sat, Aug 29, 2015 at 7:11 AM, Eric Dumazet wrote: >> >> If this needs to be kept, maybe then add following, to make sure >> we flush the list at most every BITS_PER_LONG files > > Hmm. > > I'm wondering if we should just make

[PATCH] arm64: add kc_offset_to_vaddr and kc_vaddr_to_offset macro

2015-08-30 Thread yalin wang
This patch add kc_offset_to_vaddr() and kc_vaddr_to_offset(), the default version doesn't work on arm64, because arm64 kernel address is below the PAGE_OFFSET, like module address and vmemmap address are all below PAGE_OFFSET address. Signed-off-by: yalin wang --- arch/arm64/include/asm

[PATCH] arm64: add kc_offset_to_vaddr and kc_vaddr_to_offset macro

2015-08-30 Thread yalin wang
This patch add kc_offset_to_vaddr() and kc_vaddr_to_offset(), the default version doesn't work on arm64, because arm64 kernel address is below the PAGE_OFFSET, like module address and vmemmap address are all below PAGE_OFFSET address. Signed-off-by: yalin wang yalin.wang2...@gmail.com --- arch

Re: [PATCH] task_work: remove fifo ordering guarantee

2015-08-30 Thread yalin wang
On Aug 30, 2015, at 01:08, Linus Torvalds torva...@linux-foundation.org wrote: On Sat, Aug 29, 2015 at 7:11 AM, Eric Dumazet eric.duma...@gmail.com wrote: If this needs to be kept, maybe then add following, to make sure we flush the list at most every BITS_PER_LONG files Hmm. I'm

[PATCH] arm64: ioremap: change ioremap prototype and add ioremap_cache macro

2015-08-27 Thread yalin wang
-by: yalin wang --- arch/arm64/include/asm/io.h | 5 +++-- arch/arm64/mm/ioremap.c | 6 +++--- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/arch/arm64/include/asm/io.h b/arch/arm64/include/asm/io.h index 44be1e0..500e09f 100644 --- a/arch/arm64/include/asm/io.h +++ b/arch

[PATCH] arm64: ioremap: change ioremap prototype and add ioremap_cache macro

2015-08-27 Thread yalin wang
-by: yalin wang yalin.wang2...@gmail.com --- arch/arm64/include/asm/io.h | 5 +++-- arch/arm64/mm/ioremap.c | 6 +++--- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/arch/arm64/include/asm/io.h b/arch/arm64/include/asm/io.h index 44be1e0..500e09f 100644 --- a/arch/arm64/include

Re: Problems loading firmware using built-in drivers with kernels that use initramfs.

2015-08-25 Thread yalin wang
> On Aug 26, 2015, at 04:26, Linus Torvalds > wrote: > > On Tue, Aug 25, 2015 at 12:58 PM, Dmitry Torokhov > wrote: >> >> Either build firmware in the kernel or ramdisk (so it is always >> available), or make sure request_firmware() calls are not in driver's >> probe() paths. > > The

[PATCH] arm: insn: use set_fixmap_offset to make it more clear

2015-08-25 Thread yalin wang
A little change to patch_map() function, use set_fixmap_offset() to make code more clear. Signed-off-by: yalin wang --- arch/arm/kernel/patch.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/arch/arm/kernel/patch.c b/arch/arm/kernel/patch.c index 69bda1a..5779105

Re: [PATCH] dma/ipu: change ipu_irq_handler() to remove compile warning

2015-08-25 Thread yalin wang
> On Aug 25, 2015, at 16:07, Thomas Gleixner wrote: > > On Tue, 25 Aug 2015, yalin wang wrote: >> Change ipu_irq_handler() to avoid gcc warning: >> >> drivers/dma/ipu/ipu_irq.c:305:4: warning: 'irq' may be used >> uninitialized in this function [-Wmaybe-uninit

[PATCH V2] dma/ipu: change ipu_irq_handler() to remove compile warning

2015-08-25 Thread yalin wang
Change ipu_irq_handler() to avoid gcc warning: drivers/dma/ipu/ipu_irq.c:305:4: warning: 'irq' may be used uninitialized in this function [-Wmaybe-uninitialized] generic_handle_irq(irq); Signed-off-by: yalin wang --- drivers/dma/ipu/ipu_irq.c | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH] dma/ipu: change ipu_irq_handler() to remove compile warning

2015-08-25 Thread yalin wang
Change ipu_irq_handler() to avoid gcc warning: drivers/dma/ipu/ipu_irq.c:305:4: warning: 'irq' may be used uninitialized in this function [-Wmaybe-uninitialized] generic_handle_irq(irq); Signed-off-by: yalin wang --- drivers/dma/ipu/ipu_irq.c | 6 ++ 1 file changed, 2 insertions(+), 4

[RFC] arm: change some function and data to init section

2015-08-25 Thread yalin wang
This patch change some function and the related data to init section, they are only called by free_initmem(), can be freed safely after boot up. Signed-off-by: yalin wang --- arch/arm/mm/init.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/arch/arm/mm/init.c b

Re: Problems loading firmware using built-in drivers with kernels that use initramfs.

2015-08-25 Thread yalin wang
On Aug 26, 2015, at 04:26, Linus Torvalds torva...@linux-foundation.org wrote: On Tue, Aug 25, 2015 at 12:58 PM, Dmitry Torokhov dmitry.torok...@gmail.com wrote: Either build firmware in the kernel or ramdisk (so it is always available), or make sure request_firmware() calls are not in

[RFC] arm: change some function and data to init section

2015-08-25 Thread yalin wang
This patch change some function and the related data to init section, they are only called by free_initmem(), can be freed safely after boot up. Signed-off-by: yalin wang yalin.wang2...@gmail.com --- arch/arm/mm/init.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff

[PATCH] dma/ipu: change ipu_irq_handler() to remove compile warning

2015-08-25 Thread yalin wang
Change ipu_irq_handler() to avoid gcc warning: drivers/dma/ipu/ipu_irq.c:305:4: warning: 'irq' may be used uninitialized in this function [-Wmaybe-uninitialized] generic_handle_irq(irq); Signed-off-by: yalin wang yalin.wang2...@gmail.com --- drivers/dma/ipu/ipu_irq.c | 6 ++ 1 file

[PATCH V2] dma/ipu: change ipu_irq_handler() to remove compile warning

2015-08-25 Thread yalin wang
Change ipu_irq_handler() to avoid gcc warning: drivers/dma/ipu/ipu_irq.c:305:4: warning: 'irq' may be used uninitialized in this function [-Wmaybe-uninitialized] generic_handle_irq(irq); Signed-off-by: yalin wang yalin.wang2...@gmail.com --- drivers/dma/ipu/ipu_irq.c | 2 +- 1 file changed

Re: [PATCH] dma/ipu: change ipu_irq_handler() to remove compile warning

2015-08-25 Thread yalin wang
On Aug 25, 2015, at 16:07, Thomas Gleixner t...@linutronix.de wrote: On Tue, 25 Aug 2015, yalin wang wrote: Change ipu_irq_handler() to avoid gcc warning: drivers/dma/ipu/ipu_irq.c:305:4: warning: 'irq' may be used uninitialized in this function [-Wmaybe-uninitialized

[PATCH] arm: insn: use set_fixmap_offset to make it more clear

2015-08-25 Thread yalin wang
A little change to patch_map() function, use set_fixmap_offset() to make code more clear. Signed-off-by: yalin wang yalin.wang2...@gmail.com --- arch/arm/kernel/patch.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/arch/arm/kernel/patch.c b/arch/arm/kernel/patch.c

Re: [PATCH] ipmi: add of_device_id in MODULE_DEVICE_TABLE

2015-08-24 Thread yalin wang
> On Aug 25, 2015, at 08:48, Corey Minyard wrote: > > Well, I should have compile tested first. On x86_64: > > > CC [M] drivers/char/ipmi/ipmi_si_intf.o > In file included from ../drivers/char/ipmi/ipmi_si_intf.c:42:0: > ../drivers/char/ipmi/ipmi_si_intf.c:2804:25: error: ‘ipmi_match’ >

Re: [RFC] fbdev/riva:change to use generice function to implement reverse_order()

2015-08-24 Thread yalin wang
> On Aug 22, 2015, at 15:53, Afzal Mohammed wrote: > > Hi, > > On Fri, Aug 21, 2015 at 11:01:41AM +0300, Tomi Valkeinen wrote: > Possibly the patches are still good for x86 also, but that needs to be proven. >>> not exactly, because x86_64 don’t have hardware instruction to do

Re: [x86] copy_from{to}_user question

2015-08-24 Thread yalin wang
> On Aug 22, 2015, at 17:05, Borislav Petkov wrote: > > On Fri, Aug 21, 2015 at 02:06:16PM -0700, H. Peter Anvin wrote: >> What I'm saying is that we do do STAC, which *disables* SMAP. We have >> to do that because one pointer is known to be a user space pointer. >> >> However, we currently

Re: [x86] copy_from{to}_user question

2015-08-24 Thread yalin wang
On Aug 22, 2015, at 17:05, Borislav Petkov b...@suse.de wrote: On Fri, Aug 21, 2015 at 02:06:16PM -0700, H. Peter Anvin wrote: What I'm saying is that we do do STAC, which *disables* SMAP. We have to do that because one pointer is known to be a user space pointer. However, we currently

Re: [RFC] fbdev/riva:change to use generice function to implement reverse_order()

2015-08-24 Thread yalin wang
On Aug 22, 2015, at 15:53, Afzal Mohammed afzal.mohd...@gmail.com wrote: Hi, On Fri, Aug 21, 2015 at 11:01:41AM +0300, Tomi Valkeinen wrote: Possibly the patches are still good for x86 also, but that needs to be proven. not exactly, because x86_64 don’t have hardware instruction to

Re: [PATCH] ipmi: add of_device_id in MODULE_DEVICE_TABLE

2015-08-24 Thread yalin wang
On Aug 25, 2015, at 08:48, Corey Minyard miny...@acm.org wrote: Well, I should have compile tested first. On x86_64: CC [M] drivers/char/ipmi/ipmi_si_intf.o In file included from ../drivers/char/ipmi/ipmi_si_intf.c:42:0: ../drivers/char/ipmi/ipmi_si_intf.c:2804:25: error:

Re: [RFC] fbdev/riva:change to use generice function to implement reverse_order()

2015-08-21 Thread yalin wang
> On Aug 21, 2015, at 14:41, Tomi Valkeinen wrote: > > > > On 20/08/15 14:30, yalin wang wrote: >> >>> On Aug 20, 2015, at 19:02, Tomi Valkeinen wrote: >>> >>> >>> On 10/08/15 13:12, yalin wang wrote: >>>> This change

[RFC V2] fbdev/riva:change to use generice function to implement reverse_order()

2015-08-21 Thread yalin wang
This change to use swab32(bitrev32()) to implement reverse_order() function, have better performance on some platforms. Signed-off-by: yalin wang --- drivers/video/fbdev/riva/fbdev.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/video/fbdev/riva/fbdev.c b

[RFC V2] fbdev/riva:change to use generice function to implement reverse_order()

2015-08-21 Thread yalin wang
This change to use swab32(bitrev32()) to implement reverse_order() function, have better performance on some platforms. Signed-off-by: yalin wang yalin.wang2...@gmail.com --- drivers/video/fbdev/riva/fbdev.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/video

Re: [RFC] fbdev/riva:change to use generice function to implement reverse_order()

2015-08-21 Thread yalin wang
On Aug 21, 2015, at 14:41, Tomi Valkeinen tomi.valkei...@ti.com wrote: On 20/08/15 14:30, yalin wang wrote: On Aug 20, 2015, at 19:02, Tomi Valkeinen tomi.valkei...@ti.com wrote: On 10/08/15 13:12, yalin wang wrote: This change to use swab32(bitrev32()) to implement reverse_order

[RFC V2] fbdev/nvidia:change reverse_order() macro

2015-08-20 Thread yalin wang
This change reverse_order() to swab32(bitrev32()), so that it can have better performance on some platforms. Signed-off-by: yalin wang --- drivers/video/fbdev/nvidia/nv_local.h | 21 +++-- 1 file changed, 11 insertions(+), 10 deletions(-) diff --git a/drivers/video/fbdev/nvidia

Re: [RFC] fbdev/riva:change to use generice function to implement reverse_order()

2015-08-20 Thread yalin wang
> On Aug 20, 2015, at 19:02, Tomi Valkeinen wrote: > > > On 10/08/15 13:12, yalin wang wrote: >> This change to use swab32(bitrev32()) to implement reverse_order() >> function, have better performance on some platforms. > > Which platforms? Presuming you tested

[RFC V2] fs/kcore: change copy_to_user to copy_in_user

2015-08-20 Thread yalin wang
ge kernel pagetables, and there is not any sync between read_kcore() and kernel pagetable change. I change to use copy_in_user(), so that can check both source and destination address, avoid the kernel panic() in some unlucky circumstance. Signed-off-by: yalin wang --- fs/proc/kcore.c | 8 ++-- 1 file c

Re: [RFC] fs/kcore: change copy_to_user to copy_in_user

2015-08-20 Thread yalin wang
> On Aug 20, 2015, at 17:58, Frans Klaver wrote: > > On Thu, Aug 20, 2015 at 10:59 AM, yalin wang wrote: >> the copy_to_user() here expect can fix the fault on both kernel and >> user address, this is not true on other platforms except x86, >> change to user copy_i

[PATCH] nvdimm: change to use generic kvfree()

2015-08-20 Thread yalin wang
Signed-off-by: yalin wang --- drivers/nvdimm/dimm_devs.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/nvdimm/dimm_devs.c b/drivers/nvdimm/dimm_devs.c index c05eb80..651b8d1 100644 --- a/drivers/nvdimm/dimm_devs.c +++ b/drivers/nvdimm/dimm_devs.c @@ -241,10

[RFC] fs/kcore: change copy_to_user to copy_in_user

2015-08-20 Thread yalin wang
the copy_to_user() here expect can fix the fault on both kernel and user address, this is not true on other platforms except x86, change to user copy_in_user() so that can detect the page fault, work as expected. Signed-off-by: yalin wang --- fs/proc/kcore.c | 8 ++-- 1 file changed, 6

Re: [x86] copy_from{to}_user question

2015-08-20 Thread yalin wang
> On Aug 17, 2015, at 12:16, Borislav Petkov wrote: > > On Mon, Aug 17, 2015 at 11:27:01AM +0800, yalin wang wrote: >> i just want the x86 copy_from{to,in}_user() function have >> the same behaviour as other platforms. > > Back to the original question from 2 mails

[RFC] fbdev/nvidia:change reverse_order() macro

2015-08-20 Thread yalin wang
This change reverse_order() to swab32(bitrev32()), so that it can have better performance on some platforms. Signed-off-by: yalin wang --- drivers/video/fbdev/nvidia/nv_accel.c | 4 ++-- drivers/video/fbdev/nvidia/nv_local.h | 13 - 2 files changed, 6 insertions(+), 11 deletions

[PATCH] nvdimm: change to use generic kvfree()

2015-08-20 Thread yalin wang
Signed-off-by: yalin wang yalin.wang2...@gmail.com --- drivers/nvdimm/dimm_devs.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/nvdimm/dimm_devs.c b/drivers/nvdimm/dimm_devs.c index c05eb80..651b8d1 100644 --- a/drivers/nvdimm/dimm_devs.c +++ b/drivers/nvdimm

Re: [x86] copy_from{to}_user question

2015-08-20 Thread yalin wang
On Aug 17, 2015, at 12:16, Borislav Petkov b...@suse.de wrote: On Mon, Aug 17, 2015 at 11:27:01AM +0800, yalin wang wrote: i just want the x86 copy_from{to,in}_user() function have the same behaviour as other platforms. Back to the original question from 2 mails ago: How else would

[RFC] fs/kcore: change copy_to_user to copy_in_user

2015-08-20 Thread yalin wang
the copy_to_user() here expect can fix the fault on both kernel and user address, this is not true on other platforms except x86, change to user copy_in_user() so that can detect the page fault, work as expected. Signed-off-by: yalin wang yalin.wang2...@gmail.com --- fs/proc/kcore.c | 8

Re: [RFC] fs/kcore: change copy_to_user to copy_in_user

2015-08-20 Thread yalin wang
On Aug 20, 2015, at 17:58, Frans Klaver franskla...@gmail.com wrote: On Thu, Aug 20, 2015 at 10:59 AM, yalin wang yalin.wang2...@gmail.com wrote: the copy_to_user() here expect can fix the fault on both kernel and user address, this is not true on other platforms except x86, change to user

Re: [RFC] fbdev/riva:change to use generice function to implement reverse_order()

2015-08-20 Thread yalin wang
On Aug 20, 2015, at 19:02, Tomi Valkeinen tomi.valkei...@ti.com wrote: On 10/08/15 13:12, yalin wang wrote: This change to use swab32(bitrev32()) to implement reverse_order() function, have better performance on some platforms. Which platforms? Presuming you tested this, roughly how

[RFC V2] fs/kcore: change copy_to_user to copy_in_user

2015-08-20 Thread yalin wang
, and there is not any sync between read_kcore() and kernel pagetable change. I change to use copy_in_user(), so that can check both source and destination address, avoid the kernel panic() in some unlucky circumstance. Signed-off-by: yalin wang yalin.wang2...@gmail.com --- fs/proc/kcore.c | 8 ++-- 1 file

[RFC V2] fbdev/nvidia:change reverse_order() macro

2015-08-20 Thread yalin wang
This change reverse_order() to swab32(bitrev32()), so that it can have better performance on some platforms. Signed-off-by: yalin wang yalin.wang2...@gmail.com --- drivers/video/fbdev/nvidia/nv_local.h | 21 +++-- 1 file changed, 11 insertions(+), 10 deletions(-) diff --git

[RFC] fbdev/nvidia:change reverse_order() macro

2015-08-20 Thread yalin wang
This change reverse_order() to swab32(bitrev32()), so that it can have better performance on some platforms. Signed-off-by: yalin wang yalin.wang2...@gmail.com --- drivers/video/fbdev/nvidia/nv_accel.c | 4 ++-- drivers/video/fbdev/nvidia/nv_local.h | 13 - 2 files changed, 6

Re: [x86] copy_from{to}_user question

2015-08-16 Thread yalin wang
> On Aug 14, 2015, at 00:43, Borislav Petkov wrote: > > On Thu, Aug 13, 2015 at 06:04:54PM +0800, yalin wang wrote: >> we store type into one fix register, for example r12 , >> then in fix up code, we can know the exception is caused by copy_from >> copy_to or copy_in

Re: [x86] copy_from{to}_user question

2015-08-16 Thread yalin wang
On Aug 14, 2015, at 00:43, Borislav Petkov b...@suse.de wrote: On Thu, Aug 13, 2015 at 06:04:54PM +0800, yalin wang wrote: we store type into one fix register, for example r12 , then in fix up code, we can know the exception is caused by copy_from copy_to or copy_in user function by check

Re: [x86] copy_from{to}_user question

2015-08-13 Thread yalin wang
> On Aug 12, 2015, at 18:07, Borislav Petkov wrote: > > On Wed, Aug 12, 2015 at 05:01:14PM +0800, yalin wang wrote: >> hi x86 maintainers, >> >> i have a question about copy_from{to}_user() function, >> i find on other platforms like arm/ arm64 /hexagon, >&g

Re: [x86] copy_from{to}_user question

2015-08-13 Thread yalin wang
On Aug 12, 2015, at 18:07, Borislav Petkov b...@suse.de wrote: On Wed, Aug 12, 2015 at 05:01:14PM +0800, yalin wang wrote: hi x86 maintainers, i have a question about copy_from{to}_user() function, i find on other platforms like arm/ arm64 /hexagon, all copy_from{to}_user function only

[PATCH] net/fddi: remove HWM_REVERSE() macro

2015-08-12 Thread yalin wang
HWM_REVERSE() macro is unused, remove it. Signed-off-by: yalin wang --- drivers/net/fddi/skfp/h/hwmtm.h | 9 - 1 file changed, 9 deletions(-) diff --git a/drivers/net/fddi/skfp/h/hwmtm.h b/drivers/net/fddi/skfp/h/hwmtm.h index 5924d42..4ca2341 100644 --- a/drivers/net/fddi/skfp/h

[PATCH] proc/kcore: remove unneeded white spaces

2015-08-12 Thread yalin wang
Signed-off-by: yalin wang --- fs/proc/kcore.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/fs/proc/kcore.c b/fs/proc/kcore.c index 92e6726..3782dd1 100644 --- a/fs/proc/kcore.c +++ b/fs/proc/kcore.c @@ -86,8 +86,8 @@ static size_t get_kcore_size(int *nphdr, size_t

[x86] copy_from{to}_user question

2015-08-12 Thread yalin wang
hi x86 maintainers, i have a question about copy_from{to}_user() function, i find on other platforms like arm/ arm64 /hexagon, all copy_from{to}_user function only check source address for copy_from and only check to address for copy_to user function, never check both source and dest together,

[PATCH] net/fddi: remove HWM_REVERSE() macro

2015-08-12 Thread yalin wang
HWM_REVERSE() macro is unused, remove it. Signed-off-by: yalin wang yalin.wang2...@gmail.com --- drivers/net/fddi/skfp/h/hwmtm.h | 9 - 1 file changed, 9 deletions(-) diff --git a/drivers/net/fddi/skfp/h/hwmtm.h b/drivers/net/fddi/skfp/h/hwmtm.h index 5924d42..4ca2341 100644

[x86] copy_from{to}_user question

2015-08-12 Thread yalin wang
hi x86 maintainers, i have a question about copy_from{to}_user() function, i find on other platforms like arm/ arm64 /hexagon, all copy_from{to}_user function only check source address for copy_from and only check to address for copy_to user function, never check both source and dest together,

[PATCH] proc/kcore: remove unneeded white spaces

2015-08-12 Thread yalin wang
Signed-off-by: yalin wang yalin.wang2...@gmail.com --- fs/proc/kcore.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/fs/proc/kcore.c b/fs/proc/kcore.c index 92e6726..3782dd1 100644 --- a/fs/proc/kcore.c +++ b/fs/proc/kcore.c @@ -86,8 +86,8 @@ static size_t

Re: [PATCH v2 Resend] net/fddi: remove HWM_REVERSE() macro

2015-08-11 Thread yalin wang
HWM_REVERSE() macro is unused, remove it. Signed-off-by: yalin wang --- drivers/net/fddi/skfp/h/hwmtm.h | 9 - 1 file changed, 9 deletions(-) diff --git a/drivers/net/fddi/skfp/h/hwmtm.h b/drivers/net/fddi/skfp/h/hwmtm.h index 5924d42..4ca2341 100644 --- a/drivers/net/fddi/skfp/h

Re: [PATCH v2 Resend] net/fddi: remove HWM_REVERSE() macro

2015-08-11 Thread yalin wang
HWM_REVERSE() macro is unused, remove it. Signed-off-by: yalin wang yalin.wang2...@gmail.com --- drivers/net/fddi/skfp/h/hwmtm.h | 9 - 1 file changed, 9 deletions(-) diff --git a/drivers/net/fddi/skfp/h/hwmtm.h b/drivers/net/fddi/skfp/h/hwmtm.h index 5924d42..4ca2341 100644

Re: [PATCH v2 Resend] net/fddi: remove HWM_REVERSE() macro

2015-08-10 Thread yalin wang
> On Aug 11, 2015, at 13:37, David Miller wrote: > > From: yalin wang > Date: Tue, 11 Aug 2015 13:11:22 +0800 > >> HWM_REVERSE() macro is unused, remove it. >> >> Signed-off-by: yalin wang > > You did not do as I asked you to, this patch is st

[PATCH] joystick/zhenhua: remove zhenhua_bitreverse()

2015-08-10 Thread yalin wang
This change remove zhenhua_bitreverse() function, use generic bitrev8() function instead. Signed-off-by: yalin wang --- drivers/input/joystick/zhenhua.c | 13 ++--- 1 file changed, 2 insertions(+), 11 deletions(-) diff --git a/drivers/input/joystick/zhenhua.c b/drivers/input/joystick

[PATCH] zlib_deflate/deftree: Remove bi_reverse()

2015-08-10 Thread yalin wang
This change remove bi_reverse() and use generic bitrev32() instead, have better performance on some platforms. Signed-off-by: yalin wang --- lib/zlib_deflate/deftree.c | 6 +++--- lib/zlib_deflate/defutil.h | 16 2 files changed, 3 insertions(+), 19 deletions(-) diff --git

[PATCH v2 Resend] net/fddi: remove HWM_REVERSE() macro

2015-08-10 Thread yalin wang
HWM_REVERSE() macro is unused, remove it. Signed-off-by: yalin wang --- drivers/net/fddi/skfp/h/hwmtm.h | 10 -- 1 file changed, 10 deletions(-) diff --git a/drivers/net/fddi/skfp/h/hwmtm.h b/drivers/net/fddi/skfp/h/hwmtm.h index 5924d42..4ca2341 100644 --- a/drivers/net/fddi/skfp/h

Re: [PATCH v2] net/fddi: remove HWM_REVERSE() macro

2015-08-10 Thread yalin wang
> On Aug 11, 2015, at 12:24, David Miller wrote: > > From: yalin wang > Date: Tue, 11 Aug 2015 09:57:21 +0800 > >> HWM_REVERSE() macro is unused, remove it. >> >> Signed-off-by: yalin wang > > Your email client has corrupted this patch. > >

[RFC V2] bmac:change to use bitrev8() generic function

2015-08-10 Thread yalin wang
This change to use generic bitrev8() for bmac driver. Signed-off-by: yalin wang --- drivers/net/ethernet/apple/bmac.c | 17 +++-- 1 file changed, 3 insertions(+), 14 deletions(-) diff --git a/drivers/net/ethernet/apple/bmac.c b/drivers/net/ethernet/apple/bmac.c index a65d7a6

Re: [RFC] bmac:change to use bitrev8() generic function

2015-08-10 Thread yalin wang
> On Aug 10, 2015, at 20:02, David Laight wrote: > >> From: Tobias Klauser >> Sent: 10 August 2015 12:49 >> On 2015-08-10 at 11:53:41 +0200, yalin wang wrote: >>> This change to use generic bitrev8() for bmac driver. > ... >>> @@ -871,7 +860,7 @

[PATCH v2] net/fddi: remove HWM_REVERSE() macro

2015-08-10 Thread yalin wang
HWM_REVERSE() macro is unused, remove it. Signed-off-by: yalin wang --- drivers/net/fddi/skfp/h/hwmtm.h | 10 -- 1 file changed, 10 deletions(-) diff --git a/drivers/net/fddi/skfp/h/hwmtm.h b/drivers/net/fddi/skfp/h/hwmtm.h index 5924d42..4ca2341 100644 --- a/drivers/net/fddi/skfp/h

Re: [PATCH] net/fddi:change HWM_REVERSE() macro

2015-08-10 Thread yalin wang
> On Aug 11, 2015, at 00:36, Joe Perches wrote: > > On Tue, 2015-08-11 at 00:14 +0800, yalin wang wrote: >> HWM_REVERSE > > Is unused and it would be better if removed. > ok, i will send V2 patch . -- To unsubscribe from this list: send the line "unsubscr

Re: [PATCH] isdn:remove reverse_bits(), use revbit8()

2015-08-10 Thread yalin wang
> On Aug 11, 2015, at 05:30, David Miller wrote: > > From: yalin wang > Date: Mon, 10 Aug 2015 17:15:57 +0800 > >> This change isdn driver, remove reverse_bits() function, >> use the generic revbit8() function instead. >> >> Signed-off-by: yalin wang &

[PATCH] lib/zlib_inflate:change REVERSE() macro

2015-08-10 Thread yalin wang
This patch change REVERSE() macro to use swab32() function, so that can have more better performance on some platforms. Signed-off-by: yalin wang --- lib/zlib_inflate/inflate.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/lib/zlib_inflate/inflate.c b/lib/zlib_inflate

[PATCH] net/fddi:change HWM_REVERSE() macro

2015-08-10 Thread yalin wang
change HWM_REVERSE() macro to generic le32_to_cpu() Signed-off-by: yalin wang --- drivers/net/fddi/skfp/h/hwmtm.h | 11 ++- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/drivers/net/fddi/skfp/h/hwmtm.h b/drivers/net/fddi/skfp/h/hwmtm.h index 5924d42..72701ef 100644

[PATCH] hwmon:change sht15_reverse()

2015-08-10 Thread yalin wang
This change sht15_reverse() to be generic bitrev8(). Signed-off-by: yalin wang --- drivers/hwmon/sht15.c | 20 1 file changed, 4 insertions(+), 16 deletions(-) diff --git a/drivers/hwmon/sht15.c b/drivers/hwmon/sht15.c index 497a7f8..a2fdbb7 100644 --- a/drivers/hwmon

[PATCH] fbdev/nvidia:change reverse_order() macro

2015-08-10 Thread yalin wang
This change reverse_order() to swab32(bitrev32()), so that it can have better performance on some platforms. Signed-off-by: yalin wang --- drivers/video/fbdev/nvidia/nv_accel.c | 4 ++-- drivers/video/fbdev/nvidia/nv_local.h | 13 - 2 files changed, 6 insertions(+), 11 deletions

[RFC] fbdev/riva:change to use generice function to implement reverse_order()

2015-08-10 Thread yalin wang
This change to use swab32(bitrev32()) to implement reverse_order() function, have better performance on some platforms. Signed-off-by: yalin wang --- drivers/video/fbdev/riva/fbdev.c | 19 ++- 1 file changed, 6 insertions(+), 13 deletions(-) diff --git a/drivers/video/fbdev

[RFC] bmac:change to use bitrev8() generic function

2015-08-10 Thread yalin wang
This change to use generic bitrev8() for bmac driver. Signed-off-by: yalin wang --- drivers/net/ethernet/apple/bmac.c | 17 +++-- 1 file changed, 3 insertions(+), 14 deletions(-) diff --git a/drivers/net/ethernet/apple/bmac.c b/drivers/net/ethernet/apple/bmac.c index a65d7a6

[PATCH] isdn:remove reverse_bits(), use revbit8()

2015-08-10 Thread yalin wang
This change isdn driver, remove reverse_bits() function, use the generic revbit8() function instead. Signed-off-by: yalin wang --- drivers/isdn/mISDN/dsp_audio.c | 22 +- 1 file changed, 5 insertions(+), 17 deletions(-) diff --git a/drivers/isdn/mISDN/dsp_audio.c b/drivers

Re: [PATCH v2] net/fddi: remove HWM_REVERSE() macro

2015-08-10 Thread yalin wang
On Aug 11, 2015, at 12:24, David Miller da...@davemloft.net wrote: From: yalin wang yalin.wang2...@gmail.com Date: Tue, 11 Aug 2015 09:57:21 +0800 HWM_REVERSE() macro is unused, remove it. Signed-off-by: yalin wang yalin.wang2...@gmail.com Your email client has corrupted this patch

[PATCH v2 Resend] net/fddi: remove HWM_REVERSE() macro

2015-08-10 Thread yalin wang
HWM_REVERSE() macro is unused, remove it. Signed-off-by: yalin wang yalin.wang2...@gmail.com --- drivers/net/fddi/skfp/h/hwmtm.h | 10 -- 1 file changed, 10 deletions(-) diff --git a/drivers/net/fddi/skfp/h/hwmtm.h b/drivers/net/fddi/skfp/h/hwmtm.h index 5924d42..4ca2341 100644

[PATCH] zlib_deflate/deftree: Remove bi_reverse()

2015-08-10 Thread yalin wang
This change remove bi_reverse() and use generic bitrev32() instead, have better performance on some platforms. Signed-off-by: yalin wang yalin.wang2...@gmail.com --- lib/zlib_deflate/deftree.c | 6 +++--- lib/zlib_deflate/defutil.h | 16 2 files changed, 3 insertions(+), 19

[PATCH] joystick/zhenhua: remove zhenhua_bitreverse()

2015-08-10 Thread yalin wang
This change remove zhenhua_bitreverse() function, use generic bitrev8() function instead. Signed-off-by: yalin wang yalin.wang2...@gmail.com --- drivers/input/joystick/zhenhua.c | 13 ++--- 1 file changed, 2 insertions(+), 11 deletions(-) diff --git a/drivers/input/joystick/zhenhua.c b

Re: [PATCH v2 Resend] net/fddi: remove HWM_REVERSE() macro

2015-08-10 Thread yalin wang
On Aug 11, 2015, at 13:37, David Miller da...@davemloft.net wrote: From: yalin wang yalin.wang2...@gmail.com Date: Tue, 11 Aug 2015 13:11:22 +0800 HWM_REVERSE() macro is unused, remove it. Signed-off-by: yalin wang yalin.wang2...@gmail.com You did not do as I asked you

[PATCH] net/fddi:change HWM_REVERSE() macro

2015-08-10 Thread yalin wang
change HWM_REVERSE() macro to generic le32_to_cpu() Signed-off-by: yalin wang yalin.wang2...@gmail.com --- drivers/net/fddi/skfp/h/hwmtm.h | 11 ++- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/drivers/net/fddi/skfp/h/hwmtm.h b/drivers/net/fddi/skfp/h/hwmtm.h index

[PATCH] lib/zlib_inflate:change REVERSE() macro

2015-08-10 Thread yalin wang
This patch change REVERSE() macro to use swab32() function, so that can have more better performance on some platforms. Signed-off-by: yalin wang yalin.wang2...@gmail.com --- lib/zlib_inflate/inflate.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/lib/zlib_inflate

[PATCH] hwmon:change sht15_reverse()

2015-08-10 Thread yalin wang
This change sht15_reverse() to be generic bitrev8(). Signed-off-by: yalin wang yalin.wang2...@gmail.com --- drivers/hwmon/sht15.c | 20 1 file changed, 4 insertions(+), 16 deletions(-) diff --git a/drivers/hwmon/sht15.c b/drivers/hwmon/sht15.c index 497a7f8..a2fdbb7 100644

Re: [RFC] bmac:change to use bitrev8() generic function

2015-08-10 Thread yalin wang
On Aug 10, 2015, at 20:02, David Laight david.lai...@aculab.com wrote: From: Tobias Klauser Sent: 10 August 2015 12:49 On 2015-08-10 at 11:53:41 +0200, yalin wang yalin.wang2...@gmail.com wrote: This change to use generic bitrev8() for bmac driver. ... @@ -871,7 +860,7 @@ bmac_addhash

[RFC V2] bmac:change to use bitrev8() generic function

2015-08-10 Thread yalin wang
This change to use generic bitrev8() for bmac driver. Signed-off-by: yalin wang yalin.wang2...@gmail.com --- drivers/net/ethernet/apple/bmac.c | 17 +++-- 1 file changed, 3 insertions(+), 14 deletions(-) diff --git a/drivers/net/ethernet/apple/bmac.c b/drivers/net/ethernet/apple

Re: [PATCH] isdn:remove reverse_bits(), use revbit8()

2015-08-10 Thread yalin wang
On Aug 11, 2015, at 05:30, David Miller da...@davemloft.net wrote: From: yalin wang yalin.wang2...@gmail.com Date: Mon, 10 Aug 2015 17:15:57 +0800 This change isdn driver, remove reverse_bits() function, use the generic revbit8() function instead. Signed-off-by: yalin wang yalin.wang2

Re: [PATCH] net/fddi:change HWM_REVERSE() macro

2015-08-10 Thread yalin wang
On Aug 11, 2015, at 00:36, Joe Perches j...@perches.com wrote: On Tue, 2015-08-11 at 00:14 +0800, yalin wang wrote: HWM_REVERSE Is unused and it would be better if removed. ok, i will send V2 patch . -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body

[PATCH v2] net/fddi: remove HWM_REVERSE() macro

2015-08-10 Thread yalin wang
HWM_REVERSE() macro is unused, remove it. Signed-off-by: yalin wang yalin.wang2...@gmail.com --- drivers/net/fddi/skfp/h/hwmtm.h | 10 -- 1 file changed, 10 deletions(-) diff --git a/drivers/net/fddi/skfp/h/hwmtm.h b/drivers/net/fddi/skfp/h/hwmtm.h index 5924d42..4ca2341 100644

[PATCH] isdn:remove reverse_bits(), use revbit8()

2015-08-10 Thread yalin wang
This change isdn driver, remove reverse_bits() function, use the generic revbit8() function instead. Signed-off-by: yalin wang yalin.wang2...@gmail.com --- drivers/isdn/mISDN/dsp_audio.c | 22 +- 1 file changed, 5 insertions(+), 17 deletions(-) diff --git a/drivers/isdn

[RFC] bmac:change to use bitrev8() generic function

2015-08-10 Thread yalin wang
This change to use generic bitrev8() for bmac driver. Signed-off-by: yalin wang yalin.wang2...@gmail.com --- drivers/net/ethernet/apple/bmac.c | 17 +++-- 1 file changed, 3 insertions(+), 14 deletions(-) diff --git a/drivers/net/ethernet/apple/bmac.c b/drivers/net/ethernet/apple

[RFC] fbdev/riva:change to use generice function to implement reverse_order()

2015-08-10 Thread yalin wang
This change to use swab32(bitrev32()) to implement reverse_order() function, have better performance on some platforms. Signed-off-by: yalin wang yalin.wang2...@gmail.com --- drivers/video/fbdev/riva/fbdev.c | 19 ++- 1 file changed, 6 insertions(+), 13 deletions(-) diff --git

[PATCH] fbdev/nvidia:change reverse_order() macro

2015-08-10 Thread yalin wang
This change reverse_order() to swab32(bitrev32()), so that it can have better performance on some platforms. Signed-off-by: yalin wang yalin.wang2...@gmail.com --- drivers/video/fbdev/nvidia/nv_accel.c | 4 ++-- drivers/video/fbdev/nvidia/nv_local.h | 13 - 2 files changed, 6

Re: [RFC] kcore:change kcore_read to make sure the kernel read is safe

2015-08-04 Thread yalin wang
> On Aug 5, 2015, at 05:18, Dave Hansen wrote: > > On 08/03/2015 08:37 PM, yalin wang wrote: >> This change kcore_read() to use __copy_from_user_inatomic() to >> copy data from kernel address, because kern_addr_valid() just make sure >> page table is valid dur

Re: [RFC] kcore:change kcore_read to make sure the kernel read is safe

2015-08-04 Thread yalin wang
> On Aug 5, 2015, at 06:38, Andrew Morton wrote: > > On Tue, 4 Aug 2015 11:37:57 +0800 yalin wang wrote: > >> This change kcore_read() to use __copy_from_user_inatomic() to >> copy data from kernel address, because kern_addr_valid() just make sure >> page table

Re: [PATCH -v2 6/8] jump_label: Add a new static_key interface

2015-08-04 Thread yalin wang
> On Jul 28, 2015, at 21:21, Peter Zijlstra wrote: > > There are various problems and short-comings with the current > static_key interface: > > - static_key_{true,false}() read like a branch depending on the key > value, instead of the actual likely/unlikely branch depending on > init

Re: [PATCH -v2 6/8] jump_label: Add a new static_key interface

2015-08-04 Thread yalin wang
On Jul 28, 2015, at 21:21, Peter Zijlstra pet...@infradead.org wrote: There are various problems and short-comings with the current static_key interface: - static_key_{true,false}() read like a branch depending on the key value, instead of the actual likely/unlikely branch depending on

<    1   2   3   4   >