[PATCH] lib/string: sysfs_streq works case insensitively

As the name shows, it checks the strings inputed from sysfs. It should work for both case-sensitive filesystem and case-insensitive filesystem. Therefore sysfs_streq should work case-insensitively. Signed-off-by: Gioh Kim --- lib/string.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions

Re: [PATCH v4] lib/string: Introduce sysfs_streqcase

On Fri, Apr 9, 2021 at 9:11 AM Andy Shevchenko wrote: > > > > On Friday, April 9, 2021, Andrew Morton wrote: >> >> On Thu, 8 Apr 2021 15:06:05 +0200 Gioh Kim wrote: >> >> > As the name shows, it checks if strings are equal in case insensitive >> >

Re: [PATCH v4] lib/string: Introduce sysfs_streqcase

On Thu, Apr 8, 2021 at 3:14 PM Jinpu Wang wrote: > > On Thu, Apr 8, 2021 at 3:06 PM Gioh Kim wrote: > > > > As the name shows, it checks if strings are equal in case insensitive > > manner. > > > > For example, drivers/infiniband/ulp/rtrs/rtrs-clt-s

Re: [PATCH v3] lib/string: Introduce sysfs_streqcase

Hi Jinpu, I removed #ifdef CONFIG_SYSFS ~ #endif. Could you please review again? On Thu, Apr 8, 2021 at 12:50 PM Jinpu Wang wrote: > > On Thu, Apr 8, 2021 at 11:34 AM Gioh Kim wrote: > > > > As the name shows, it checks if strings are equal in case insensitive > > man

[PATCH v4] lib/string: Introduce sysfs_streqcase

es using strncasecmp to check the entire string such as rtrs-clt-sysfs.c does. drivers/pnp/interface.c checks "disable" command with strncasecmp but it would also work if the command is "disable-wrong". Signed-off-by: Gioh Kim --- include/linux/stri

[PATCH v3] lib/string: Introduce sysfs_streqcase

es using strncasecmp to check the entire string such as rtrs-clt-sysfs.c does. drivers/pnp/interface.c checks "disable" command with strncasecmp but it would also work if the command is "disable-wrong". Signed-off-by: Gioh Kim --- include/linux/stri

Re: [PATCH] lib/string: Introduce sysfs_streqcase

On Wed, Apr 7, 2021 at 10:07 PM Nick Desaulniers wrote: > > On Tue, Apr 6, 2021 at 11:15 PM Gioh Kim wrote: > > > > As the name shows, it checks if strings are equal in case insensitive > > manner. > > > > For example, drivers/infiniband/ulp/rtrs/rtrs-clt-s

[PATCH] lib/string: Introduce sysfs_streqcase

es using strncasecmp to check the entire string such as rtrs-clt-sysfs.c does. drivers/pnp/interface.c checks "disable" command with strncasecmp but it would also work if the command is "disable-wrong". Signed-off-by: Gioh Kim --- include/linux/string.h | 1 + lib/string.c

Re: [PATCH] lib/string: Introduce sysfs_streqcase

On Fri, Apr 2, 2021 at 8:24 PM Kees Cook wrote: > > On Fri, Apr 02, 2021 at 11:17:13AM -0700, Nick Desaulniers wrote: > > Thanks for the patch! > > > > + akpm (please remember to run ./scripts/get_maintainer.pl on your patch > > files) > > > > On F

Re: [PATCH] lib/string: Introduce sysfs_streqcase

On Fri, Apr 2, 2021 at 8:17 PM Nick Desaulniers wrote: > > Thanks for the patch! > > + akpm (please remember to run ./scripts/get_maintainer.pl on your patch > files) > > On Fri, Apr 2, 2021 at 2:41 AM Gioh Kim wrote: > > > > As the name shows, it checks if stri

[PATCH] lib/string: Introduce sysfs_streqcase

ot;mi". But it would work even-if the input is "min-wrongcommand". And also drivers/pnp/interface.c checks "disable" command with strncasecmp but it would also work if the command is "disable-wrong". Signed-off-by: Gioh Kim --- lib/string.c | 23 ++

Re: can the idle value of /proc/stat decrease?

> The results of the two readings from /proc/stat are shown as below, the > interval between the two readings is 150 ms: > > cat /proc/stat > cpu0 5536 10 14160 4118960 0 0 227128 0 0 0 > > cat /proc/stat > cpu0 5536 10 14160 4118959 0 0 227143 0 0 0 -- Gioh Kim Cloud server

[PATCH 2/2] RDMA/rtrs: check before free

