[PATCH] staging: sm750fb: Remove typedefs from enums

2018-02-11 Thread Christian Luetke-Stetzkamp
Fixes checkpatch.pl warning: do not add new typedefs. Signed-off-by: Christian Luetke-Stetzkamp --- drivers/staging/sm750fb/ddk750_chip.c | 4 ++-- drivers/staging/sm750fb/ddk750_chip.h | 14 ++ drivers/staging/sm750fb/ddk750_mode.c | 2 +-

Re: [PATCH] staging: lustre: llite: replace variable length array

2018-02-11 Thread James Simmons
> On 01/30/2018 03:04 AM, Dilger, Andreas wrote: > > On Jan 27, 2018, at 14:42, Sven Dziadek wrote: > >> > >> The functionality of the removed variable length array is already > >> implemented by the function xattr_full_name in fs/xattr.c > >> > >> This fixes the sparse

Re: [PATCH] staging: lustre: lnet: return of an error code should be negative

2018-02-11 Thread James Simmons
> Return value of error codes should typically be negative. > Issue reported by checkpatch.pl > Reviewed-by: James Simmons > Signed-off-by: Sumit Pundir > --- > drivers/staging/lustre/lnet/selftest/framework.c | 2 +- > 1 file changed, 1

RE: [PATCH char-misc 1/1] Drivers: hv: vmbus: Fix ring buffer signaling

