[RFC v3] zswap: Add CONFIG_ZSWAP_IO_SWITCH to handle swap IO issue

2019-09-22 Thread Hui Zhu
/2019/9/20/90 [3] https://lkml.org/lkml/2019/9/20/1076 Signed-off-by: Hui Zhu --- include/linux/swap.h | 3 +++ mm/Kconfig | 18 mm/page_io.c | 16 +++ mm/zswap.c | 58 4 files changed

[RFC v2] zswap: Add CONFIG_ZSWAP_IO_SWITCH to handle swap IO issue

2019-09-20 Thread Hui Zhu
will be stored to zswap only when the IO in flight number of swap device is bigger than zswap_read_in_flight_limit or zswap_write_in_flight_limit when zswap is enabled. Then the zswap just work when the IO in flight number of swap device is low. Signed-off-by: Hui Zhu --- include/linux/swap.h

[PATCH v5] gpio/mpc8xxx: change irq handler from chained to normal

2019-09-16 Thread Hui Song
From: Song Hui More than one gpio controllers can share one interrupt, change the driver to request shared irq. Signed-off-by: Laurentiu Tudor Signed-off-by: Alex Marginean Signed-off-by: Song Hui --- Changes in v5: - add traverse every bit function. Changes in v4: - convert

[PATCH for vm-scalability] usemem: Add new option -Z|--read-again

2019-09-13 Thread Hui Zhu
usemem will read memory again after access the memory with this option. It can help test the speed that load page from swap to memory. Signed-off-by: Hui Zhu --- usemem.c | 46 -- 1 file changed, 40 insertions(+), 6 deletions(-) diff --git a/usemem.c

[PATCH] zswap: Add CONFIG_ZSWAP_IO_SWITCH

2019-09-11 Thread Hui Zhu
55716864 bytes / 14457505 usecs = 571159 KB/s 365961 usecs to free memory Signed-off-by: Hui Zhu --- include/linux/swap.h | 3 +++ mm/Kconfig | 11 +++ mm/page_io.c | 16 +++ mm/zswap.c | 55 4 fi

[PATCH] gpio/mpc8xxx: change irq handler from chained to normal

2019-09-11 Thread Hui Song
From: Song Hui More than one gpio controllers can share one interrupt, change the driver to request shared irq. Signed-off-by: Laurentiu Tudor Signed-off-by: Alex Marginean Signed-off-by: Song Hui --- Changes in v4: - convert 'pr_err' to 'dev_err'. Changes in v3: - update

[PATCH] gpio/mpc8xxx: change irq handler from chained to normal

2019-09-11 Thread Hui Song
From: Song Hui More than one gpio controllers can share one interrupt, change the driver to request shared irq. Signed-off-by: Laurentiu Tudor Signed-off-by: Alex Marginean Signed-off-by: Song Hui --- Changes in v4: - convert 'pr_err' to 'dev_err'. Changes in v3: - update

[PATCH v3] gpio/mpc8xxx: change irq handler from chained to normal

2019-09-08 Thread Hui Song
From: Song Hui More than one gpio controllers can share one interrupt, change the driver to request shared irq. Signed-off-by: Laurentiu Tudor Signed-off-by: Alex Marginean Signed-off-by: Song Hui --- Changes in v3: - update the patch description. Changes in v2: - delete

[PATCH] gpio/mpc8xxx: change irq handler from chained to normal

2019-09-06 Thread Hui Song
From: Song Hui more one gpio controller use share one interrupt, make request interrupt to be shared. Signed-off-by: Laurentiu Tudor Signed-off-by: Alex Marginean Signed-off-by: Song Hui --- drivers/gpio/gpio-mpc8xxx.c | 20 +--- 1 file changed, 13 insertions(+), 7 deletions

[PATCH] gpio/mpc8xxx: change irq handler from chained to normal

2019-09-06 Thread Hui Song
From: Song Hui more one gpio controller use share one interrupt, make request interrupt to be shared. Signed-off-by: Laurentiu Tudor Signed-off-by: Alex Marginean Signed-off-by: Song Hui --- drivers/gpio/gpio-mpc8xxx.c | 21 ++--- 1 file changed, 14 insertions(+), 7

Re: WARNING in hso_free_net_device

2019-09-05 Thread Hui Peng
On 9/5/2019 7:24 AM, Andrey Konovalov wrote: > On Thu, Sep 5, 2019 at 4:20 AM Hui Peng wrote: >> >> Can you guys have a look at the attached patch? > > Let's try it: > > #syz test: https://github.com/google/kasan.git eea39f24 > > FYI: there are two mor

Re: WARNING in hso_free_net_device

2019-09-04 Thread Hui Peng
Can you guys have  a look at the attached patch? On 9/4/19 6:41 PM, Stephen Hemminger wrote: > On Wed, 4 Sep 2019 16:27:50 -0400 > Hui Peng wrote: > >> Hi, all: >> >> I looked at the bug a little. >> >> The issue is that in the error handling c

Re: WARNING in hso_free_net_device

2019-09-04 Thread Hui Peng
Hi, all: I looked at the bug a little. The issue is that in the error handling code, hso_free_net_device unregisters the net_device (hso_net->net)  by calling unregister_netdev. In the error handling code path, hso_net->net has not been registered yet. I think there are two ways to solve the