From: Gioh Kim If rtrs_iu_alloc failed to allocate buffer or map dma, there are some allocated addresses and some NULL addresses in the array. rtrs_iu_free should check data before free. Signed-off-by: Gioh Kim --- drivers/infiniband/ulp/rtrs/rtrs.c | 8 ++-- 1 file changed, 6 insertions

[PATCH 1/2] RDMA/rtrs: remove unnecessary argument dir of rtrs_iu_free

From: Gioh Kim The direction of DMA operation is already in the rtrs_iu. Signed-off-by: Gioh Kim --- drivers/infiniband/ulp/rtrs/rtrs-clt.c | 14 ++ drivers/infiniband/ulp/rtrs/rtrs-pri.h | 3 +-- drivers/infiniband/ulp/rtrs/rtrs-srv.c | 14 ++ drivers/infiniband/ulp

[PATCH] RDMA/rtrs: remove unused field of rtrs_iu

From: Gioh Kim list field is not used anywhere Signed-off-by: Gioh Kim --- drivers/infiniband/ulp/rtrs/rtrs-pri.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/infiniband/ulp/rtrs/rtrs-pri.h b/drivers/infiniband/ulp/rtrs/rtrs-pri.h index 0a93c87ef92b..b8e43dc4d95a 100644

[PATCH] block/rnbd: send_msg_close if any error occurs after send_msg_open

From: Gioh Kim After send_msg_open is done, send_msg_close should be done if any error occurs and it is necessary to recover what has been done. Signed-off-by: Gioh Kim --- drivers/block/rnbd/rnbd-clt.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/block/rnbd

[RFC] replace comments for bvl_vec with bio_vec

I think bvl_vec means bio_vec_list that was replaced with bio_vec long time ago. If I am correct, I think some comments for blk_vec need to be replaced with bio_vec. Signed-off-by: Gioh Kim --- include/linux/blk_types.h | 2 +- include/linux/bvec.h | 2 +- 2 files changed, 2 insertions

[RFC] replace comments for bvl_vec with bio_vec

I think bvl_vec means bio_vec_list that was replaced with bio_vec long time ago. If I am correct, I think some comments for blk_vec need to be replaced with bio_vec. Signed-off-by: Gioh Kim --- include/linux/blk_types.h | 2 +- include/linux/bvec.h | 2 +- 2 files changed, 2 insertions

[PATCH] md/raid1: add error handling of read error from FailFast device

should handle the read IO error and write IO error equally. So I think read IO error should set rdev faulty. Signed-off-by: Gioh Kim <gi-oh@profitbricks.com> --- drivers/md/raid1.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/md/raid1.c b/drivers/md/raid1.c index e9e330

[PATCH] md/raid1: add error handling of read error from FailFast device

should handle the read IO error and write IO error equally. So I think read IO error should set rdev faulty. Signed-off-by: Gioh Kim --- drivers/md/raid1.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/md/raid1.c b/drivers/md/raid1.c index e9e3308cb0a7..4445179aa4c8 100644

[PATCH] setlocalversion: use any tag

both of annotated tag and lightweight tag, it would be more convenient because it is common to set lightweight tag during developmene process. Signed-off-by: Gioh Kim <gi-oh@profitbricks.com> --- scripts/setlocalversion | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/s

[PATCH] setlocalversion: use any tag

both of annotated tag and lightweight tag, it would be more convenient because it is common to set lightweight tag during developmene process. Signed-off-by: Gioh Kim --- scripts/setlocalversion | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/setlocalversion b/scripts

[PATCH v2] mm/memblock.c: make the index explicit argument of for_each_memblock_type

obious. This patch is a mere cleanup and it shouldn't introduce any functional change. Acked-by: Michal Hocko <mho...@suse.com> Signed-off-by: Gioh Kim <gi-oh@profitbricks.com> --- include/linux/memblock.h | 8 mm/memblock.c| 8 2 files changed, 8 inse

[PATCH v2] mm/memblock.c: make the index explicit argument of for_each_memblock_type

obious. This patch is a mere cleanup and it shouldn't introduce any functional change. Acked-by: Michal Hocko Signed-off-by: Gioh Kim --- include/linux/memblock.h | 8 mm/memblock.c| 8 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/include/linux

[PATCH] mm/memblock.c: make the index explicit argument of for_each_memblock_type

for_each_memblock_type macro function uses idx variable that is the local variable of caller. This patch makes for_each_memblock_type use only its own arguments. Signed-off-by: Gioh Kim <gi-oh@profitbricks.com> --- include/linux/memblock.h | 8 mm/memblock.c

[PATCH] mm/memblock.c: make the index explicit argument of for_each_memblock_type