2018-02-11 Thread Michael Kelley (EOSG)
> -Original Message- > From: KY Srinivasan > Sent: Sunday, February 11, 2018 5:14 PM --- snip --- > > if (rbi->ring_buffer->feature_bits.feat_pending_send_sz) { > > u32 pending_sz = READ_ONCE(rbi->ring_buffer- > > >pending_send_sz); > > > > /* > > +

Re: [PATCH char-misc 1/1] Drivers: hv: vmbus: Fix ring buffer signaling

2018-02-11 Thread Stephen Hemminger
On Sat, 10 Feb 2018 20:48:49 + Michael Kelley wrote: > + u32 delta = rbi->ring_buffer->read_index < rbi->priv_read_index ? > + (rbi->priv_read_index - rbi->ring_buffer->read_index) : > + (rbi->ring_datasize -

Re: [lustre-devel] [PATCH 41/80] staging: lustre: lmv: separate master object with master stripe

2018-02-11 Thread NeilBrown
On Sat, Feb 10 2018, James Simmons wrote: >> > On Feb 8, 2018, at 10:10 PM, NeilBrown wrote: >> > >> > On Thu, Feb 08 2018, Oleg Drokin wrote: >> > >> >>> On Feb 8, 2018, at 8:39 PM, NeilBrown wrote: >> >>> >> >>> On Tue, Aug 16 2016, James Simmons wrote: >>

[PATCH 00/12] Drivers: hv: Miscellaneous fixes

2018-02-11 Thread kys
From: "K. Y. Srinivasan" Miscellaneous fixes. Dexuan Cui (1): tools: hv: fix compiler warnings about major/target_fname Haiyang Zhang (2): tools/hv: Fix IP reporting by KVP daemon with SRIOV hv_vmbus: Correct the stale comments regarding cpu affinity Jia-Ju Bai (1):

Re: [PATCH] staging: lustre: update the TODO list

2018-02-11 Thread Joe Perches
On Sun, 2018-02-11 at 18:00 -0500, James Simmons wrote: > As more people become involved with the progression of the lustre > client it needs to more clear what needs to be done to leave > staging. Update the TODO list with the various bugs and changes > to accomplish this. Some are simple bugs

[PATCH 12/12] hv_balloon: trace post_status

2018-02-11 Thread kys
From: Vitaly Kuznetsov Hyper-V balloon driver makes non-trivial calculations to convert Linux's representation of free/used memory to what Hyper-V host expects to see. Add a tracepoint to see what's being sent and where the data comes from. Signed-off-by: Vitaly Kuznetsov

Re: [lustre-devel] [PATCH 41/80] staging: lustre: lmv: separate master object with master stripe

2018-02-11 Thread Oleg Drokin
> On Feb 11, 2018, at 6:44 PM, NeilBrown wrote: > > On Thu, Feb 08 2018, Oleg Drokin wrote: >> >> Certain things that sound useless (like the debug subsystem in Lustre) >> is very useful when you have a 10k nodes in a cluster and need to selectively >> pull stuff from a run to

[PATCH] staging: lustre: update the TODO list

2018-02-11 Thread James Simmons
As more people become involved with the progression of the lustre client it needs to more clear what needs to be done to leave staging. Update the TODO list with the various bugs and changes to accomplish this. Some are simple bugs and others are far more complex task that will change many lines

Re: [lustre-devel] [PATCH 41/80] staging: lustre: lmv: separate master object with master stripe

2018-02-11 Thread Oleg Drokin
> On Feb 11, 2018, at 6:50 PM, NeilBrown wrote: > > Maybe - as you suggest in another email - it is due to some > client/server incompatibility. I guess it is unavoidable with an fs > like lustre to have incompatible protocol changes. Is there any > mechanism for detecting the

[PATCH 10/12] hv_balloon: simplify hv_online_page()/hv_page_online_one()

2018-02-11 Thread kys
From: Vitaly Kuznetsov Instead of doing pfn_to_page() and continuosly casting page to unsigned long just cache the pfn of the page with page_to_pfn(). Signed-off-by: Vitaly Kuznetsov Signed-off-by: K. Y. Srinivasan ---

[PATCH 06/12] vmbus/ring_buffer: remove some redundant helper function.

2018-02-11 Thread kys
From: "lantianyu1...@gmail.com" Some hv_get/set** helper functions in ring_buffer code are only called once or not used. This patch is to clear up these codes. Signed-off-by: Tianyu Lan Signed-off-by: K. Y. Srinivasan ---

[PATCH 05/12] tools: hv: include string.h in hv_fcopy_daemon

2018-02-11 Thread kys
From: Olaf Hering The usage of strchr requires inclusion of string.h. Fixes: 0c38cda64aec ("tools: hv: remove unnecessary header files and netlink related code") Signed-off-by: Olaf Hering Signed-off-by: K. Y. Srinivasan ---

[PATCH 02/12] hyper-v: use GFP_KERNEL for hv_context.hv_numa_map

2018-02-11 Thread kys
From: Jia-Ju Bai The kzalloc function is called with GFP_ATOMIC. But according to driver call graph, it is not in atomic context, namely no spinlock is held nor in an interrupt handler. This GFP_ATOMIC is unnecessary, and replace with GFP_KERNEL. Signed-off-by: Jia-Ju

[PATCH 03/12] hv: Synthetic typo correction

2018-02-11 Thread kys
From: Joe Perches Just a trivial tyop fix. Signed-off-by: Joe Perches Signed-off-by: K. Y. Srinivasan --- drivers/hv/hv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/hv/hv.c b/drivers/hv/hv.c index

[PATCH 07/12] hv_vmbus: Correct the stale comments regarding cpu affinity

2018-02-11 Thread kys
From: Haiyang Zhang The comments doesn't match what the current code does, also have a typo. This patch corrects them. Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/hv/channel_mgmt.c | 6 ++

[PATCH 08/12] Drivers: hv: vmbus: Implement Direct Mode for stimer0

2018-02-11 Thread kys
From: Michael Kelley The 2016 version of Hyper-V offers the option to operate the guest VM per-vcpu stimer's in Direct Mode, which means the timer interupts on its own vector rather than queueing a VMbus message. Direct Mode reduces timer processing overhead in both the

[PATCH 01/12] tools/hv: Fix IP reporting by KVP daemon with SRIOV

2018-02-11 Thread kys
From: Haiyang Zhang On Hyper-V the VF NIC has the same MAC as the related synthetic NIC. VF NIC can work under the synthetic NIC transparently, without its own IP address. The existing KVP daemon only gets IP from the first NIC matching a MAC address, and may not be able

[PATCH 09/12] hv_balloon: fix printk loglevel

2018-02-11 Thread kys
From: Vitaly Kuznetsov We have a mix of different ideas of which loglevel should be used. Unify on the following: - pr_info() for normal operation - pr_warn() for 'strange' host behavior - pr_err() for all errors. Signed-off-by: Vitaly Kuznetsov

[PATCH 04/12] tools: hv: fix compiler warnings about major/target_fname

2018-02-11 Thread kys
From: Dexuan Cui This patch fixes the below warnings with new glibc and gcc: hv_vss_daemon.c:100:13: warning: In the GNU C Library, "major" is defined by . For historical compatibility, it is currently defined by as well, but we plan to remove this soon. To use "major",

RE: [PATCH char-misc 1/1] Drivers: hv: vmbus: Fix ring buffer signaling

2018-02-11 Thread KY Srinivasan
> -Original Message- > From: Michael Kelley [mailto:mhkel...@outlook.com] > Sent: Saturday, February 10, 2018 12:49 PM > To: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; > de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; > vkuzn...@redhat.com;

Re: [PATCH] staging: android: ion: Initialize dma_address of new sg list

2018-02-11 Thread Dan Carpenter
On Fri, Feb 09, 2018 at 10:16:56PM -0800, Liam Mark wrote: > Fix the dup_sg_table function to initialize the dma_address of the new > sg list entries instead of the source dma_address entries. > > Fixes: 17fd283f3870 ("staging: android: ion: Duplicate sg_table") > Signed-off-by: Liam Mark

Re: [PATCH] staging: vt6656: Fix "Possible unnecessary 'out of memory' message" checkpatch.pl warning

2018-02-11 Thread Dan Carpenter
On Sat, Feb 10, 2018 at 09:46:18PM +0530, Dileep Sankhla wrote: > Signed-off-by: Dileep Sankhla The subject is too long and you need to have a changelog. regards, dan carpenter ___ devel mailing list

Re: [PATCH 41/80] staging: lustre: lmv: separate master object with master stripe

2018-02-11 Thread Dan Carpenter
On Fri, Feb 09, 2018 at 12:39:18PM +1100, NeilBrown wrote: > On Tue, Aug 16 2016, James Simmons wrote: > > > > > +static inline bool > > +lsm_md_eq(const struct lmv_stripe_md *lsm1, const struct lmv_stripe_md > > *lsm2) > > +{ > > + int idx; > > + > > + if (lsm1->lsm_md_magic !=

Re: [PATCH] staging: lustre: update the TODO list

2018-02-11 Thread Joe Perches
On Mon, 2018-02-12 at 02:06 +, James Simmons wrote: > > On Sun, 2018-02-11 at 18:00 -0500, James Simmons wrote: > > > As more people become involved with the progression of the lustre > > > client it needs to more clear what needs to be done to leave > > > staging. Update the TODO list with

Re: [PATCH] staging: lustre: update the TODO list

2018-02-11 Thread James Simmons
> On Sun, 2018-02-11 at 18:00 -0500, James Simmons wrote: > > As more people become involved with the progression of the lustre > > client it needs to more clear what needs to be done to leave > > staging. Update the TODO list with the various bugs and changes > > to accomplish this. Some are