Re: [PATCH 2/2] Fix a stack buffer overflow bug in check_input_term

2019-09-01 Thread Hui Peng
On 9/1/19 9:00 AM, Salvatore Bonaccorso wrote: > Hi Hui, > > On Fri, Aug 30, 2019 at 05:47:29PM -0400, Hui Peng wrote: >> `check_input_term` recursively calls itself with input from >> device side (e.g., uac_input_terminal_descriptor.bCSourceID) >> as argument

Re: [PATCH 1/2] Fix an OOB bug in parse_audio_mixer_unit

2019-09-01 Thread Hui Peng
On 9/1/19 8:58 AM, Salvatore Bonaccorso wrote: > On Fri, Aug 30, 2019 at 05:46:49PM -0400, Hui Peng wrote: >> The `uac_mixer_unit_descriptor` shown as below is read from the >> device side. In `parse_audio_mixer_unit`, `baSourceID` field is >> accessed from index 0 to `bNrInP

Re: [PATCH] Fix a double free bug in rsi_91x_deinit

2019-08-31 Thread Hui Peng
On 8/31/19 2:18 PM, Guenter Roeck wrote: > On Mon, Aug 19, 2019 at 06:02:29PM -0400, Hui Peng wrote: >> `dev` (struct rsi_91x_usbdev *) field of adapter >> (struct rsi_91x_usbdev *) is allocated and initialized in >> `rsi_init_usb_interface`. If any error is detected in inf

[PATCH 1/2] Fix an OOB bug in parse_audio_mixer_unit

2019-08-30 Thread Hui Peng
[]; } ``` This patch fixes the bug by add a sanity check on the length of the descriptor. CVE: CVE-2018-15117 Reported-by: Hui Peng Reported-by: Mathias Payer Signed-off-by: Hui Peng --- sound/usb/mixer.c | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/usb/mixer.c b/sound/usb/mixer.c

[PATCH 2/2] Fix a stack buffer overflow bug in check_input_term

2019-08-30 Thread Hui Peng
stack overflow. This patch fixes the bug by adding a bitmap to `struct mixer_build` to keep track of the checked ids and stop the execution if some id has been checked (similar to how parse_audio_unit handles unitid argument). CVE: CVE-2018-15118 Reported-by: Hui Peng Reported-by: Mathias Payer

Re: [PATCH] Fix an OOB access bug in technisat_usb2_get_ir

2019-08-20 Thread Hui Peng
+0x816/0x14d0 kernel/workqueue.c:2153  worker_thread+0x9b/0xce0 kernel/workqueue.c:2296  kthread+0x33d/0x400 kernel/kthread.c:246  ret_from_fork+0x3a/0x50 arch/x86/entry/entry_64.S:352 On 8/20/19 2:19 PM, Hui Peng wrote: > In the while loop of technisat_usb2_get_ir, it scans through > a fix

[PATCH] Fix an OOB access bug in technisat_usb2_get_ir

2019-08-20 Thread Hui Peng
. Reported-by: Hui Peng Reported-by: Mathias Payer Signed-off-by: Hui Peng --- drivers/media/usb/dvb-usb/technisat-usb2.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/media/usb/dvb-usb/technisat-usb2.c b/drivers/media/usb/dvb-usb/technisat-usb2.c index c659e18b358b

[PATCH v4] arm64: dts: ls1088a: fix gpio node

2019-08-20 Thread Hui Song
From: Song Hui add ls1088a gpio specify compatible. Signed-off-by: Song Hui --- Changes in v4: - update the patch description. Changes in v3: - delete the attribute of little-endian. Changes in v2: - update the subject. diff --git a/arch/arm64/boot/dts/freescale/fsl

[PATCH] Fix a double free bug in rsi_91x_deinit

2019-08-19 Thread Hui Peng
path, the `dev` field is not (and thus needs to be) freed. This bug was found in v4.19, but is also present in the latest version of kernel. Reported-by: Hui Peng Reported-by: Mathias Payer Signed-off-by: Hui Peng --- drivers/net/wireless/rsi/rsi_91x_usb.c | 1 - 1 file changed, 1 deletion

[PATCH] Fix an OOB bug in uac_mixer_unit_bmControls

2019-08-19 Thread Hui Peng
-by: Hui Peng Reported-by: Mathias Payer Signed-off-by: Hui Peng --- sound/usb/mixer.c | 25 ++--- 1 file changed, 18 insertions(+), 7 deletions(-) diff --git a/sound/usb/mixer.c b/sound/usb/mixer.c index b5927c3d5bc0..00e6274a63c3 100644 --- a/sound/usb/mixer.c +++ b/sound/usb

[PATCH] Fix an OOB bug in uac_mixer_unit_bmControls

2019-08-16 Thread Hui Peng
-by: Hui Peng Reported-by: Mathias Payer Signed-off-by: Hui Peng --- sound/usb/mixer.c | 25 ++--- 1 file changed, 18 insertions(+), 7 deletions(-) diff --git a/sound/usb/mixer.c b/sound/usb/mixer.c index b5927c3d5bc0..00e6274a63c3 100644 --- a/sound/usb/mixer.c +++ b/sound/usb

[PATCH v3] arm64: dts: ls1088a: fix gpio node