for_each_memblock_type macro function uses idx variable that is the local variable of caller. This patch makes for_each_memblock_type use only its own arguments. Signed-off-by: Gioh Kim --- include/linux/memblock.h | 8 mm/memblock.c| 8 2 files changed, 8

[RFC] guarantee inode of alias's parent

ess? I checked kernel code of v4.10. It also does not have any lock before accessing alias->d_parent. Therefore I'm attaching a patch to show my idea, not solve a problem. This patch is based on v4.10.0 Signed-off-by: Gioh Kim <gi-oh@profitbricks.com> --- fs/dcache.c | 5 + 1 fi

[RFC] guarantee inode of alias's parent

ess? I checked kernel code of v4.10. It also does not have any lock before accessing alias->d_parent. Therefore I'm attaching a patch to show my idea, not solve a problem. This patch is based on v4.10.0 Signed-off-by: Gioh Kim --- fs/dcache.c | 5 + 1 file changed, 5 insertions(+) diff -

Re: [RFC] KVM: SVM: ignore type when setting segment registers

On Tue, May 30, 2017 at 02:54:21PM +0200, Radim Krčmář wrote: > 2017-05-29 15:24+0200, Gioh Kim: > > Current code sets unusable as 1 if present is 1 and type is 0. > > In Long mode, type value in segment descriptor is ignored. > > So I think type should be ignored whe

Re: [RFC] KVM: SVM: ignore type when setting segment registers

On Tue, May 30, 2017 at 02:54:21PM +0200, Radim Krčmář wrote: > 2017-05-29 15:24+0200, Gioh Kim: > > Current code sets unusable as 1 if present is 1 and type is 0. > > In Long mode, type value in segment descriptor is ignored. > > So I think type should be ignored whe

[PATCH] KVM: SVM: ignore type when setting segment registers

e can be 0 if it is read-only data segment. Therefore type value is not related to unusable flag. This patch is based on linux-next v4.12.0-rc3. Signed-off-by: Gioh Kim <gi-oh@profitbricks.com> --- arch/x86/kvm/svm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch

[PATCH] KVM: SVM: ignore type when setting segment registers

e can be 0 if it is read-only data segment. Therefore type value is not related to unusable flag. This patch is based on linux-next v4.12.0-rc3. Signed-off-by: Gioh Kim --- arch/x86/kvm/svm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/kvm/svm.c b/arch/x86/kvm/s

[RFC] KVM: SVM: ignore type when setting segment registers

Current code sets unusable as 1 if present is 1 and type is 0. In Long mode, type value in segment descriptor is ignored. So I think type should be ignored when setting the segment registers, if type means the descriptor type in the segment descriptor. Is the type field of struct kvm_segment the

[RFC] KVM: SVM: ignore type when setting segment registers

Current code sets unusable as 1 if present is 1 and type is 0. In Long mode, type value in segment descriptor is ignored. So I think type should be ignored when setting the segment registers, if type means the descriptor type in the segment descriptor. Is the type field of struct kvm_segment the

[PATCHv2] mdadm.c: fix compile error "switch condition has boolean value"

Remove a boolean expression in switch condition to prevent compile error of some compilers, for example, gcc version 5.2.1 20151010 (Ubuntu 5.2.1-22ubuntu2). Signed-off-by: Gioh Kim <gi-oh@profitbricks.com> --- mdadm.c | 14 ++ 1 file changed, 6 insertions(+), 8 del

[PATCHv2] mdadm.c: fix compile error "switch condition has boolean value"

Remove a boolean expression in switch condition to prevent compile error of some compilers, for example, gcc version 5.2.1 20151010 (Ubuntu 5.2.1-22ubuntu2). Signed-off-by: Gioh Kim --- mdadm.c | 14 ++ 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/mdadm.c b/mdadm.c

Re: [PATCH 2/2] mdadm.c: fix compile error "switch condition has boolean value"

On Thu, Mar 30, 2017 at 08:38:35AM +1100, NeilBrown wrote: > On Thu, Mar 30 2017, jes.soren...@gmail.com wrote: > > > Gioh Kim <gi-oh@profitbricks.com> writes: > >> Remove a boolean expression in switch condition > >> to prevent compile error of some c

Re: [PATCH 2/2] mdadm.c: fix compile error "switch condition has boolean value"

On Thu, Mar 30, 2017 at 08:38:35AM +1100, NeilBrown wrote: > On Thu, Mar 30 2017, jes.soren...@gmail.com wrote: > > > Gioh Kim writes: > >> Remove a boolean expression in switch condition > >> to prevent compile error of some compilers. > > > > P

Re: [PATCH 2/2] mdadm.c: fix compile error "switch condition has boolean value"

