* Re: [PATCH] hugetlb:Make the function vma_shareable bool [not found] <1435281568-22312-1-git-send-email-xerofoify@gmail.com> @ 2015-06-26 4:18 ` Mike Kravetz 2015-06-26 5:09 ` Naoya Horiguchi 2015-06-30 22:12 ` David Rientjes 2 siblings, 0 replies; 4+ messages in thread From: Mike Kravetz @ 2015-06-26 4:18 UTC (permalink / raw) To: Nicholas Krause, akpm Cc: n-horiguchi, rientjes, dave, lcapitulino, linux-mm, linux-kernel On 06/25/2015 06:19 PM, Nicholas Krause wrote: > This makes the function vma_shareable bool now due to this > particular function only ever returning either one or zero > as its return value. > > Signed-off-by: Nicholas Krause <xerofoify@gmail.com> > --- > mm/hugetlb.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/mm/hugetlb.c b/mm/hugetlb.c > index 75c0eef..c1be0d1 100644 > --- a/mm/hugetlb.c > +++ b/mm/hugetlb.c > @@ -3789,7 +3789,7 @@ static unsigned long page_table_shareable(struct vm_area_struct *svma, > return saddr; > } > > -static int vma_shareable(struct vm_area_struct *vma, unsigned long addr) > +static bool vma_shareable(struct vm_area_struct *vma, unsigned long addr) > { > unsigned long base = addr & PUD_MASK; > unsigned long end = base + PUD_SIZE; > @@ -3799,8 +3799,8 @@ static int vma_shareable(struct vm_area_struct *vma, unsigned long addr) > */ > if (vma->vm_flags & VM_MAYSHARE && > vma->vm_start <= base && end <= vma->vm_end) > - return 1; > - return 0; > + return true; > + return false; > } > > /* > I was looking at the page table sharing code just last week. Acked-by: Mike Kravetz <mike.kravetz@oracle.com> -- Mike Kravetz ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] hugetlb:Make the function vma_shareable bool [not found] <1435281568-22312-1-git-send-email-xerofoify@gmail.com> 2015-06-26 4:18 ` [PATCH] hugetlb:Make the function vma_shareable bool Mike Kravetz @ 2015-06-26 5:09 ` Naoya Horiguchi 2015-06-30 22:12 ` David Rientjes 2 siblings, 0 replies; 4+ messages in thread From: Naoya Horiguchi @ 2015-06-26 5:09 UTC (permalink / raw) To: Nicholas Krause Cc: akpm@linux-foundation.org, rientjes@google.com, dave@stgolabs.net, mike.kravetz@oracle.com, lcapitulino@redhat.com, linux-mm@kvack.or, linux-kernel@vger.kernel.org On Thu, Jun 25, 2015 at 09:19:28PM -0400, Nicholas Krause wrote: > This makes the function vma_shareable bool now due to this > particular function only ever returning either one or zero > as its return value. > > Signed-off-by: Nicholas Krause <xerofoify@gmail.com> Acked-by: Naoya Horiguchi <n-horiguchi@ah.jp.nec.com> ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] hugetlb:Make the function vma_shareable bool [not found] <1435281568-22312-1-git-send-email-xerofoify@gmail.com> 2015-06-26 4:18 ` [PATCH] hugetlb:Make the function vma_shareable bool Mike Kravetz 2015-06-26 5:09 ` Naoya Horiguchi @ 2015-06-30 22:12 ` David Rientjes [not found] ` <55949A95.3020506@gmail.com> 2 siblings, 1 reply; 4+ messages in thread From: David Rientjes @ 2015-06-30 22:12 UTC (permalink / raw) To: Nicholas Krause Cc: akpm, n-horiguchi, dave, mike.kravetz, lcapitulino, linux-mm, linux-kernel On Thu, 25 Jun 2015, Nicholas Krause wrote: > This makes the function vma_shareable bool now due to this > particular function only ever returning either one or zero > as its return value. > > Signed-off-by: Nicholas Krause <xerofoify@gmail.com> Acked-by: David Rientjes <rientjes@google.com> ^ permalink raw reply [flat|nested] 4+ messages in thread
[parent not found: <55949A95.3020506@gmail.com>]
* Re: [PATCH] hugetlb:Make the function vma_shareable bool [not found] ` <55949A95.3020506@gmail.com> @ 2015-07-08 23:51 ` David Rientjes 0 siblings, 0 replies; 4+ messages in thread From: David Rientjes @ 2015-07-08 23:51 UTC (permalink / raw) To: nick Cc: akpm, n-horiguchi, dave, mike.kravetz, lcapitulino, linux-mm, linux-kernel On Wed, 1 Jul 2015, nick wrote: > David, > This was already acknowledged, however I am over 150 different patches lying > around on my system for various fixes/cleanups in many different subsystems > and based on my rep people seem to be just ignoring them. If someone looks > at them first they should go through. Below is a patch for fixing a bug in > the Intel network drivers. ./scripts/get_maintainer.pl suggests this should go to Jeff Kirsher <jeffrey.t.kirsher@intel.com> for review. It looks like a cleanup that would only suppress output to the kernel log, so might be better to just make this function void. You might also have luck with the "TRIVIAL PATCHES" section of ./MAINTAINERS for things like this. > Nick > From 950ace789c2cc371233ea0204e9baccfea7630bc Mon Sep 17 00:00:00 2001 > From: Nicholas Krause <xerofoify@gmail.com> > Date: Mon, 29 Jun 2015 22:55:22 -0400 > Subject: [PATCH] i40e:Return -ENODEV if hardware has no support for certain > required features in the function i40e_init_pf_fcoe > > This makes the function i40e_init_pf_fcoe correctly return -ENODEV > as a error return for when the underlying hardware does not have > support for FCOE or DCB rather then 0 to it's callers in order > to comply with checks run by these particular function's callers > to find out if the underlying hardware supports these features. > > Signed-off-by: Nicholas Krause <xerofoify@gmail.com> > --- > drivers/net/ethernet/intel/i40e/i40e_fcoe.c | 7 ++++--- > 1 file changed, 4 insertions(+), 3 deletions(-) > > diff --git a/drivers/net/ethernet/intel/i40e/i40e_fcoe.c b/drivers/net/ethernet/intel/i40e/i40e_fcoe.c > index c8b621e..24a44dd 100644 > --- a/drivers/net/ethernet/intel/i40e/i40e_fcoe.c > +++ b/drivers/net/ethernet/intel/i40e/i40e_fcoe.c > @@ -273,7 +273,8 @@ out: > * i40e_fcoe_sw_init - sets up the HW for FCoE > * @pf: pointer to PF > * > - * Returns 0 if FCoE is supported otherwise the error code > + * Returns 0 if FCoE is supported otherwise -ENODEV if > + * the hardware does not support it > **/ > int i40e_init_pf_fcoe(struct i40e_pf *pf) > { > @@ -287,13 +288,13 @@ int i40e_init_pf_fcoe(struct i40e_pf *pf) > > if (!pf->hw.func_caps.fcoe) { > dev_info(&pf->pdev->dev, "FCoE capability is disabled\n"); > - return 0; > + return -ENODEV; > } > > if (!pf->hw.func_caps.dcb) { > dev_warn(&pf->pdev->dev, > "Hardware is not DCB capable not enabling FCoE.\n"); > - return 0; > + return -ENODEV; > } > > /* enable FCoE hash filter */ ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-07-08 23:51 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1435281568-22312-1-git-send-email-xerofoify@gmail.com>
2015-06-26 4:18 ` [PATCH] hugetlb:Make the function vma_shareable bool Mike Kravetz
2015-06-26 5:09 ` Naoya Horiguchi
2015-06-30 22:12 ` David Rientjes
[not found] ` <55949A95.3020506@gmail.com>
2015-07-08 23:51 ` David Rientjes
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.