2019-08-15 Thread Hui Song
From: Song Hui Update the nodes to include little-endian property to be consistent with the hardware and add ls1088a gpio specify compatible. Signed-off-by: Song Hui --- Changes in v3: - delete the attribute of little-endian. Changes in v2: - update the subject. arch

Re: [PATCH] Fix a stack buffer overflow bug check_input_term

2019-08-14 Thread Hui Peng
feedback. On 8/15/19 12:35 AM, Hui Peng wrote: > `check_input_term` recursively calls itself with input > from device side (e.g., uac_input_terminal_descriptor.bCSourceID) > as argument (id). In `check_input_term`, if `check_input_term` > is called with the same `id` argument a

[PATCH] Fix a stack buffer overflow bug check_input_term

2019-08-14 Thread Hui Peng
stack overflow. This patch fixes the bug by adding a bitmap to `struct mixer_build` to keep track of the checked ids by `check_input_term` and stop the execution if some id has been checked (similar to how parse_audio_unit handles unitid argument). Reported-by: Hui Peng Reported-by: Mathias Payer

[PATCH] Fix an OOB bug in parse_audio_mixer_unit

2019-08-13 Thread Hui Peng
[]; } ``` This patch fixes the bug by add a sanity check on the length of the descriptor. Signed-off-by: Hui Peng Reported-by: Hui Peng Reported-by: Mathias Payer --- sound/usb/mixer.c | 9 + 1 file changed, 9 insertions(+) diff --git a/sound/usb/mixer.c b/sound/usb/mixer.c index

[PATCH v2] arm64: dts: ls1088a: fix gpio node

2019-08-12 Thread Hui Song
From: Song Hui Update the nodes to include little-endian property to be consistent with the hardware and add ls1088a gpio specify compatible. Signed-off-by: Song Hui --- arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi | 12 1 file changed, 8 insertions(+), 4 deletions(-) diff

[PATCH v1 2/3] arm64: dts: fix gpio node

2019-08-08 Thread Hui Song
From: Song Hui Update the nodes to include little-endian property to be consistent with the hardware and add ls1088a gpio specify compatible. Signed-off-by: Song Hui --- arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi | 12 1 file changed, 8 insertions(+), 4 deletions(-) diff

[PATCH v1 3/3] gpio: mpc8xxx: add ls1088a platform special function

2019-08-08 Thread Hui Song
From: Song Hui ls1028a and ls1088a platform share common special function. The gpio hardware what they use is the same version. Signed-off-by: Song Hui --- drivers/gpio/gpio-mpc8xxx.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpio/gpio-mpc8xxx.c b/drivers/gpio/gpio-mpc8xxx.c

[PATCH v1 1/3] gpio: mpc8xxx: add ls1088a platform gpio node DT binding description

2019-08-08 Thread Hui Song
From: Song Hui ls1088a and ls1028a platform share common gpio node. Signed-off-by: Song Hui --- Documentation/devicetree/bindings/gpio/gpio-mpc8xxx.txt | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Documentation/devicetree/bindings/gpio/gpio-mpc8xxx.txt b

[PATCH v3] gpio: mpc8xxx: Add new platforms GPIO DT node description

2019-08-06 Thread Hui Song
From: Song Hui Update the NXP GPIO node dt-binding file for QorIQ and Layerscape platforms, and add one more example with ls1028a GPIO node. Signed-off-by: Song Hui --- Documentation/devicetree/bindings/gpio/gpio-mpc8xxx.txt | 16 +++- 1 file changed, 15 insertions(+), 1 deletion

[PATCH v2] gpio: mpc8xxx: Add new platforms GPIO DT node description

2019-08-05 Thread Hui Song
From: Song Hui Update the NXP GPIO node dt-binding file for QorIQ and Layerscape platforms, and add one more example with ls1028a GPIO node. Signed-off-by: Song Hui --- Documentation/devicetree/bindings/gpio/gpio-mpc8xxx.txt | 14 ++ 1 file changed, 14 insertions(+) diff --git

[PATCH v1] gpio: mpc8xxx: Add new platforms GPIO DT node description

2019-08-05 Thread Hui Song
From: Song Hui Update the NXP GPIO node dt-binding file for QorIQ and Layerscape platforms, and add one more example with ls1028a GPIO node. Signed-off-by: Song Hui --- Documentation/devicetree/bindings/gpio/gpio-mpc8xxx.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH v2] arm64: dts: ls1028a: fix gpio nodes

2019-08-05 Thread Hui Song
From: Song Hui Update the nodes to include little-endian property to be consistent with the hardware. Signed-off-by: Song Hui --- arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/arch/arm64/boot/dts/freescale/fsl

[PATCH v1] gpio: mpc8xxx: Add new platforms GPIO DT node description

2019-08-05 Thread Hui Song
From: Song Hui Update the NXP GPIO node dt-binding file for QorIQ and Layerscape platforms, and add one more example with ls1028a GPIO node. Signed-off-by: Song Hui --- Documentation/devicetree/bindings/gpio/gpio-mpc8xxx.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH] arm64: dts: ls1028a: fix gpio nodes

2019-08-05 Thread Hui Song
From: Song Hui Update the nodes to include little-endian property to be consistent with the hardware. Signed-off-by: Song Hui --- arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/arch/arm64/boot/dts/freescale/fsl

[PATCH 1/2] Fix a NULL-ptr-deref bug in ath6kl_usb_alloc_urb_from_pipe

2019-08-03 Thread Hui Peng
ot report complete addresses, it may trigger NULL-ptr-deref `ath6kl_usb_alloc_urb_from_pipe` and `ath6kl_usb_free_urb_to_pipe`. This patch fixes the bug by preventing potential NULL-ptr-deref. Signed-off-by: Hui Peng Reported-by: Hui Peng Reported-by: Mathias Payer --- drivers/net/wireless/at

[PATCH 2/2] gpio: mpc8xxx: Add ls1028a device specify function.

2019-07-18 Thread Hui Song
From: Song Hui There is a device specify register(named GPIO_IBE) on ls1028a need to enable in initial stage. Signed-off-by: Song Hui --- drivers/gpio/gpio-mpc8xxx.c | 31 +++ 1 file changed, 31 insertions(+) diff --git a/drivers/gpio/gpio-mpc8xxx.c b/drivers/gpio

[PATCH 1/2] arm64: dts: ls1028a: Fix GPIO work fail.

2019-07-18 Thread Hui Song
From: Song Hui Add ls1028a device specify compatible. Make gpio as little-endian deal. Signed-off-by: Song Hui --- arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi b/arch

Re: audio lost from speaker after reboot from windows on the device ALC295

2019-07-04 Thread Hui Wang
Maybe your machine has an external amplifier which needs to be set by software before it can work. On 2019/7/4 下午8:02, He, Bo wrote: Hi, patch_realtek.c maintainer: I see one issue that reboot from windows and boot to ubuntu, the audio lost from speaker, I suspect there are some bugs

Re: 答复: 答复: 答复: 答复: [PATCH] input: alps-fix the issue alps cs19 trackstick do not work.

2019-06-18 Thread Hui Wang
eed for CS19 device. So if trackpoint.c driver is working fine with this configuration, it is just needed to instruct alps.c to not take this device. Hello Pali, When you have time, could you take a look at the patch of v4? It is implemented according to our discussion. Thanks, Hui. Best Regards

Re: 答复: 答复: 答复: [PATCH] input: alps-fix the issue alps cs19 trackstick do not work.

2019-06-10 Thread Hui Wang
On 2019/6/11 上午11:23, Hui Wang wrote: On 2019/6/11 上午11:05, Xiaoxiao Liu wrote: Hi Pali, I discussed with our FW team about this problem. We think the V8 method means a touchpad feature  and does not fit the CS19 trackpoint device. CS19 TrackPoint needn't  use any Absolute (Raw) mode

Re: 答复: 答复: 答复: [PATCH] input: alps-fix the issue alps cs19 trackstick do not work.

2019-06-10 Thread Hui Wang
mode for CS19 trackpoint. Maybe here is some mis-understanding,  the mouse mode here doesn't mean we use psmouse-base.c for cs19 (bare ps/2 mouse), we plan to use trackpoint.c to drive this HW, so this trackpoint has all features a trackpoint should have. Regards, Hui. Best Regards Shona

[PATCH V3 2/2] zswap: Use movable memory if zpool support allocate movable memory

2019-06-05 Thread Hui Zhu
e page blocks is decreased when the kernel has this commit. Signed-off-by: Hui Zhu --- mm/zswap.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/mm/zswap.c b/mm/zswap.c index a4e4d36ec085..c6bf92bf5890 100644 --- a/mm/zswap.c +++ b/mm/zswap.c @@ -1006,6 +1006,7 @@ static int

[PATCH V3 1/2] zpool: Add malloc_support_movable to zpool_driver

2019-06-05 Thread Hui Zhu
zpool_malloc_support_movable check malloc_support_movable to make sure if a zpool support allocate movable memory. Signed-off-by: Hui Zhu --- include/linux/zpool.h | 3 +++ mm/zpool.c| 16 mm/zsmalloc.c | 19 ++- 3 files changed, 29 insertions(+), 9 deletions

Re: [PATCH V2 2/2] zswap: Add module parameter malloc_movable_if_support

2019-06-04 Thread Hui Zhu
Shakeel Butt 于2019年6月5日周三 上午1:12写道: > > On Sun, Jun 2, 2019 at 2:47 AM Hui Zhu wrote: > > > > This is the second version that was updated according to the comments > > from Sergey Senozhatsky in https://lkml.org/lkml/2019/5/29/73 > > > > zswap compresses swa

[PATCH V2 2/2] zswap: Add module parameter malloc_movable_if_support

2019-06-02 Thread Hui Zhu
0 00 You can see that the number of unmovable page blocks is decreased when malloc_movable_if_support is enabled. Signed-off-by: Hui Zhu --- mm/zswap.c | 16 +--- 1 file changed, 13 insertions(+), 3 deletions(-) diff --git a/mm/zswap.c b/mm/zswap.c index

[PATCH V2 1/2] zpool: Add malloc_support_movable to zpool_driver

2019-06-02 Thread Hui Zhu
zpool_malloc_support_movable check malloc_support_movable to make sure if a zpool support allocate movable memory. Signed-off-by: Hui Zhu --- include/linux/zpool.h | 3 +++ mm/zpool.c| 16 mm/zsmalloc.c | 19 ++- 3 files changed, 29 insertions(+), 9 deletions

[UPSTREAM KERNEL] mm/zsmalloc.c: Add module parameter malloc_force_movable

2019-05-28 Thread Hui Zhu
can see that the number of unmovable page blocks is decreased when malloc_force_movable is enabled. Signed-off-by: Hui Zhu --- mm/zsmalloc.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/mm/zsmalloc.c b/mm/zsmalloc.c index 0787d33b80d8..7d44c7ccd882 100644 --- a/mm/zsmall

Re: 答复: 答复: 答复: [PATCH] input: alps-fix the issue the special alps trackpoint do not work.

2019-05-24 Thread Hui Wang
On 2019/5/24 下午6:58, Peter Hutterer wrote: On Fri, May 24, 2019 at 06:43:58PM +0800, Hui Wang wrote: On 2019/5/24 下午5:37, Peter Hutterer wrote: On Fri, May 24, 2019 at 03:37:57PM +0800, Hui Wang wrote: OK, that is sth we need to do.  But anyway it is a bit risky to backport that much code

Re: 答复: 答复: 答复: [PATCH] input: alps-fix the issue the special alps trackpoint do not work.

2019-05-24 Thread Hui Wang
On 2019/5/24 下午3:26, Pali Rohár wrote: On Friday 24 May 2019 13:50:53 Hui Wang wrote: On 2019/5/24 下午1:36, Peter Hutterer wrote: On Fri, May 24, 2019 at 01:25:52PM +0800, Hui Wang wrote: On 2019/5/23 下午2:01, Peter Hutterer wrote: On Wed, May 22, 2019 at 09:40:30AM +0200, Pali Rohár wrote

Re: 答复: 答复: 答复: [PATCH] input: alps-fix the issue the special alps trackpoint do not work.

2019-05-23 Thread Hui Wang
On 2019/5/24 下午1:36, Peter Hutterer wrote: On Fri, May 24, 2019 at 01:25:52PM +0800, Hui Wang wrote: On 2019/5/23 下午2:01, Peter Hutterer wrote: On Wed, May 22, 2019 at 09:40:30AM +0200, Pali Rohár wrote: On Wednesday 22 May 2019 07:30:43 Xiaoxiao Liu wrote: Hi Pali, Ok, and cannot you set

Re: 答复: 答复: 答复: [PATCH] input: alps-fix the issue the special alps trackpoint do not work.

2019-05-23 Thread Hui Wang
we could set POINTINGSTICK_CONST_ACCEL=0.25 If the libinput version is higher than 1.12.0, we could set AttrTrackpointMultiplier=0.25 But if we use libinput-1.10.0,  how could we adjust the speed? Many thanks in advance. Hui. There are also different speeds depending on which xorg dri

Re: 答复: [PATCH] input: alps-fix the issue the special alps trackpoint do not work.

2019-05-20 Thread Hui Wang
Tested-by: Hui Wang On 2019/5/21 上午9:07, Xiaoxiao Liu wrote: Add Pali Rohár. -邮件原件- 发件人: XiaoXiao Liu 发送时间: Monday, May 20, 2019 7:02 PM 收件人: dmitry.torok...@gmail.com 抄送: linux-in...@vger.kernel.org; linux-kernel@vger.kernel.org; hui.w...@canonical.com; 曹 曉建 Xiaojian Cao ; zhang

[tip:x86/cpu] x86/topology: Use total_cpus for max logical packages calculation

2018-12-18 Thread tip-bot for Hui Wang
Commit-ID: aa02ef099cff042c2a9109782ec2bf1bffc955d4 Gitweb: https://git.kernel.org/tip/aa02ef099cff042c2a9109782ec2bf1bffc955d4 Author: Hui Wang AuthorDate: Wed, 7 Nov 2018 10:36:43 +0800 Committer: Thomas Gleixner CommitDate: Tue, 18 Dec 2018 13:38:37 +0100 x86/topology: Use

Re: [PATCH 4.19 096/139] ALSA: hda/realtek - fix the pop noise on headphone for lenovo laptops

2018-12-12 Thread Hui Wang
h word spoken. ("pop" Front "pop" left). You can repeat with the left headphone and see similar results. I even tried shifting the balance all the way to the left/right and get the same results. Thanks, Hui. On 2018/12/13 上午7:45, Thomas Zeitlhofer wrote: I have sent a new

Re: [alsa-devel] [PATCH] ALSA: hda/realtek - Add quirks for Dell Optiplex 5060, 7060 for audio out while using headphones

2018-12-11 Thread Hui Wang
On 2018/12/12 上午6:49, Girija Kumar Kasinadhuni wrote: There are two audio ports on these machines - headset and mic. Looks like the two audio ports are headset and lineout. There' s no sound when hp is inserted into the headset port, though the hp is detected. Without pulseaudio, this issue is

[PATCH] x86/topology: use total_cpus for max logical packages calculation

2018-11-06 Thread Hui Wang
, for example: taskset -c 26 echo c > /proc/sysrq-trigger Use total_cpus which will not be limited by nr_cpus command line to calculate the value of __max_logical_packages. Signed-off-by: Hui Wang --- arch/x86/kernel/smpboot.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/a

[PATCH] x86/topology: use total_cpus for max logical packages calculation

2018-11-06 Thread Hui Wang
, for example: taskset -c 26 echo c > /proc/sysrq-trigger Use total_cpus which will not be limited by nr_cpus command line to calculate the value of __max_logical_packages. Signed-off-by: Hui Wang --- arch/x86/kernel/smpboot.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/a

[tip:x86/urgent] x86/intel_rdt: Fix incorrect returned value when creating rdgroup sub-directory in resctrl file system

2018-02-22 Thread tip-bot for Wang Hui
Commit-ID: 36e74d355297dde6e69a39c838d24710e442babe Gitweb: https://git.kernel.org/tip/36e74d355297dde6e69a39c838d24710e442babe Author: Wang Hui <john.wang...@huawei.com> AuthorDate: Thu, 22 Feb 2018 19:26:03 -0800 Committer: Ingo Molnar <mi...@kernel.org> CommitDate: Fri, 23

[tip:x86/urgent] x86/intel_rdt: Fix incorrect returned value when creating rdgroup sub-directory in resctrl file system

2018-02-22 Thread tip-bot for Wang Hui
Commit-ID: 36e74d355297dde6e69a39c838d24710e442babe Gitweb: https://git.kernel.org/tip/36e74d355297dde6e69a39c838d24710e442babe Author: Wang Hui AuthorDate: Thu, 22 Feb 2018 19:26:03 -0800 Committer: Ingo Molnar CommitDate: Fri, 23 Feb 2018 08:03:21 +0100 x86/intel_rdt: Fix incorrect

Re: [PATCH] USB: serial: Clean up the return values of read_reg and write_reg

2017-11-28 Thread Gimcuan Hui
On 28 November 2017 at 08:47, Johan Hovold <jo...@kernel.org> wrote: > On Mon, Nov 27, 2017 at 03:36:51PM +, Gimcuan Hui wrote: >> write_reg returns 0 on success, we can make it more explicit by returning >> number 0 instead of result variable. >> >> read_reg

Re: [PATCH] USB: serial: Clean up the return values of read_reg and write_reg

2017-11-28 Thread Gimcuan Hui
On 28 November 2017 at 08:47, Johan Hovold wrote: > On Mon, Nov 27, 2017 at 03:36:51PM +, Gimcuan Hui wrote: >> write_reg returns 0 on success, we can make it more explicit by returning >> number 0 instead of result variable. >> >> read_reg should return 0 on s

[PATCH] USB: serial: Clean up the return values of read_reg and write_reg

2017-11-27 Thread Gimcuan Hui
write_reg returns 0 on success, we can make it more explicit by returning number 0 instead of result variable. read_reg should return 0 on success since this is a more common pattern. The user of read_reg has been clean-up and should be at the same commit. Signed-off-by: Gimcuan Hui <g

[PATCH] USB: serial: Clean up the return values of read_reg and write_reg

2017-11-27 Thread Gimcuan Hui
write_reg returns 0 on success, we can make it more explicit by returning number 0 instead of result variable. read_reg should return 0 on success since this is a more common pattern. The user of read_reg has been clean-up and should be at the same commit. Signed-off-by: Gimcuan Hui

[PATCH] USB: serial: Correct return value on read

2017-11-26 Thread Gimcuan Hui
It's meaningless to return buf[0] on read. Because the caller of this interface checks the return value negative or not. Instead, we should return the result variable. Signed-off-by: Gimcuan Hui <gimc...@gmail.com> --- drivers/usb/serial/ark3116.c | 4 +--- 1 file changed, 1 insertion

[PATCH] USB: serial: Correct return value on read

2017-11-26 Thread Gimcuan Hui
It's meaningless to return buf[0] on read. Because the caller of this interface checks the return value negative or not. Instead, we should return the result variable. Signed-off-by: Gimcuan Hui --- drivers/usb/serial/ark3116.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff

[PATCH] drivers: base: omit redundant interations

2017-11-10 Thread Gimcuan Hui
When error happens, these interators return the error, no interation should be continued, so make the change for getting out of while immediately. Signed-off-by: Gimcuan Hui <gimc...@gmail.com> --- drivers/base/bus.c| 2 +- drivers/base/core.c | 2 +- drivers/base/driver.c | 2 +- 3

[PATCH] drivers: base: omit redundant interations

2017-11-10 Thread Gimcuan Hui
When error happens, these interators return the error, no interation should be continued, so make the change for getting out of while immediately. Signed-off-by: Gimcuan Hui --- drivers/base/bus.c| 2 +- drivers/base/core.c | 2 +- drivers/base/driver.c | 2 +- 3 files changed, 3

[PATCH 2/5] tty: serial: jsm: add blank line after declarations

2017-11-04 Thread Gimcuan Hui
This patch fixes checkpatch.pl warning: Missing a blank line after declarations. Signed-off-by: Gimcuan Hui <gimc...@gmail.com> --- drivers/tty/serial/jsm/jsm_tty.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/tty/serial/jsm/jsm_tty.c b/drivers/tty/serial/jsm/jsm_tty.c

[PATCH 2/5] tty: serial: jsm: add blank line after declarations

2017-11-04 Thread Gimcuan Hui
This patch fixes checkpatch.pl warning: Missing a blank line after declarations. Signed-off-by: Gimcuan Hui --- drivers/tty/serial/jsm/jsm_tty.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/tty/serial/jsm/jsm_tty.c b/drivers/tty/serial/jsm/jsm_tty.c index 729d2a083a74

[PATCH 4/5] tty: serial: jsm: fix coding style

2017-11-04 Thread Gimcuan Hui
This patch fixes the checkpatch.pl complain: ERROR: else should follow close brace '}'. Signed-off-by: Gimcuan Hui <gimc...@gmail.com> --- drivers/tty/serial/jsm/jsm_tty.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/tty/serial/jsm/jsm_tty.c b/drivers/tty/

[PATCH 5/5] tty: serial: jsm: add space before the open parenthesis '('

2017-11-04 Thread Gimcuan Hui
This patch fixes the checkpatch.pl complains: space required before the open parenthesis '('. Signed-off-by: Gimcuan Hui <gimc...@gmail.com> --- drivers/tty/serial/jsm/jsm_tty.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/tty/serial/jsm/jsm_tty.c b/d

[PATCH 4/5] tty: serial: jsm: fix coding style

2017-11-04 Thread Gimcuan Hui
This patch fixes the checkpatch.pl complain: ERROR: else should follow close brace '}'. Signed-off-by: Gimcuan Hui --- drivers/tty/serial/jsm/jsm_tty.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/tty/serial/jsm/jsm_tty.c b/drivers/tty/serial/jsm/jsm_tty.c

[PATCH 5/5] tty: serial: jsm: add space before the open parenthesis '('

2017-11-04 Thread Gimcuan Hui
This patch fixes the checkpatch.pl complains: space required before the open parenthesis '('. Signed-off-by: Gimcuan Hui --- drivers/tty/serial/jsm/jsm_tty.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/tty/serial/jsm/jsm_tty.c b/drivers/tty/serial/jsm

[PATCH 3/5] tty: serial: jsm: delete space between function name and '('

2017-11-04 Thread Gimcuan Hui
This patch fixes checkpatch.pl warning: space prohibited between function name and open parenthesis '('. Signed-off-by: Gimcuan Hui <gimc...@gmail.com> --- drivers/tty/serial/jsm/jsm_tty.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/tty/serial/jsm/jsm_t

[PATCH 0/5] TTY/JSM coding style fixes

2017-11-04 Thread Gimcuan Hui
This patch set fixes several warnings and errors reported by checkpatch.pl. Gimcuan Hui (5): tty: serial: jsm: change the type of local variable tty: serial: jsm: add blank line after declarations tty: serial: jsm: delete space between function name and '(' tty: serial: jsm: fix coding

[PATCH 1/5] tty: serial: jsm: change the type of local variable

2017-11-04 Thread Gimcuan Hui
The return type of jsm_get_mstat was int, and the local var result was for the return should be int, make the change. This patch fixes the checkpatch.pl warning: Prefer 'unsigned int' to bare use of 'unsigned'. Signed-off-by: Gimcuan Hui <gimc...@gmail.com> --- drivers/tty/serial/jsm/jsm

[PATCH 3/5] tty: serial: jsm: delete space between function name and '('

2017-11-04 Thread Gimcuan Hui
This patch fixes checkpatch.pl warning: space prohibited between function name and open parenthesis '('. Signed-off-by: Gimcuan Hui --- drivers/tty/serial/jsm/jsm_tty.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/tty/serial/jsm/jsm_tty.c b/drivers/tty/serial/jsm

[PATCH 0/5] TTY/JSM coding style fixes

2017-11-04 Thread Gimcuan Hui
This patch set fixes several warnings and errors reported by checkpatch.pl. Gimcuan Hui (5): tty: serial: jsm: change the type of local variable tty: serial: jsm: add blank line after declarations tty: serial: jsm: delete space between function name and '(' tty: serial: jsm: fix coding

[PATCH 1/5] tty: serial: jsm: change the type of local variable

2017-11-04 Thread Gimcuan Hui
The return type of jsm_get_mstat was int, and the local var result was for the return should be int, make the change. This patch fixes the checkpatch.pl warning: Prefer 'unsigned int' to bare use of 'unsigned'. Signed-off-by: Gimcuan Hui --- drivers/tty/serial/jsm/jsm_tty.c | 2 +- 1 file

[PATCH] x86: kvm: mmu: make kvm_mmu_clear_all_pte_masks static

2017-11-03 Thread Gimcuan Hui
The kvm_mmu_clear_all_pte_masks interface is only used by kvm_mmu_module_init locally, and does not need to be called by other module, make it static. This patch cleans up sparse warning: symbol 'kvm_mmu_clear_all_pte_masks' was not declared. Should it be static? Signed-off-by: Gimcuan Hui <g

[PATCH] x86: kvm: mmu: make kvm_mmu_clear_all_pte_masks static

2017-11-03 Thread Gimcuan Hui
The kvm_mmu_clear_all_pte_masks interface is only used by kvm_mmu_module_init locally, and does not need to be called by other module, make it static. This patch cleans up sparse warning: symbol 'kvm_mmu_clear_all_pte_masks' was not declared. Should it be static? Signed-off-by: Gimcuan Hui

[RFC 2/4] BloodTest: perf

2017-10-13 Thread Hui Zhu
_config", "perf_freq", "perf_period", "perf_type" can set the options of perf. After record, access "str" will get the record data in string. Access "cpu0/page" will get the record data in binary that is format is in "bin_format". Signed-of

[RFC 2/4] BloodTest: perf

2017-10-13 Thread Hui Zhu
_config", "perf_freq", "perf_period", "perf_type" can set the options of perf. After record, access "str" will get the record data in string. Access "cpu0/page" will get the record data in binary that is format is in "bin_format". Signed-off-by:

[RFC 0/4] BloodTest: kernel status

2017-10-13 Thread Hui Zhu
overhead and full coverage. BloodTest is for it. It is a interface can acess function of other analysing tools and records to internal buffer that user or application can access very quickly (mmap). Now, BloodTest just support record cpu, perf and task infomation in one seconds. Hui Zhu (2

[RFC 0/4] BloodTest: kernel status

2017-10-13 Thread Hui Zhu
overhead and full coverage. BloodTest is for it. It is a interface can acess function of other analysing tools and records to internal buffer that user or application can access very quickly (mmap). Now, BloodTest just support record cpu, perf and task infomation in one seconds. Hui Zhu (2

[RFC 1/4] BloodTest: kernel status

2017-10-13 Thread Hui Zhu
p record. bt_insert and bt_pullout will call analysing tools. Signed-off-by: Hui Zhu <zhu...@xiaomi.com> --- fs/proc/stat.c | 8 +-- include/linux/kernel_stat.h| 3 ++ init/Kconfig | 3 ++ kernel/Makefile| 2 + kernel/bloodtest/Makefile

[RFC 1/4] BloodTest: kernel status

2017-10-13 Thread Hui Zhu
p record. bt_insert and bt_pullout will call analysing tools. Signed-off-by: Hui Zhu --- fs/proc/stat.c | 8 +-- include/linux/kernel_stat.h| 3 ++ init/Kconfig | 3 ++ kernel/Makefile| 2 + kernel/bloodtest/Makefile | 1 + kernel/bloodtest

[RFC 4/4] BloodTest: task

2017-10-13 Thread Hui Zhu
fomation. After record, access "str" will get the record data in string. Access "page" will get the record data in binary that is format is in "bin_format". Signed-off-by: Hui Zhu <zhu...@xiaomi.com> --- include/linux/bloodtest.h | 10 + kernel/bloodtest/

[RFC 4/4] BloodTest: task

2017-10-13 Thread Hui Zhu
fomation. After record, access "str" will get the record data in string. Access "page" will get the record data in binary that is format is in "bin_format". Signed-off-by: Hui Zhu --- include/linux/bloodtest.h | 10 + kernel/bloodtest/Makefile | 2 +- kernel/blood

[RFC 3/4] module: add /proc/modules_update_version

2017-10-13 Thread Hui Zhu
module, it also need the address of modules from /proc/modules. Add /proc/modules_update_version will help the application to get if the kernel modules address is changed or not. Signed-off-by: Hui Zhu <zhu...@xiaomi.com> --- kernel/module.c | 19 +++ 1 file changed, 19 insertio

[RFC 3/4] module: add /proc/modules_update_version

2017-10-13 Thread Hui Zhu
module, it also need the address of modules from /proc/modules. Add /proc/modules_update_version will help the application to get if the kernel modules address is changed or not. Signed-off-by: Hui Zhu --- kernel/module.c | 19 +++ 1 file changed, 19 insertions(+) diff --git a/kern

[PATCH] tty: serial: jsm: Add space before the open brace

2017-10-02 Thread Gimcuan Hui
This patch fixes the checkpatch.pl error complain: ERROR: space required before the open brace '{' Signed-off-by: Gimcuan Hui <gimc...@gmail.com> --- drivers/tty/serial/jsm/jsm_tty.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/tty/serial/jsm/jsm_tty.c b/d

[PATCH] tty: serial: jsm: Add space before the open brace

2017-10-02 Thread Gimcuan Hui
This patch fixes the checkpatch.pl error complain: ERROR: space required before the open brace '{' Signed-off-by: Gimcuan Hui --- drivers/tty/serial/jsm/jsm_tty.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/tty/serial/jsm/jsm_tty.c b/drivers/tty/serial/jsm

Re: [RFC 0/2] Use HighAtomic against long-term fragmentation

2017-09-26 Thread Hui Zhu
2017-09-26 17:51 GMT+08:00 Mel Gorman <mgor...@techsingularity.net>: > On Tue, Sep 26, 2017 at 04:46:42PM +0800, Hui Zhu wrote: >> Current HighAtomic just to handle the high atomic page alloc. >> But I found that use it handle the normal unmovable continuous page >> allo

Re: [RFC 0/2] Use HighAtomic against long-term fragmentation

2017-09-26 Thread Hui Zhu
2017-09-26 17:51 GMT+08:00 Mel Gorman : > On Tue, Sep 26, 2017 at 04:46:42PM +0800, Hui Zhu wrote: >> Current HighAtomic just to handle the high atomic page alloc. >> But I found that use it handle the normal unmovable continuous page >> alloc will help to against lo

[RFC 1/2] Try to use HighAtomic if try to alloc umovable page that order is not 0

2017-09-26 Thread Hui Zhu
The page add a new condition to let gfp_to_alloc_flags return alloc_flags with ALLOC_HARDER if the order is not 0 and migratetype is MIGRATE_UNMOVABLE. Then alloc umovable page that order is not 0 will try to use HighAtomic. Signed-off-by: Hui Zhu <zhu...@xiaomi.com> --- mm/page_alloc

<    1   2   3   4   5   6   7   8   9   10   >