On Wed, Mar 29, 2017 at 11:47:28AM -0400, jes.soren...@gmail.com wrote: > Gioh Kim <gi-oh@profitbricks.com> writes: > > Remove a boolean expression in switch condition > > to prevent compile error of some compilers. > > Please be specific, which compile is unable to

Re: [PATCH 2/2] mdadm.c: fix compile error "switch condition has boolean value"

On Wed, Mar 29, 2017 at 11:47:28AM -0400, jes.soren...@gmail.com wrote: > Gioh Kim writes: > > Remove a boolean expression in switch condition > > to prevent compile error of some compilers. > > Please be specific, which compile is unable to handle this? > >

[PATCH 2/2] mdadm.c: fix compile error "switch condition has boolean value"

Remove a boolean expression in switch condition to prevent compile error of some compilers. Signed-off-by: Gioh Kim <gi-oh@profitbricks.com> --- mdadm.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/mdadm.c b/mdadm.c index 08ddcab..a98a051 100644 --- a/m

[PATCH 2/2] mdadm.c: fix compile error "switch condition has boolean value"

Remove a boolean expression in switch condition to prevent compile error of some compilers. Signed-off-by: Gioh Kim --- mdadm.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/mdadm.c b/mdadm.c index 08ddcab..a98a051 100644 --- a/mdadm.c +++ b/mdadm.c @@ -1905,11

[PATCH 1/2] super1: replace hard-coded values with bit definitions

Some hard-coded values for disk status are replaced with bit definitions. Signed-off-by: Gioh Kim <gi-oh@profitbricks.com> --- super1.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/super1.c b/super1.c index f3520ac..e8b9f61 100644 --- a/super1.c +++ b/su

[PATCH 1/2] super1: replace hard-coded values with bit definitions

Some hard-coded values for disk status are replaced with bit definitions. Signed-off-by: Gioh Kim --- super1.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/super1.c b/super1.c index f3520ac..e8b9f61 100644 --- a/super1.c +++ b/super1.c @@ -1010,7 +1010,7 @@ static

Re: [PATCHv2 0/2] mdadm: setting device role of raid1 disk with failfast

Hi, Is nobody interested in those patches? On Mon, Mar 20, 2017 at 10:51:55AM +0100, Gioh Kim wrote: > Hi, > > I've found a case that failfast option of mdadm set a disk faulty wrongly. > Following is my test case. > > mdadm --create /dev/md100 -l 1 --failfast -e 1.2 -n 2

Re: [PATCHv2 0/2] mdadm: setting device role of raid1 disk with failfast

Hi, Is nobody interested in those patches? On Mon, Mar 20, 2017 at 10:51:55AM +0100, Gioh Kim wrote: > Hi, > > I've found a case that failfast option of mdadm set a disk faulty wrongly. > Following is my test case. > > mdadm --create /dev/md100 -l 1 --failfast -e 1.2 -n 2

[PATCHv2 0/2] mdadm: setting device role of raid1 disk with failfast

. If not, it was spare. This patch fixes a corner case for setting device role and prints device role if it's faulty. This patch is based on "mdadm - v4.0-8-g72b616a - 2017-03-07". v2: fix a typo of v1 Gioh Kim (1): super1: ignore failfast flag for setting device role Jack Wang (1): super1: check

[PATCHv2 1/2] super1: ignore failfast flag for setting device role

There is corner case for setting device role, if new device has failfast flag. The failfast flag should be ignored. Signed-off-by: Gioh Kim <gi-oh@profitbricks.com> Signed-off-by: Jack Wang <jinpu.w...@profitbricks.com> --- super1.c | 8 +--- 1 file changed, 5 insertions(+),

[PATCHv2 2/2] super1: check and output faulty dev role

From: Jack Wang <jinpu.w...@profitbricks.com> Output the real dev role in examine_super1, it will help to find problem. Signed-off-by: Jack Wang <jinpu.w...@profitbricks.com> Reviewed-by: Gioh Kim <gi-oh@profitbricks.com> --- super1.c | 6 -- 1 file changed, 4 insertio

[PATCHv2 0/2] mdadm: setting device role of raid1 disk with failfast

. If not, it was spare. This patch fixes a corner case for setting device role and prints device role if it's faulty. This patch is based on "mdadm - v4.0-8-g72b616a - 2017-03-07". v2: fix a typo of v1 Gioh Kim (1): super1: ignore failfast flag for setting device role Jack Wang (1): super1: check

[PATCHv2 1/2] super1: ignore failfast flag for setting device role

