Signed-off-by: Yaowei Bai bywxiao...@163.com
---
MAINTAINERS | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 3589d67..d46a77c 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -270,12 +270,12 @@ F:drivers/acpi/
F: drivers/pnp
Signed-off-by: Yaowei Bai bywxiao...@163.com
---
Documentation/acpi/enumeration.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/acpi/enumeration.txt
b/Documentation/acpi/enumeration.txt
index b60d2ab..9b121a5 100644
--- a/Documentation/acpi/enumeration.txt
Signed-off-by: Yaowei Bai bywxiao...@163.com
---
Documentation/vm/balance | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/vm/balance b/Documentation/vm/balance
index c46e68c..3a2cacf 100644
--- a/Documentation/vm/balance
+++ b/Documentation/vm/balance
@@ -8,7
Signed-off-by: Yaowei Bai bywxiao...@163.com
---
mm/page_alloc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/mm/page_alloc.c b/mm/page_alloc.c
index 7633c50..c88d495 100644
--- a/mm/page_alloc.c
+++ b/mm/page_alloc.c
@@ -172,7 +172,7 @@ static void __free_pages_ok(struct
Alter 'taks' - 'task'
Signed-off-by: Yaowei Bai bywxiao...@163.com
---
mm/oom_kill.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/mm/oom_kill.c b/mm/oom_kill.c
index 642f38c..5d6a458 100644
--- a/mm/oom_kill.c
+++ b/mm/oom_kill.c
@@ -408,7 +408,7 @@ bool
On Fri, Mar 06, 2015 at 09:59:38AM -0600, Josh Poimboeuf wrote:
On Tue, Mar 03, 2015 at 10:08:02PM +0800, Yaowei Bai wrote:
As we have moved to 4.x, it should be reflected in README.
Signed-off-by: Yaowei Bai bywxiao...@163.com
---
README | 34 +-
1
On Mon, Mar 09, 2015 at 11:34:50AM -0500, Josh Poimboeuf wrote:
Personally I don't think the 4.x numbers add anything to the
understanding of the document.
Maybe the reason you think so is that you are quite familiar with the kernel,
but we should also take care of the ones who are not familiar
Add parentheses to make the two deltas consistent.
Signed-off-by: Yaowei Bai bywxiao...@163.com
---
mm/page_alloc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/mm/page_alloc.c b/mm/page_alloc.c
index 7abfa70..12c96ad 100644
--- a/mm/page_alloc.c
+++ b/mm/page_alloc.c
Alter 'controls' - 'control'.
Signed-off-by: Yaowei Bai bywxiao...@163.com
---
mm/page_alloc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/mm/page_alloc.c b/mm/page_alloc.c
index 12c96ad..5158fa2 100644
--- a/mm/page_alloc.c
+++ b/mm/page_alloc.c
@@ -5716,7 +5716,7
As we have moved to 4.x, it should be reflected in README.
Signed-off-by: Yaowei Bai bywxiao...@163.com
---
README | 34 +-
1 file changed, 17 insertions(+), 17 deletions(-)
diff --git a/README b/README
index a24ec89..67fc7a8 100644
--- a/README
+++ b/README
Kernel.org is only hosting patches and kernel compressed with xz,
so change the old gzip/bzip2 instances to xz.
Signed-off-by: Yaowei Bai bywxiao...@163.com
---
README | 18 +-
1 file changed, 5 insertions(+), 13 deletions(-)
diff --git a/README b/README
index 67fc7a8..69c68fb
As we have moved to 4.x, it should be reflected in README.
Signed-off-by: Yaowei Bai bywxiao...@163.com
---
README | 34 +-
1 file changed, 17 insertions(+), 17 deletions(-)
diff --git a/README b/README
index a24ec89..67fc7a8 100644
--- a/README
+++ b/README
The comment says that the per-cpu batchsize and zone watermarks
are determined by present_pages which is definitely wrong, they
are both calculated from managed_pages. Fix it.
Signed-off-by: Yaowei Bai bywxiao...@163.com
---
mm/page_alloc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
's/amd/and/'
Signed-off-by: Yaowei Bai bywxiao...@163.com
---
include/linux/memblock.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/linux/memblock.h b/include/linux/memblock.h
index cc4b019..273aad7 100644
--- a/include/linux/memblock.h
+++ b/include/linux
The major portion of check_new_page() and free_pages_check() are same,
introduce a helper function check_one_page() for readablity.
Signed-off-by: Yaowei Bai bywxiao...@163.com
---
mm/page_alloc.c | 54 +++---
1 file changed, 23 insertions(+), 31
There is no 'crashkernel=auto' entry in kernel-parameters.txt, borrow it
from kexec-kdump-howto.txt file in the kexec-tools-2.0.0 package.
Signed-off-by: Yaowei Bai bywxiao...@163.com
---
Documentation/kernel-parameters.txt | 9 +
1 file changed, 9 insertions(+)
diff --git
There is no 'crashkernel=auto' entry in kernel-parameters.txt, borrow it
from kexec-kdump-howto.txt file in the kexec-tools-2.0.0 package.
Signed-off-by: Yaowei Bai bywxiao...@163.com
---
Documentation/kernel-parameters.txt | 9 +
1 file changed, 9 insertions(+)
diff --git
We use sysctl_lowmem_reserve_ratio rather than sysctl_lower_zone_reserve_ratio
to
determine how aggressive the kernel is in defending lowmem from the possibility
of
being captured into pinned user memory. To avoid misleading, correct it in some
comments.
Signed-off-by: Yaowei Bai bywxiao...@163
Every zone's protection is calculated from managed_pages not
present_pages, to avoid misleading, correct it.
Signed-off-by: Yaowei Bai bywxiao...@163.com
---
Documentation/sysctl/vm.txt | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/Documentation/sysctl/vm.txt b
New_valid_dev() always returns true, so that's unnecessary to perform
new_valid_dev() checks in some filesystems. Most checks of new_valid_dev()
have been removed so let's drop this last one and then we can remove
new_valid_dev() from the source code.
No functional change.
Signed-off-by: Yaowei
As all new_valid_dev() checks have been removed it's time to drop
new_valid_dev() itself.
No functional change.
Signed-off-by: Yaowei Bai <baiyao...@cmss.chinamobile.com>
---
include/linux/kdev_t.h | 5 -
1 file changed, 5 deletions(-)
diff --git a/include/linux/kdev_t.h b/include
This patch makes kvm_is_visible_gfn return bool due to this particular
function only using either one or zero as its return value.
No functional change.
Signed-off-by: Yaowei Bai <baiyao...@cmss.chinamobile.com>
---
include/linux/kvm_host.h | 2 +-
virt/kvm/kvm_main.c | 6 +++---
2
This patch makes kvm_para_has_feature return bool due to this
particular function only using either one or zero as its return
value.
No functional change.
Signed-off-by: Yaowei Bai <baiyao...@cmss.chinamobile.com>
---
include/linux/kvm_para.h | 6 ++
1 file changed, 2 insertions
This patch makes initrd_load return bool due to this particular
function only using either one or zero as its return value.
No functional change.
Signed-off-by: Yaowei Bai <baiyao...@cmss.chinamobile.com>
---
init/do_mounts.h| 4 ++--
init/do_mounts_initrd.c | 6 +++---
2 files c
This patch makes obsolete_checksetup return bool due to this
particular function only using either one or zero as its return
value.
No functional change.
Signed-off-by: Yaowei Bai <baiyao...@cmss.chinamobile.com>
---
init/main.c | 10 +-
1 file changed, 5 insertions(+), 5 del
On Mon, Nov 16, 2015 at 02:05:46AM -0800, David Rientjes wrote:
> On Mon, 16 Nov 2015, Yaowei Bai wrote:
>
> > diff --git a/include/linux/gfp.h b/include/linux/gfp.h
> > index 6523109..1da03f5 100644
> > --- a/include/linux/gfp.h
> > +++ b/include/linux/gfp.h
&
On Mon, Nov 16, 2015 at 02:10:10AM -0800, David Rientjes wrote:
> On Mon, 16 Nov 2015, Yaowei Bai wrote:
>
> > diff --git a/include/linux/mm_inline.h b/include/linux/mm_inline.h
> > index cf55945..af73135 100644
> > --- a/include/linux/mm_inline.h
> > +++ b/include
On Mon, Nov 16, 2015 at 01:45:01PM +0100, Michal Hocko wrote:
> On Mon 16-11-15 14:51:26, Yaowei Bai wrote:
> [...]
> > @@ -72,16 +72,10 @@ struct zoneref *next_zones_zonelist(struct zoneref *z,
> > }
> >
> > #ifdef CONFIG_ARCH_HAS_HOLES_MEMORYMODEL
> > -i
On Sat, Nov 14, 2015 at 06:07:51PM +0800, Amos Jianjun Kong wrote:
> On Sat, Nov 14, 2015 at 11:21 AM, Yaowei Bai
> <baiyao...@cmss.chinamobile.com> wrote:
> > This patch makes kvm_is_visible_gfn return bool due to this particular
> > function only using either one or
In another patch kvm_is_visible_gfn is maken return bool due to this
function only returns zero or one as its return value, let's also make
kvmppc_visible_gpa return bool to keep consistent.
No functional change.
Signed-off-by: Yaowei Bai <baiyao...@cmss.chinamobile.com>
---
arch/power
This patch makes memmap_valid_within return bool due to this
particular function only using either one or zero as its return
value.
This patch also refactors memmap_valid_within for simplicity.
No functional change.
Signed-off-by: Yaowei Bai <baiyao...@cmss.chinamobile.com>
---
include
Since commit a0b8cab3 ("mm: remove lru parameter from __pagevec_lru_add
and remove parts of pagevec API") there's no user of this function anymore,
so remove it.
Signed-off-by: Yaowei Bai <baiyao...@cmss.chinamobile.com>
---
include/linux/mmzone.h | 5 -
1 file changed, 5 de
This patch makes gfp_zonelist return bool due to this
particular function only using either one or zero as its return
value.
This patch also makes gfp_zonelist return directly by removing if.
No functional change.
Signed-off-by: Yaowei Bai <baiyao...@cmss.chinamobile.com>
---
include
This patch makes page_is_file_cache return bool to improve
readability due to this particular function only using either
one or zero as its return value.
No functional change.
Signed-off-by: Yaowei Bai <baiyao...@cmss.chinamobile.com>
---
include/linux/mm_inline.h | 6 +++---
1 file chan
This patch makes is_file_shm_hugepages return bool to improve
readability due to this particular function only using either
one or zero as its return value.
No functional change.
Signed-off-by: Yaowei Bai <baiyao...@cmss.chinamobile.com>
---
include/linux/shm.h | 6 +++---
ipc
This patchset only performs some small improvement to mm. First, make
several functions return bool to improve readability, and then remove
unused is_unevictable_lru function and refactor memmap_valid_within
for simplicity.
No functional change.
Yaowei Bai (7):
ipc/shm: is_file_shm_hugepages
This patch makes memblock_is_memory/reserved return bool to improve
readability due to this particular function only using either
one or zero as its return value.
No functional change.
Signed-off-by: Yaowei Bai <baiyao...@cmss.chinamobile.com>
---
include/linux/memblock.h | 4 ++
This patch makes is_file_hugepages return bool to improve
readability due to this particular function only using either
one or zero as its return value.
This patch also removed the if condition to make is_file_hugepages
return directly.
No functional change.
Signed-off-by: Yaowei Bai <bai
On Thu, Oct 29, 2015 at 02:09:01PM -0400, Theodore Ts'o wrote:
> On Mon, Sep 28, 2015 at 09:50:37PM +0800, Yaowei Bai wrote:
> > As new_valid_dev always returns 1, so !new_valid_dev check is not
> > needed, remove it.
> >
> > Signed-off-by: Yaowei Bai <bywxiao...
On Thu, Oct 01, 2015 at 10:49:41PM +0200, Mikulas Patocka wrote:
>
>
> On Wed, 30 Sep 2015, Yaowei Bai wrote:
>
> > On Tue, Sep 29, 2015 at 05:03:24PM +0200, Mikulas Patocka wrote:
> > > Hi
> > >
> > > I suppose that you should remove the function
On Tue, Sep 29, 2015 at 04:47:30PM +0300, Boaz Harrosh wrote:
> On 09/28/2015 04:43 PM, Yaowei Bai wrote:
> > As new_valid_dev always returns 1, so !new_valid_dev check is not
> > needed, remove it.
> >
> > Signed-off-by: Yaowei Bai <bywxiao...@163.com
This patch makes lockdep_nfnl_is_held return bool to improve
readability due to this particular function only using either
one or zero as its return value.
No functional change.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
include/linux/netfilter/nfnetlink.h | 6 +++---
net/net
This patch makes bad_mask return bool due to this particular function
only using either one or zero as its return value.
No functional change.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
include/linux/inetdevice.h | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff
This patch makes inet_ifa_match return bool due to this
particular function only using either one or zero as its return
value.
No functional change.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
include/linux/inetdevice.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
This patch makes lockdep_rtnl_is_held return bool due to this
particular function only using either one or zero as its return
value.
In another patch lockdep_is_held is also made return bool.
No functional change.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
include/linux/rtnetlink
Hi Andrew,
Could you please consider take patchset to mm-tree:
http://marc.info/?l=linux-kernel=144344695929685=2
This patchset removes unnessary new_valid_dev() check in several
filesystems because this function always return true.
This patchset also removes huge_valid_dev() function because
This patch makes can_dropped_invalid_skb return bool due to this
particular function only using either one or zero as its return
value.
No functional change.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
include/linux/can/dev.h | 6 +++---
1 file changed, 3 insertions(+), 3 del
This patch makes dccp_list_has_service return bool due to this
particular function only using either one or zero as its return
value.
No functional change.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
include/linux/dccp.h | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
This patchset makes several functions in net return bool to improve
readability and/or simplicity because these functions only use one
or zero as their return value.
No functional changes.
Yaowei Bai (9):
net/netlink: lockdep_genl_is_held can be boolean
net/ieee80211: ieee80211_is_* can
This patch makes lockdep_genl_is_held return bool to improve
readability due to this particular function only using either
one or zero as its return value.
No functional change.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
include/linux/genetlink.h | 2 +-
net/netlink/genetlink.c
This patch makes dccp_bad_service_code return bool due to these
particular functions only using either one or zero as their return
value.
dccp_list_has_service is also been made return bool in this patchset.
No functional change.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
ne
This patch makes ieee80211_is_* return bool to improve
readability due to these particular functions only using either
one or zero as their return value.
No functional change.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
include/linux/ieee80211.
On Fri, Oct 09, 2015 at 12:14:31PM +0200, Marc Kleine-Budde wrote:
> On 10/08/2015 03:28 PM, Yaowei Bai wrote:
> > This patch makes can_dropped_invalid_skb return bool due to this
> > particular function only using either one or zero as its return
> > value.
> >
This patch makes memmap_valid_within return bool due to this
particular function only using either one or zero as its return
value.
No functional change.
Signed-off-by: Yaowei Bai <baiyao...@cmss.chinamobile.com>
---
include/linux/mmzone.h | 6 +++---
mm/mmzone.c| 8 -
Since commit 3bc1fa8a ("LSM: remove BSD secure level security module")
there is no user of cap_is_fs_cap any more, so remove it.
Signed-off-by: Yaowei Bai <baiyao...@cmss.chinamobile.com>
---
include/linux/capability.h | 6 --
1 file changed, 6 deletions(-)
diff --git
This patch makes cap_issubset/isclear return bool due to these
functions only using either one or zero as their return
value.
No functional change.
Signed-off-by: Yaowei Bai <baiyao...@cmss.chinamobile.com>
---
include/linux/capability.h | 16
1 file changed, 8 insertions
Hardcoding index to zonelists array in gfp_zonelist() is not a good idea,
let's enumerate it to improve readability.
No functional change.
Signed-off-by: Yaowei Bai <baiyao...@cmss.chinamobile.com>
---
include/linux/gfp.h| 4 ++--
include/linux/mmzone.h | 20 +---
2
This patch makes path_is_under return bool to improve
readability due to this particular function only using either
one or zero as its return value.
No functional change.
Signed-off-by: Yaowei Bai <baiyao...@cmss.chinamobile.com>
---
fs/namespace.c | 4 ++--
include/linux/fs.h | 2
On Mon, Nov 16, 2015 at 09:44:11PM -0800, David Rientjes wrote:
> On Tue, 17 Nov 2015, Yaowei Bai wrote:
>
> > diff --git a/include/linux/gfp.h b/include/linux/gfp.h
> > index 6523109..14a6249 100644
> > --- a/include/linux/gfp.h
> > +++ b/include/linux/gfp.h
> &g
This patch makes ceph_frag_contains_value return bool to improve
readability due to this particular function only using either one or
zero as its return value.
No functional change.
Signed-off-by: Yaowei Bai <baiyao...@cmss.chinamobile.com>
---
include/linux/ceph/ceph_frag.h | 2 +-
These functions were introduced in commit 3d14c5d2b ("ceph: factor
out libceph from Ceph file system"). Howover, there's no user of
these functions since then, so remove them for simplicity.
Signed-off-by: Yaowei Bai <baiyao...@cmss.chinamobile.com>
---
include/linux/ceph/
This patch makes is_subdir return bool to improve
readability due to this particular function only using either
one or zero as its return value.
No functional change.
Signed-off-by: Yaowei Bai <baiyao...@cmss.chinamobile.com>
---
fs/dcache.c| 14 +++---
include/linux/fs.
Currently when CONFIG_BLOCK is defined sb_is_blkdev_sb returns bool,
while when CONFIG_BLOCK is not defined it returns int. Let's keep
consistent to make sb_is_blkdev_sb return bool as well when CONFIG_BLOCK
isn't defined.
No functional change.
Signed-off-by: Yaowei Bai <bai
-off-by: Yaowei Bai bywxiao...@163.com
---
mm/page_alloc.c | 54 +++---
1 file changed, 23 insertions(+), 31 deletions(-)
diff --git a/mm/page_alloc.c b/mm/page_alloc.c
index 5b5240b..0c9c82a 100644
--- a/mm/page_alloc.c
+++ b/mm/page_alloc.c
On Tue, Aug 25, 2015 at 04:03:22PM +0200, Michal Hocko wrote:
On Tue 25-08-15 21:26:30, Yaowei Bai wrote:
[...]
static inline int check_new_page(struct page *page)
{
- const char *bad_reason = NULL;
- unsigned long bad_flags = 0;
-
- if (unlikely(page_mapcount(page
On Tue, Aug 25, 2015 at 05:06:33PM +0200, Michal Hocko wrote:
On Tue 25-08-15 22:01:31, Yaowei Bai wrote:
Every zone's protection is calculated from managed_pages not
present_pages, to avoid misleading, correct it.
This can be folded in to your previous patch
http://marc.info/?l=linux-mmm
to be used to tell whether the call of these two functions is
being made on behalf of a hotplug event, or happening at boot-time.
However, init_currently_empty_zone() does not use this parameter at all,
so remove it.
Signed-off-by: Yaowei Bai bywxiao...@163.com
---
include/linux/mmzone.h | 3
On Tue, Aug 25, 2015 at 04:03:22PM +0200, Michal Hocko wrote:
On Tue 25-08-15 21:26:30, Yaowei Bai wrote:
[...]
static inline int check_new_page(struct page *page)
{
- const char *bad_reason = NULL;
- unsigned long bad_flags = 0;
-
- if (unlikely(page_mapcount(page
On Mon, Aug 24, 2015 at 11:26:23AM -0600, Jonathan Corbet wrote:
On Mon, 24 Aug 2015 23:04:29 +0800
Yaowei Bai bywxiao...@163.com wrote:
There is no 'crashkernel=auto' entry in kernel-parameters.txt, borrow it
from kexec-kdump-howto.txt file in the kexec-tools-2.0.0 package.
OK, so I
The major portion of check_new_page() and free_pages_check() are same,
introduce a helper function check_one_page() for simplification.
Change in v2:
- use bad_flags as parameter directly per Michal Hocko
Signed-off-by: Yaowei Bai bywxiao...@163.com
---
mm/page_alloc.c | 49
On Tue, Sep 08, 2015 at 09:19:07PM +0200, Vlastimil Babka wrote:
> On 08/27/2015 02:51 PM, Yaowei Bai wrote:
> > The major portion of check_new_page() and free_pages_check() are same,
> > introduce a helper function check_one_page() for simplification.
> >
> >
On Wed, Sep 09, 2015 at 03:39:54PM +0200, Vlastimil Babka wrote:
> On 09/08/2015 09:19 PM, Vlastimil Babka wrote:
> >bloat-o-meter looks favorably with my gcc, although there shouldn't be a real
> >reason for it, as the inlining didn't change:
> >
> >add/remove: 1/1 grow/shrink: 1/1 up/down:
-off-by: Yaowei Bai <bywxiao...@163.com>
---
mm/compaction.c | 26 ++
1 file changed, 14 insertions(+), 12 deletions(-)
diff --git a/mm/compaction.c b/mm/compaction.c
index c5c627a..a8e6593 100644
--- a/mm/compaction.c
+++ b/mm/compaction.c
@@ -1197,6 +1197,15 @@
Delete unnecessary if to let inactive_anon_is_low_global return
directly.
No functional changes.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
mm/vmscan.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/mm/vmscan.c b/mm/vmscan.c
index 2d978b2..2785d8e
Introduce is_sysrq_oom helper function indicating oom kill triggered
by sysrq to improve readability.
No functional changes.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
mm/oom_kill.c | 17 +
1 file changed, 13 insertions(+), 4 deletions(-)
diff --git a/mm/oom_kill.
As currently new_valid_dev always returns 1, so new_valid_dev check is not
needed, remove it.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
fs/ubifs/dir.c | 3 ---
fs/ubifs/misc.h | 9 ++---
2 files changed, 2 insertions(+), 10 deletions(-)
diff --git a/fs/ubifs/dir.c b/fs/ubifs
On Mon, Sep 28, 2015 at 09:20:32PM +0200, Richard Weinberger wrote:
> On Mon, Sep 28, 2015 at 4:53 PM, Yaowei Bai <bywxiao...@163.com> wrote:
> > As new_valid_dev always returns 1, so new_valid_dev check is not
> > needed, remove it.
> >
> > Signed-off-b
This patch makes audit_tree_match return bool to improve readability
due to this particular function only using either one or zero as its
return value.
No functional change.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
kernel/audit.h | 2 +-
kernel/audit_tree.c | 6 +++---
2
This patch makes audit_string_contains_control return bool to improve
readability due to this particular function only using either one or
zero as its return value.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
include/linux/audit.h | 2 +-
kernel/audit.c| 6 +++---
2 files c
This patch makes report_latency return bool to improve readability,
indicating whether this new latency should be reported/recorded.
No functional change.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
kernel/trace/trace_sched_wakeup.c | 8
1 file changed, 4 insertions
This patch makes is_legal_op return bool to improve readability
due to this particular function only using either one or zero as its
return value.
No functional change.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
kernel/trace/trace_events_filter.c | 8
1 file chan
This patch makes audit_dummy_context return bool due to this
particular function only using either one or zero as its return
value.
No functional change.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
include/linux/audit.h | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
This patch makes rb_is_reader_page return bool to improve readability
due to this particular function only using either one or zero as its
return value.
No functional change.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
kernel/trace/ring_buffer.c | 2 +-
1 file changed, 1 insertion
This patch makes rb_event_is_commit return bool to improve readability
due to this particular function only using either one or zero as its
return value.
No functional change.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
kernel/trace/ring_buffer.c | 4 ++--
1 file changed, 2 inse
This patch makes rb_per_cpu_empty return bool to improve readability.
No functional change.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
kernel/trace/ring_buffer.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/kernel/trace/ring_buffer.c b/kernel
This patch makes ring_buffer_empty and ring_buffer_empty_cpu return
bool to improve readability.
No functional change.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
include/linux/ring_buffer.h | 4 ++--
kernel/trace/ring_buffer.c | 10 +-
2 files changed, 7 insertions
This patch makes ftrace_event_is_function return bool to improve readability
due to this particular function only using either one or zero as its
return value.
No functional change.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
kernel/trace/trace.h| 2 +-
kernel
This patchset just make some functions return bool to improve readability
and/or simplicity.
No functional change.
Yaowei Bai (8):
kernel/trace: report_latency in trace_sched_wakeup.c can be boolean
kernel/trace: report_latency in trace_irqsoff.c can be boolean
kernel/trace
This patch makes report_latency return bool due to this
particular function only using either one or zero as its
return value.
No functional change.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
kernel/trace/trace_irqsoff.c | 8
1 file changed, 4 insertions(+), 4 del
On Tue, Sep 29, 2015 at 04:47:30PM +0300, Boaz Harrosh wrote:
> On 09/28/2015 04:43 PM, Yaowei Bai wrote:
> > As new_valid_dev always returns 1, so !new_valid_dev check is not
> > needed, remove it.
> >
> > Signed-off-by: Yaowei Bai <bywxiao...@163.com
valid_dev,new_valid_dev)(x)
so i just left new_valid_dev untouched and removed other users in fs first.
>
> Mikulas
>
>
> On Mon, 28 Sep 2015, Yaowei Bai wrote:
>
> > As new_valid_dev always returns 1, so !new_valid_dev check is not
> > needed, remove it.
> >
This patch makes lock_is_held return bool to improve
readability due to this particular function only using either
one or zero as its return value.
No functional change.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
include/linux/lockdep.h | 2 +-
kernel/locking/lockdep.
As new_valid_dev always returns 1, so !new_valid_dev check is not
needed, remove it.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
fs/jfs/namei.c | 3 ---
fs/jfs/super.c | 3 ---
2 files changed, 6 deletions(-)
diff --git a/fs/jfs/namei.c b/fs/jfs/namei.c
index 35976bd..9d7551f
As new_valid_dev always returns 1, so !new_valid_dev check is not
needed, remove it.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
fs/hpfs/namei.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/fs/hpfs/namei.c b/fs/hpfs/namei.c
index 9e92c9c..ae4d5a1 100644
--- a/fs/hpfs/namei.c
As new_valid_dev always returns 1, so !new_valid_dev check is not
needed, remove it.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
fs/nilfs2/namei.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/fs/nilfs2/namei.c b/fs/nilfs2/namei.c
index 37dd6b0..c9a1a49 100644
--- a/fs/
As new_valid_dev always returns 1, so !new_valid_dev check is not
needed, remove it.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
fs/nfs/dir.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/fs/nfs/dir.c b/fs/nfs/dir.c
index 3d8e4ff..ce5a218 100644
--- a/fs/nfs/dir.c
+++ b/
As new_valid_dev always returns 1, so !new_valid_dev check is not
needed, remove it.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
fs/ext2/namei.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/fs/ext2/namei.c b/fs/ext2/namei.c
index b4841e3..3267a80 100644
--- a/fs/ext2/n
As new_valid_dev always returns 1, so new_valid_dev check is not
needed, remove it.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
fs/ubifs/dir.c | 3 ---
fs/ubifs/misc.h | 9 ++---
2 files changed, 2 insertions(+), 10 deletions(-)
diff --git a/fs/ubifs/dir.c b/fs/ubifs/dir.c
As new_valid_dev always returns 1, so !new_valid_dev check is not
needed, remove it.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
fs/ncpfs/dir.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/fs/ncpfs/dir.c b/fs/ncpfs/dir.c
index 93575e9..2d9891a 100644
--- a/fs/ncpfs/dir.c
++
As new_valid_dev always returns 1, so !new_valid_dev check is not
needed, remove it.
Signed-off-by: Yaowei Bai <bywxiao...@163.com>
---
fs/stat.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/fs/stat.c b/fs/stat.c
index 1aa..d4a61d8 100644
--- a/fs/stat.c
+++ b/fs/stat.c
@@
1 - 100 of 391 matches
Mail list logo