Re: [PATCH v2] cxl: Fix error path on bad ioctl

Hi Fred, Thanks for addressing the review comments. Frederic Barrat writes: > Fix error path if we can't copy user structure on CXL_IOCTL_START_WORK > ioctl. We shouldn't unlock the context status mutex as it was not > locked (yet). > > Signed-off-by: Frederic

[RESEND PATCH] input/joystick/analog: Use get_cycles() on PPC

The analog joystick driver spits a warning at us: drivers/input/joystick/analog.c:176:2: warning: #warning Precise timer not defined for this architecture. PPC has get_cycles() so use that. Signed-off-by: Michael Ellerman --- drivers/input/joystick/analog.c | 2 +- 1

Re: [PATCH] include/linux/vfio.h: Guard powerpc-specific functions with CONFIG_VFIO_SPAPR_EEH

Hi, How did you manage to have CONFIG_EEH=y and CONFIG_VFIO_SPAPR_EEH=n? "make oldconfig" fixes this to CONFIG_VFIO_SPAPR_EEH=y. On 08/06/17 02:31, kbuild test robot wrote: > Hi Murilo, > > [auto build test ERROR on linus/master] > [also build test ERROR on v4.12-rc4 nex

Re: [PATCH] include/linux/vfio.h: Guard powerpc-specific functions with CONFIG_VFIO_SPAPR_EEH

e error below, what am I missing here? > > > > On 08/06/17 02:31, kbuild test robot wrote: >> Hi Murilo, >> >> [auto build test ERROR on linus/master] >> [also build test ERROR on v4.12-rc4 next-20170607] >> [if your patch is applied to the wrong

Re: [PATCH v2] cxl: Fix error path on bad ioctl

Reviewed-by: Andrew Donnellan On 06/06/17 19:43, Frederic Barrat wrote: Fix error path if we can't copy user structure on CXL_IOCTL_START_WORK ioctl. We shouldn't unlock the context status mutex as it was not locked (yet). Signed-off-by: Frederic Barrat

Re: 4.12-rc ppc64 4k-page needs costly allocations

Christoph Lameter writes: > On Thu, 1 Jun 2017, Hugh Dickins wrote: > >> Thanks a lot for working that out. Makes sense, fully understood now, >> nothing to worry about (though makes one wonder whether it's efficient >> to use ctors on high-alignment caches; or whether an

Re: 4.12-rc ppc64 4k-page needs costly allocations

Hugh Dickins writes: > On Fri, 2 Jun 2017, Michael Ellerman wrote: >> Hugh Dickins writes: >> > On Thu, 1 Jun 2017, Christoph Lameter wrote: >> >> >> >> Ok so debugging was off but the slab cache has a ctor callback which >> >> mandates that the free pointer

Re: [PATCH] macintosh: move mac_hid driver to input/mouse.

On Sun, 28 May 2017 10:55:40 -0700 Dmitry Torokhov wrote: > On Sun, May 28, 2017 at 11:47:58AM +0200, Michal Suchanek wrote: > > On Tue, 9 May 2017 17:43:27 -0700 > > Dmitry Torokhov wrote: > > > > > Hi Michal, > > > > > > On Tue, May

Re: power server power consumption problem

* ?? <1363017...@qq.com> [2017-06-06 19:59:24]: > I use ipmitool measuring power server power consumption, getting the > following results without total power consumption as the following picture, > do I add all power values as total power consumption? I am eager to receive > someone's

Re: [PATCH] include/linux/vfio.h: Guard powerpc-specific functions with CONFIG_VFIO_SPAPR_EEH

Hi Murilo, [auto build test ERROR on linus/master] [also build test ERROR on v4.12-rc4 next-20170607] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Murilo-Opsfelder-Araujo/include-linux-vfio-h

RESEND Re: [Patch 2/2]: powerpc/hotplug/mm: Fix hot-add memory node assoc

Here is the information from 2 different kernels. I have not been able to retrieve the information matching yesterday's attachments, yet, as those dumps were acquired in April. Attached please find 2 dumps of similar material from kernels running with my current patches (Linux 4.4, Linux

Re: [kernel-hardening] [PATCH] powerpc: Increase ELF_ET_DYN_BASE to 1TB for 64-bit applications

On Wed, Jun 7, 2017 at 2:29 AM, Michael Ellerman wrote: > Daniel Micay writes: > >> Rather than doing this, the base should just be split for an ELF >> interpreter like PaX. > > I don't quite parse that, I think you mean PaX uses a different base for >

Re: [PATCH] macintosh: move mac_hid driver to input/mouse.

On Wed, Jun 07, 2017 at 06:53:51PM +0200, Michal Suchánek wrote: > On Sun, 28 May 2017 10:55:40 -0700 > Dmitry Torokhov wrote: > > > On Sun, May 28, 2017 at 11:47:58AM +0200, Michal Suchanek wrote: > > > On Tue, 9 May 2017 17:43:27 -0700 > > > Dmitry Torokhov

Re: [PATCH 12/17] tty: New RISC-V SBI Console Driver

CC (hypervisor) console folks On Wed, Jun 7, 2017 at 1:00 AM, Palmer Dabbelt wrote: > This patch adds a new driver for the console availiable via the RISC-V > SBI. This console is specified to be used for early boot messages, and > is designed to be a very simple (albiet

Re: [PATCH] powernv: Properly mask POWER9 DD1 PVR for different chip types

Michael Neuling writes: > Bits 48:51 in the PVR for POWER9 represent different chip types (scale > up vs out and 12 vs 24 core). Current chips have 0 here, but could be > non-zero in the future. > > This changes the POWER9 DD1 mask to correctly ignore these bits 48:51. > >

Re: [PATCH] PCI: dwc: Constify dw_pcie_host_ops structures

On Monday 05 June 2017 02:23 PM, Jisheng Zhang wrote: > The dw_pcie_host_ops structures are never modified. Constify these > structures such that these can be write-protected. > > Signed-off-by: Jisheng Zhang Acked-by: Kishon Vijay Abraham I > --- >

Re: [kernel-hardening] [PATCH] powerpc: Increase ELF_ET_DYN_BASE to 1TB for 64-bit applications

Daniel Micay writes: > Rather than doing this, the base should just be split for an ELF > interpreter like PaX. I don't quite parse that, I think you mean PaX uses a different base for an ELF interpreter vs a regular ET_DYN? That would be cool. How do you know that it's

Re: [PATCH 12/17] tty: New RISC-V SBI Console Driver

On Wed, Jun 7, 2017 at 9:15 AM, Geert Uytterhoeven wrote: > CC (hypervisor) console folks > > On Wed, Jun 7, 2017 at 1:00 AM, Palmer Dabbelt wrote: >> This patch adds a new driver for the console availiable via the RISC-V >> SBI. This console is

Re: [PATCH] driver core: remove CLASS_ATTR usage

On Wed, Jun 07, 2017 at 11:40:12AM +1000, Michael Ellerman wrote: > Greg KH writes: > > > From: Greg Kroah-Hartman > > > > There was only 2 remaining users of CLASS_ATTR() so let's finally get > > rid of them and force everyone to use the

[PATCH 10/16 v2] powerpc: vio: use dev_groups and not dev_attrs for bus_type

From: Greg Kroah-Hartman The dev_attrs field has long been "depreciated" and is finally being removed, so move the driver to use the "correct" dev_groups field instead for struct bus_type. Cc: Benjamin Herrenschmidt Cc: Paul Mackerras

Re: [Patch 2/2]: powerpc/hotplug/mm: Fix hot-add memory node assoc

On Thu, 2017-06-01 at 16:33 -0500, Reza Arbab wrote: > On Thu, Jun 01, 2017 at 07:36:31PM +1000, Michael Ellerman wrote: > > I don't think that's what the patch does. It just marks 32 (!?) nodes > > as online. Or if you're talking about reverting 3af229f2071f that > > leaves you with 256

Re: [PATCH] macintosh: move mac_hid driver to input/mouse.

On Wed, 7 Jun 2017 10:16:22 -0700 Dmitry Torokhov wrote: > On Wed, Jun 07, 2017 at 06:53:51PM +0200, Michal Suchánek wrote: > > On Sun, 28 May 2017 10:55:40 -0700 > > Dmitry Torokhov wrote: > > > > > On Sun, May 28, 2017 at 11:47:58AM

Re: [kernel-hardening] [PATCH] powerpc: Increase ELF_ET_DYN_BASE to 1TB for 64-bit applications

On Wed, Jun 7, 2017 at 2:59 PM, Michael Ellerman wrote: > Daniel Micay writes: > >> Rather than doing this, the base should just be split for an ELF >> interpreter like PaX. > > I don't quite parse that, I think you mean PaX uses a different base for >

Re: [PATCH 08/16] powerpc: ps3: use dev_groups and not dev_attrs for bus_type

On Tue, Jun 06, 2017 at 09:22:13PM +0200, Greg Kroah-Hartman wrote: > The dev_attrs field has long been "depreciated" and is finally being > removed, so move the driver to use the "correct" dev_groups field > instead for struct bus_type. > > Cc: Geoff Levand > Cc: Benjamin

Re: [PATCH v2 1/9] powerpc/lib/code-patching: Enhance code patching

On Wed, Jun 7, 2017 at 8:25 PM, Christophe LEROY wrote: > Hi Balbir > > Le 04/06/2017 à 01:45, kbuild test robot a écrit : >> >> Hi Balbir, >> >> [auto build test ERROR on powerpc/next] >> [also build test ERROR on v4.12-rc3 next-20170602] >> [if your patch is applied to

Re: [PATCH v3 1/3] EDAC: mv64x60: check driver registration success

On Wed, Jun 07, 2017 at 05:29:55AM +, Chris Packham wrote: > What do you think? Just send me a fix ontop, moving the edac_op_state assignment before the platform_register_drivers() call and write in the commit message why we're doing that. Thanks. -- Regards/Gruss, Boris. Good mailing

Re: [PATCH v2 1/9] powerpc/lib/code-patching: Enhance code patching

Hi Balbir Le 04/06/2017 à 01:45, kbuild test robot a écrit : Hi Balbir, [auto build test ERROR on powerpc/next] [also build test ERROR on v4.12-rc3 next-20170602] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH v9 07/10] powerpc/perf: PMU functions for Core IMC and hotplugging

On Wednesday 07 June 2017 11:14 AM, Anju T Sudhakar wrote: Hi Thomas, On Tuesday 06 June 2017 03:39 PM, Thomas Gleixner wrote: On Mon, 5 Jun 2017, Anju T Sudhakar wrote: +static void cleanup_all_core_imc_memory(struct imc_pmu *pmu_ptr) +{ +struct imc_mem_info *ptr = pmu_ptr->mem_info; +

Re: [PATCH] include/linux/vfio.h: Guard powerpc-specific functions with CONFIG_VFIO_SPAPR_EEH

Murilo Opsfelder Araujo writes: > When CONFIG_EEH=y and CONFIG_VFIO_SPAPR_EEH=n, build fails with the > following: > > drivers/vfio/pci/vfio_pci.o: In function `.vfio_pci_release': > vfio_pci.c:(.text+0xa98): undefined reference to > `.vfio_spapr_pci_eeh_release' >

Re: [Patch 2/2]: powerpc/hotplug/mm: Fix hot-add memory node assoc

Here is the information from 2 different kernels.  I have not been able to retrieve the information matching yesterday's attachments, yet, as those dumps were acquired in April.   Attached please find 2 dumps of similar material from kernels running with my current patches (Linux 4.4, Linux 4.12).

Re: [PATCH v3 6/6] dt-bindings: net: btusb: DT fix s/interrupt-name/interrupt-names/

On Fri, Jun 02, 2017 at 02:38:49PM +0200, Geert Uytterhoeven wrote: > Fixes: fd913ef7ce619467 ("Bluetooth: btusb: Add out-of-band wakeup support") > Signed-off-by: Geert Uytterhoeven > --- > v3: > - New. > --- > Documentation/devicetree/bindings/net/btusb.txt | 2 +- >

Re: power server power consumption problem

>From a peer.. Rafael Peria de Sene... At Unicamp we developed a power monitor based on IPMI output to monitor our Minsky usage. Take a look at http://177.220.10.134/powergraph On Wed, Jun 7, 2017 at 11:00 AM, Vaidyanathan Srinivasan < sva...@linux.vnet.ibm.com> wrote: > * ??

Re: [PATCH v3 5/6] dt-bindings: display: amlogic: DT fix s/resets-names/reset-names/

On Fri, Jun 02, 2017 at 02:38:48PM +0200, Geert Uytterhoeven wrote: > Fixes: 8cceda5349377e30 ("dt-bindings: Add bindings for the Amlogic Meson > dw-hdmi extension") > Signed-off-by: Geert Uytterhoeven > --- > v3: > - New. > --- >

[PATCH v2] include/linux/vfio.h: Guard powerpc-specific functions with CONFIG_VFIO_SPAPR_EEH

When CONFIG_EEH=y and CONFIG_VFIO_SPAPR_EEH=n, build fails with the following: drivers/vfio/pci/vfio_pci.o: In function `.vfio_pci_release': vfio_pci.c:(.text+0xa98): undefined reference to `.vfio_spapr_pci_eeh_release' drivers/vfio/pci/vfio_pci.o: In function `.vfio_pci_open':

[PATCH 3.10 078/250] powerpc: Fix build warning on 32-bit PPC

From: Larry Finger commit 8ae679c4bc2ea2d16d92620da8e3e9332fa4039f upstream. I am getting the following warning when I build kernel 4.9-git on my PowerBook G4 with a 32-bit PPC processor: AS arch/powerpc/kernel/misc_32.o

Re: [PATCH] macintosh: move mac_hid driver to input/mouse.

On Wed, Jun 07, 2017 at 09:17:37PM +0200, Michal Suchánek wrote: > On Wed, 7 Jun 2017 10:16:22 -0700 > Dmitry Torokhov wrote: > > > On Wed, Jun 07, 2017 at 06:53:51PM +0200, Michal Suchánek wrote: > > > On Sun, 28 May 2017 10:55:40 -0700 > > > Dmitry Torokhov

Re: [PATCH] cxl: Fix error path on bad ioctl

Frederic Barrat writes: > Le 06/06/2017 à 11:20, Michael Ellerman a écrit : >> Frederic Barrat writes: >> >>> Fix error path if we can't copy user structure on >>> CXL_IOCTL_START_WORK ioctl. >> >> To be clear the error is that returning

Re: [Patch 2/2]: powerpc/hotplug/mm: Fix hot-add memory node assoc

Michael Bringmann writes: > On 06/06/2017 04:48 AM, Michael Ellerman wrote: >> Michael Bringmann writes: >>> On 06/01/2017 04:36 AM, Michael Ellerman wrote: Do you actually see mention of nodes 0 and 8 in the dmesg? >>> >>> When the

Re: [Patch 2/2]: powerpc/hotplug/mm: Fix hot-add memory node assoc

Reza Arbab writes: > On Thu, Jun 01, 2017 at 07:36:31PM +1000, Michael Ellerman wrote: >>I don't think that's what the patch does. It just marks 32 (!?) nodes >>as online. Or if you're talking about reverting 3af229f2071f that >>leaves you with 256 possible nodes.

Re: [PATCH] powerpc/numa: Fix percpu allocations to be NUMA aware

Balbir Singh writes: > On Fri, Jun 2, 2017 at 3:14 PM, Michael Ellerman wrote: >> diff --git a/arch/powerpc/include/asm/topology.h >> b/arch/powerpc/include/asm/topology.h >> index 8b3b46b7b0f2..8f3b2ec09b9e 100644 >> ---

Re: [PATCH v2 1/9] powerpc/lib/code-patching: Enhance code patching

Balbir Singh writes: > On Wed, Jun 7, 2017 at 8:25 PM, Christophe LEROY > wrote: >> Le 04/06/2017 à 01:45, kbuild test robot a écrit : >>> [auto build test ERROR on powerpc/next] >>> [also build test ERROR on v4.12-rc3 next-20170602] >>> [if your

Re: [PATCH] powernv: Properly mask POWER9 DD1 PVR for different chip types

On Wed, 2017-06-07 at 17:24 +1000, Michael Ellerman wrote: > Michael Neuling writes: > > > Bits 48:51 in the PVR for POWER9 represent different chip types (scale > > up vs out and 12 vs 24 core). Current chips have 0 here, but could be > > non-zero in the future. > > > > This

[PATCH v2 2/6] ima: Simplify policy_func_show.

If the func_tokens array uses the same indices as enum ima_hooks, policy_func_show can be a lot simpler, and the func_* enum becomes unnecessary. Also, if we use the same macro trick used by kernel_read_file_id_str we can use one hooks list for both the enum and the string array, making sure they

[PATCH v2 6/6] ima: Support module-style appended signatures for appraisal

This patch introduces the modsig keyword to the IMA policy syntax to specify that a given hook should expect the file to have the IMA signature appended to it. Here is how it can be used in a rule: appraise func=KEXEC_KERNEL_CHECK appraise_type=modsig|imasig With this rule, IMA will accept

[PATCH v2 5/6] MODSIGN: Export module signature definitions.

IMA will use the module_signature format for append signatures, so export the relevant definitions and factor out the code which verifies that the appended signature trailer is valid. Also, add function verify_pkcs7_message_signature which takes a struct pkcs7_message for verification isntead of

[PATCH v2 1/6] integrity: Small code improvements

These changes are too small to warrant their own patches: The keyid and sig_size members of struct signature_v2_hdr are in BE format, so use a type that makes this assumption explicit. Also, use beXX_to_cpu instead of __beXX_to_cpu to read them. Change integrity_kernel_read to take a void *

Re: [RFC 2/2] powerpc/powernv : Add support for OPAL-OCC command/response interface

On Mon, 2017-06-05 at 11:43 +0530, Shilpasri G Bhat wrote: > In P9, OCC (On-Chip-Controller) supports shared memory based > commad-response interface. Within the shared memory there is an OPAL > command buffer and OCC response buffer that can be used to send > inband commands to OCC. This patch

Re: [PATCH] powerpc/powernv: Rework local TLB flush for boot and MCE on POWER9

Hi Nicholas, [auto build test ERROR on powerpc/next] [also build test ERROR on v4.12-rc4 next-20170607] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Nicholas-Piggin/powerpc-powernv-Rework

[PATCH v2 0/6] Appended signatures support for IMA appraisal

On the OpenPOWER platform, secure boot and trusted boot are being implemented using IMA for taking measurements and verifying signatures. Since the kernel image on Power servers is an ELF binary, kernels are signed using the scripts/sign-file tool and thus use the same signature format as signed

[PATCH v2 4/6] integrity: Introduce struct evm_hmac_xattr

Even though struct evm_ima_xattr_data includes a fixed-size array to hold a SHA1 digest, most of the code ignores the array and uses the struct to mean "type indicator followed by data of unspecified size" and tracks the real size of what the struct represents in a separate length variable. The

[PATCH v2 3/6] ima: Log the same audit cause whenever a file has no signature

If the file doesn't have an xattr, ima_appraise_measurement sets cause to "missing-hash" while if there's an xattr but it's a digest instead of a signature it sets cause to "IMA-signature-required". Fix it by setting cause to "IMA-signature-required" in both cases. Signed-off-by: Thiago Jung

Re: [PATCH] powernv: Properly mask POWER9 DD1 PVR for different chip types

Michael Neuling writes: > On Wed, 2017-06-07 at 17:24 +1000, Michael Ellerman wrote: >> Michael Neuling writes: >> >> > Bits 48:51 in the PVR for POWER9 represent different chip types (scale >> > up vs out and 12 vs 24 core). Current chips have 0 here, but

Re: powerpc/perf: Fix Power9 test_adder field

On Fri, 2017-05-26 at 08:08:27 UTC, Madhavan Srinivasan wrote: > Commit 8d911904f3ce4 ('powerpc/perf: Add restrictions to PMC5 in power9 DD1') > was added to restrict the use of PMC5 in Power9 DD1. Intend is to diable > the use of PMC5 using raw event code. Commit instead of updating >

Re: powerpc/kernel: improve FP and vector registers restoration

On Fri, 2017-06-02 at 21:43:30 UTC, Breno Leitao wrote: > Currently tsk->thread->load_vec and load_fp are not initialized during a > task creation, which set garbage to these variables (non-zero value). > > These variables will be checked later at restore_math() to validate if the > FP and

Re: [v2] cxl: Avoid double free_irq() for psl,slice interrupts

On Fri, 2017-06-02 at 16:56:48 UTC, Vaibhav Jain wrote: > During an eeh call to cxl_remove can result in double free_irq of > psl,slice interrupts. This can happen if perst_reloads_same_image == 1 > and call to cxl_configure_adapter() fails during slot_reset > callback. In such a case we see a

Re: powerpc/kernel: Initialize load_tm on task creation

On Mon, 2017-06-05 at 14:40:59 UTC, Breno Leitao wrote: > Currently tsk->thread.load_tm is not initialized in the task creation > and can contain garbage on a new task. > > This is an undesired behaviour, since it affects the timing to enable > and disable the transactional memory laziness

Re: [v2] powerpc/numa: Fix percpu allocations to be NUMA aware

On Tue, 2017-06-06 at 10:23:57 UTC, Michael Ellerman wrote: > In commit 8c272261194d ("powerpc/numa: Enable USE_PERCPU_NUMA_NODE_ID"), we > switched to the generic implementation of cpu_to_node(), which uses a percpu > variable to hold the NUMA node for each CPU. > > Unfortunately we neglected to

Re: [v3,1/3] powerpc/mm/book(e)(3s)/64: Add page table accounting

On Tue, 2017-05-02 at 05:17:04 UTC, Balbir Singh wrote: > Introduce a helper pgtable_gfp_flags() which > just returns the current gfp flags and adds > __GFP_ACCOUNT to account for page table allocation. > The generic helper is added to include/asm/pgalloc.h > and has two variants - WARNING ugly

Re: cxl: Unlock on error in probe

On Fri, 2017-05-05 at 05:34:58 UTC, Dan Carpenter wrote: > We should unlock if get_cxl_adapter() fails. > > Fixes: 594ff7d067ca ("cxl: Support to flash a new image on the adapter from a > guest") > Signed-off-by: Dan Carpenter > Acked-by: Frederic Barrat

Re: [v2] powerpc/mm: convert tlbie to tlbiel when no batch is active

On Thu, 2017-05-25 at 07:28:32 UTC, Balbir Singh wrote: > Do the checks that __flush_tlb_pending() does and check if > a local flush will do when batch->active is false inside of > hpte_need_flush(). I've checked the changes with tlbie tracing, > I see local flushes as applicable now and I've also

Re: powerpc/64s: machine check handle ifetch from foreign real address for POWER9

On Mon, 2017-05-29 at 06:26:44 UTC, Nicholas Piggin wrote: > The i-side 0111b case was missed by 7b9f71f974 ("powerpc/64s: POWER9 > machine check handler"). > > It is possible to trigger this exception by branching to a foreign real > address (bits [8:12] != 0) with instruction relocation off,

Re: [1/7] powerpc/mm: rename map_page() to map_kernel_page() on PPC32

On Mon, 2017-05-29 at 15:31:56 UTC, Christophe Leroy wrote: > Signed-off-by: Christophe Leroy > Acked-by: Balbir Singh Applied to powerpc next, thanks. https://git.kernel.org/powerpc/c/4386c096c2ffa1b3232d701e9d7ff8 cheers

Re: [1/2] MAINTAINERS: update entry for Freescale SoC drivers

On Fri, 2017-06-02 at 23:20:22 UTC, Yang Li wrote: > Add myself as the maintainer for drivers/fsl/soc/ and fix the scope for > device tree bindings. > > Signed-off-by: Li Yang > Acked-by: Scott Wood Applied to powerpc next, thanks.

Re: [2/2] MAINTAINERS: update my email address from freescale to nxp

On Fri, 2017-06-02 at 23:20:23 UTC, Yang Li wrote: > Signed-off-by: Li Yang Applied to powerpc next, thanks. https://git.kernel.org/powerpc/c/64d09f5ecb237974bc824800e5e501 cheers

Re: selftests/powerpc: context_switch use private futexes with threads

On Sun, 2017-06-04 at 12:06:22 UTC, Nicholas Piggin wrote: > This reduces overhead of mutex locking and increases context switch > rate significantly (which helps to measure and profile the context > switch path). > > Signed-off-by: Nicholas Piggin Applied to powerpc next,

Re: [PATCH v3 3/9] powerpc/kprobes/optprobes: Move over to patch_instruction

On 2017/06/07 03:46PM, Balbir Singh wrote: > On Wed, 2017-06-07 at 00:42 +0530, Naveen N. Rao wrote: > > On 2017/06/06 02:29PM, Balbir Singh wrote: > > > With text moving to read-only migrate optprobes to using > > > the patch_instruction infrastructure. Without this optprobes > > > will fail and