There is corner case for setting device role, if new device has failfast flag. The failfast flag should be ignored. Signed-off-by: Gioh Kim Signed-off-by: Jack Wang --- super1.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/super1.c b/super1.c index 882cd61

[PATCHv2 2/2] super1: check and output faulty dev role

From: Jack Wang Output the real dev role in examine_super1, it will help to find problem. Signed-off-by: Jack Wang Reviewed-by: Gioh Kim --- super1.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/super1.c b/super1.c index f3520ac..c903371 100644 --- a/super1.c

[PATCH 1/2] super1: ignore failfast flag for setting device role

There is corner case for setting device role, if new device has failfast flag. The failfast flag should be ignored. Signed-off-by: Gioh Kim <gi-oh@profitbricks.com> Signed-off-by: Jack Wang <jinpu.w...@profitbricks.com> --- super1.c | 8 +--- 1 file changed, 5 insertions(+),

[PATCH 2/2] super1: check and output faulty dev role

From: Jack Wang <jinpu.w...@profitbricks.com> Output the real dev role in examine_super1, it will help to find problem. Signed-off-by: Jack Wang <jinpu.w...@profitbricks.com> Reviewed-by: Gioh Kim <gi-oh@profitbricks.com> --- super1.c | 6 -- 1 file changed, 4 insertio

[PATCH 0/2] mdadm: setting device role of raid1 disk with failfast

. If not, it was spare. This patch fixes a corner case for setting device role and prints device role if it's faulty. This patch is based on "mdadm - v4.0-8-g72b616a - 2017-03-07". Gioh Kim (1): super1: ignore failfast flag for setting device role Jack Wang (1): super1: check and output faulty dev rol

[PATCH 1/2] super1: ignore failfast flag for setting device role

There is corner case for setting device role, if new device has failfast flag. The failfast flag should be ignored. Signed-off-by: Gioh Kim Signed-off-by: Jack Wang --- super1.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/super1.c b/super1.c index 882cd61

[PATCH 2/2] super1: check and output faulty dev role

From: Jack Wang Output the real dev role in examine_super1, it will help to find problem. Signed-off-by: Jack Wang Reviewed-by: Gioh Kim --- super1.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/super1.c b/super1.c index 1da33ef..0bf4715 100644 --- a/super1.c

[PATCH 0/2] mdadm: setting device role of raid1 disk with failfast

. If not, it was spare. This patch fixes a corner case for setting device role and prints device role if it's faulty. This patch is based on "mdadm - v4.0-8-g72b616a - 2017-03-07". Gioh Kim (1): super1: ignore failfast flag for setting device role Jack Wang (1): super1: check and output faulty dev rol

[PATCH 1/2] staging: wlan-ng: introduce a macro read_u16

read_u16 is wrapper of le16_to_cpu to read u16 variable, rather than __le16. Signed-off-by: Gioh Kim <gi-oh@profitbricks.com> --- drivers/staging/wlan-ng/prism2mgmt.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/staging/wlan-ng/prism2mgmt.h b/drivers/staging/w

[PATCH 2/2] staging: wlan-ng: prism2sta.c: apply read_u16 macro

Remove Sparse warnings "warning: cast to restricted __le16" Signed-off-by: Gioh Kim <gi-oh@profitbricks.com> --- drivers/staging/wlan-ng/prism2sta.c | 128 ++-- 1 file changed, 64 insertions(+), 64 deletions(-) diff --git a/drivers/staging/wla

[PATCH 1/2] staging: wlan-ng: introduce a macro read_u16

read_u16 is wrapper of le16_to_cpu to read u16 variable, rather than __le16. Signed-off-by: Gioh Kim --- drivers/staging/wlan-ng/prism2mgmt.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/staging/wlan-ng/prism2mgmt.h b/drivers/staging/wlan-ng/prism2mgmt.h index 88b979f

[PATCH 2/2] staging: wlan-ng: prism2sta.c: apply read_u16 macro

Remove Sparse warnings "warning: cast to restricted __le16" Signed-off-by: Gioh Kim --- drivers/staging/wlan-ng/prism2sta.c | 128 ++-- 1 file changed, 64 insertions(+), 64 deletions(-) diff --git a/drivers/staging/wlan-ng/prism2sta.c b/drivers/stagi

[PATCH 0/2] staging: wlan-ng: remove Sparse warnings for reading __le16

nd convert it into u32 type. I hope this could help. Gioh Kim (2): taging: wlan-ng: introduce a macro read_u16 staging: wlan-ng: prism2sta.c: apply read_u16 macro drivers/staging/wlan-ng/prism2mgmt.h | 3 + drivers/staging/wlan-ng/prism2sta.c | 128 +-

[PATCH 0/2] staging: wlan-ng: remove Sparse warnings for reading __le16

nd convert it into u32 type. I hope this could help. Gioh Kim (2): taging: wlan-ng: introduce a macro read_u16 staging: wlan-ng: prism2sta.c: apply read_u16 macro drivers/staging/wlan-ng/prism2mgmt.h | 3 + drivers/staging/wlan-ng/prism2sta.c | 128 +-

[PATCH 2/2] staging: greybus: fix symbolic permission coding style issues

Fix "Octal permissions are preffered than symbolic ones" issues. Signed-off-by: Gioh Kim <gi-oh@profitbricks.com> --- drivers/staging/greybus/es2.c | 4 ++-- drivers/staging/greybus/loopback.c | 2 +- drivers/staging/greybus/svc.c | 6 +++--- 3 files changed, 6

[PATCH 2/2] staging: greybus: fix symbolic permission coding style issues

Fix "Octal permissions are preffered than symbolic ones" issues. Signed-off-by: Gioh Kim --- drivers/staging/greybus/es2.c | 4 ++-- drivers/staging/greybus/loopback.c | 2 +- drivers/staging/greybus/svc.c | 6 +++--- 3 files changed, 6 insertions(+), 6 deletions(-)

[PATCH 1/2] staging: greybus: fix "line over 80 characters" coding style issues

This patch fixes only obvious lines. There are still more issues. Signed-off-by: Gioh Kim <gi-oh@profitbricks.com> --- drivers/staging/greybus/arche-apb-ctrl.c | 5 +++- drivers/staging/greybus/arche-platform.c | 43 +--- drivers/staging/greybus/audio_c

[PATCH 1/2] staging: greybus: fix "line over 80 characters" coding style issues

This patch fixes only obvious lines. There are still more issues. Signed-off-by: Gioh Kim --- drivers/staging/greybus/arche-apb-ctrl.c | 5 +++- drivers/staging/greybus/arche-platform.c | 43 +--- drivers/staging/greybus/audio_codec.c| 5 +++- drivers/staging

[PATCH] staging: fbtft: fix some coding style issues

There are many coding style issues. I fixed only obvious issues. Fixied issues: - Use 4 digit octal (0777) not decimal permissions - line over 80 characters - Avoid unnecessary line continuations I checked all files were built without error. Signed-off-by: Gioh Kim <gi-oh@profitbricks.

[PATCH] staging: fbtft: fix some coding style issues

There are many coding style issues. I fixed only obvious issues. Fixied issues: - Use 4 digit octal (0777) not decimal permissions - line over 80 characters - Avoid unnecessary line continuations I checked all files were built without error. Signed-off-by: Gioh Kim --- drivers/staging/fbtft

[PATCH] staging: greybus: es2.c: fix some coding style issues

Fix two coding style issues: 1. line over 80 characters 2. Octal permissions are preffered than symbolic ones Signed-off-by: Gioh Kim <gi-oh@profitbricks.com> --- drivers/staging/greybus/es2.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/staging/g

[PATCH] staging: greybus: es2.c: fix some coding style issues

Fix two coding style issues: 1. line over 80 characters 2. Octal permissions are preffered than symbolic ones Signed-off-by: Gioh Kim --- drivers/staging/greybus/es2.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/staging/greybus/es2.c b/drivers/staging

[RFC] Too low cache-misses ratio

Hi, Some of my AMD servers often reported "L3 ECC error and L2 cache error". I thought I could generated the errors if I set the machines on the heavy load. So I ran stress-ng tool with various options but system did not generate any error. I wanted to check the stress-ng tool really

[RFC] Too low cache-misses ratio

Hi, Some of my AMD servers often reported "L3 ECC error and L2 cache error". I thought I could generated the errors if I set the machines on the heavy load. So I ran stress-ng tool with various options but system did not generate any error. I wanted to check the stress-ng tool really

Re: [RFC] IB/srp: export reset-host via sysfs

On 04.10.2016 16:24, Bart Van Assche wrote: On 10/04/16 07:00, Gioh Kim wrote: Standard SCSI device has host_reset sysfs interface. This patch exports the host_reset interface for upper layer. Signed-off-by: Gioh Kim <gi-oh@profitbricks.com> --- drivers/infiniband/ulp/srp/ib_srp.

Re: [RFC] IB/srp: export reset-host via sysfs

On 04.10.2016 16:24, Bart Van Assche wrote: On 10/04/16 07:00, Gioh Kim wrote: Standard SCSI device has host_reset sysfs interface. This patch exports the host_reset interface for upper layer. Signed-off-by: Gioh Kim --- drivers/infiniband/ulp/srp/ib_srp.c | 10 ++ 1 file changed

[RFC] IB/srp: export reset-host via sysfs

Standard SCSI device has host_reset sysfs interface. This patch exports the host_reset interface for upper layer. Signed-off-by: Gioh Kim <gi-oh@profitbricks.com> --- drivers/infiniband/ulp/srp/ib_srp.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/drivers/infiniba

[RFC] IB/srp: export reset-host via sysfs

Standard SCSI device has host_reset sysfs interface. This patch exports the host_reset interface for upper layer. Signed-off-by: Gioh Kim --- drivers/infiniband/ulp/srp/ib_srp.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/drivers/infiniband/ulp/srp/ib_srp.c b/drivers

Re: [PATCH v1 03/19] fs/anon_inodes: new interface to create new inode

On 11.03.2016 09:05, Al Viro wrote: On Fri, Mar 11, 2016 at 04:30:07PM +0900, Minchan Kim wrote: From: Gioh Kim <guru...@hanmail.net> The anon_inodes has already complete interfaces to create manage many anonymous inodes but don't have interface to get new inode. Other sub-modules can

Re: [PATCH v1 03/19] fs/anon_inodes: new interface to create new inode

On 11.03.2016 09:05, Al Viro wrote: On Fri, Mar 11, 2016 at 04:30:07PM +0900, Minchan Kim wrote: From: Gioh Kim The anon_inodes has already complete interfaces to create manage many anonymous inodes but don't have interface to get new inode. Other sub-modules can create anonymous inode

Re: [PATCH 0/5] Add Korean translation of memory-barriers.txt

deletions(-) create mode 100644 Documentation/ko_KR/memory-barriers.txt Hello SeongJae, This document is a little bit difficult to me. Your translation should be very helpful. Let me read it in detail for weekend and give you feedback. I thank you for your contribution. -- Best regards, Gioh Kim

Re: [PATCH 0/5] Add Korean translation of memory-barriers.txt

deletions(-) create mode 100644 Documentation/ko_KR/memory-barriers.txt Hello SeongJae, This document is a little bit difficult to me. Your translation should be very helpful. Let me read it in detail for weekend and give you feedback. I thank you for your contribution. -- Best regards, Gioh Kim

[PATCH] hwmon: (fam15h_power) Add bit masking for tdp_limit

Add bit masking to read ApmTdpLimit precisely Signed-off-by: Gioh Kim --- drivers/hwmon/fam15h_power.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/drivers/hwmon/fam15h_power.c b/drivers/hwmon/fam15h_power.c index f77eb97..4f695d8 100644 --- a/drivers/hwmon

[PATCH] hwmon: (fam15h_power) Add bit masking for tdp_limit

Add bit masking to read ApmTdpLimit precisely Signed-off-by: Gioh Kim <gi-oh@profitbricks.com> --- drivers/hwmon/fam15h_power.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/drivers/hwmon/fam15h_power.c b/drivers/hwmon/fam15h_power.c index f77eb97..4

[RFC] hwmon: (fam15h_power) Add bit masking for tdp_limit

The bits [31:29] of D18F5xE8 TDP Limit3 are reserved. I think it'd better to add masking to read ApmTdpLimit[28:16] precisely. Signed-off-by: Gioh Kim --- drivers/hwmon/fam15h_power.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/hwmon/fam15h_power.c b/drivers

[RFC] hwmon: (fam15h_power) Add bit masking for tdp_limit

The bits [31:29] of D18F5xE8 TDP Limit3 are reserved. I think it'd better to add masking to read ApmTdpLimit[28:16] precisely. Signed-off-by: Gioh Kim <gi-oh@profitbricks.com> --- drivers/hwmon/fam15h_power.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

[PATCH] include/linux/genhd.h: remove comments for deprecated function

Remove comments about disk_stat_* functions which are deprecated. Signed-off-by: Gioh Kim --- include/linux/genhd.h | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/include/linux/genhd.h b/include/linux/genhd.h index 847cc1d..f34a767 100644 --- a/include/linux/genhd.h

[PATCH] include/linux/genhd.h: remove comments for deprecated function

Remove comments about disk_stat_* functions which are deprecated. Signed-off-by: Gioh Kim <gi-oh@profitbricks.com> --- include/linux/genhd.h | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/include/linux/genhd.h b/include/linux/genhd.h index 847cc1d..f34a767

Re: [PATCH 1/1] staging: ion: Fix error handling in ion_buffer_create

return ERR_PTR(ret); Suggested-by: Pintu Kumar Reviewed-by: Pintu Kumar It's nice! Reviewed-by: Gioh Kim -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: [PATCH 1/1] staging: ion: Fix error handling in ion_buffer_create

;free(buffer); err2: kfree(buffer); return ERR_PTR(ret); Suggested-by: Pintu Kumar <pint...@samsung.com> Reviewed-by: Pintu Kumar <pint...@samsung.com> It's nice! Reviewed-by: Gioh Kim <gioh@lge.com> -- To unsubscribe from this list: send the line &qu

Re: [PATCH 0/4] enable migration of driver pages

2015-07-29 오후 9:46에 Daniel Vetter 이(가) 쓴 글: On Wed, Jul 29, 2015 at 01:16:14PM +0100, Mel Gorman wrote: On Wed, Jul 29, 2015 at 12:55:54PM +0200, Daniel Vetter wrote: On Wed, Jul 29, 2015 at 11:49:45AM +0100, Mel Gorman wrote: On Mon, Jul 13, 2015 at 05:35:15PM +0900, Gioh Kim wrote: My

Re: [PATCH 0/4] enable migration of driver pages

2015-07-29 오후 9:46에 Daniel Vetter 이(가) 쓴 글: On Wed, Jul 29, 2015 at 01:16:14PM +0100, Mel Gorman wrote: On Wed, Jul 29, 2015 at 12:55:54PM +0200, Daniel Vetter wrote: On Wed, Jul 29, 2015 at 11:49:45AM +0100, Mel Gorman wrote: On Mon, Jul 13, 2015 at 05:35:15PM +0900, Gioh Kim wrote: My

Re: [PATCH 4/4] mm: remove direct calling of migration

2015-07-27 오후 10:58에 Vlastimil Babka 이(가) 쓴 글: On 07/13/2015 10:35 AM, Gioh Kim wrote: From: Gioh Kim Migration is completely generalized so that migrating mobile page is processed with lru-pages in move_to_new_page. Signed-off-by: Gioh Kim Acked-by: Rafael Aquini Why not just fold

Re: [PATCH 2/4] mm/compaction: enable mobile-page migration

On Mon, Jul 27, 2015 at 4:55 PM, Vlastimil Babka wrote: On 07/13/2015 10:35 AM, Gioh Kim wrote: From: Gioh Kim Add framework to register callback functions and check page mobility. There are some modes for page isolation so that isolate interface has arguments of page address

Re: [PATCH 2/4] mm/compaction: enable mobile-page migration

On Mon, Jul 27, 2015 at 4:55 PM, Vlastimil Babka vba...@suse.cz wrote: On 07/13/2015 10:35 AM, Gioh Kim wrote: From: Gioh Kim guru...@hanmail.net Add framework to register callback functions and check page mobility. There are some modes for page isolation so that isolate interface has

Re: [PATCH 4/4] mm: remove direct calling of migration

2015-07-27 오후 10:58에 Vlastimil Babka 이(가) 쓴 글: On 07/13/2015 10:35 AM, Gioh Kim wrote: From: Gioh Kim guru...@hanmail.net Migration is completely generalized so that migrating mobile page is processed with lru-pages in move_to_new_page. Signed-off-by: Gioh Kim gioh@lge.com Acked

Re: [PATCH 0/4] enable migration of driver pages

2015-07-13 오후 6:24에 Konstantin Khlebnikov 이(가) 쓴 글: On Mon, Jul 13, 2015 at 11:35 AM, Gioh Kim wrote: From: Gioh Kim Hello, This series try to enable migration of non-LRU pages, such as driver's page. My ARM-based platform occured severe fragmentation problem after long-term (several

Re: [RFCv3 2/5] mm/compaction: enable mobile-page migration

@@ -51,6 +54,66 @@ extern void compaction_defer_reset(struct zone *zone, int order, bool alloc_success); extern bool compaction_restarting(struct zone *zone, int order); +static inline bool mobile_page(struct page *page) +{ + return page->mapping &&

[PATCH 4/4] mm: remove direct calling of migration

From: Gioh Kim Migration is completely generalized so that migrating mobile page is processed with lru-pages in move_to_new_page. Signed-off-by: Gioh Kim Acked-by: Rafael Aquini --- mm/migrate.c | 15 --- 1 file changed, 15 deletions(-) diff --git a/mm/migrate.c b/mm/migrate.c

[PATCH 3/4] mm/balloon: apply mobile page migratable into balloon

From: Gioh Kim Apply mobile page migration into balloon driver. The balloong driver has an anonymous inode that manages address_space_operation for page migration. Compaction calls interfaces of mobile page migration instead of calling balloon migration directly. Signed-off-by: Gioh Kim Acked

  1   2   3   4   5   >