Re: [PATCH V2 0/9] Add ppc64el support to src:linux
Ben, This still adds configuration for 'powerpc' and 'ppc64' flavours, which do not exist for ppc64el! Please send a new version of this patch. Sure. I will review and test it on the other ports. I guess I am confusing the symlinks for the #include paths in some form. I just sent a V3 for this one. As for the other patches plus the transactional-memory option you added, I confirm the SVN source pkg is working fine on KVM guests and PowerNV. Thanks, -- Mauricio Faria de Oliveira IBM Linux Technology Center -- To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: https://lists.debian.org/539b1369.9020...@linux.vnet.ibm.com
Re: [PATCH V2 0/9] Add ppc64el support to src:linux
On Thu, 2014-06-12 at 19:16 -0300, Mauricio Faria de Oliveira wrote: > Hi Ben, > > > I have not yet applied: > > > > [4] ppc64el: kernel: config: enable HTM as common powerpc64 option > > > > because it results in a module ABI change for existing powerpc64 > > configurations. I could make the config change just for ppc64el at the > > moment, if you like. We can apply it when we have a reason to bump the > > ABI for all architectures (whether that's to 3.14-2 or 3.15-1). > > Ok, that is fine. Would you please do the config change for ppc64el? [...] Will do. Ben. -- Ben Hutchings If the facts do not conform to your theory, they must be disposed of. signature.asc Description: This is a digitally signed message part
Re: [PATCH V2 0/9] Add ppc64el support to src:linux
Hi Ben, > I have not yet applied: [4] ppc64el: kernel: config: enable HTM as common powerpc64 option because it results in a module ABI change for existing powerpc64 configurations. I could make the config change just for ppc64el at the moment, if you like. We can apply it when we have a reason to bump the ABI for all architectures (whether that's to 3.14-2 or 3.15-1). Ok, that is fine. Would you please do the config change for ppc64el? I also have not applied: [7] ppc64el: installer: new files This still adds configuration for 'powerpc' and 'ppc64' flavours, which do not exist for ppc64el! Please send a new version of this patch. Sure. I will review and test it on the other ports. I guess I am confusing the symlinks for the #include paths in some form. Thank you very much. -- Mauricio Faria de Oliveira IBM Linux Technology Center -- To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: https://lists.debian.org/539a26b5.5080...@linux.vnet.ibm.com
Re: [PATCH V2 0/9] Add ppc64el support to src:linux
On Thu, 2014-06-12 at 10:46 -0300, Mauricio Faria de Oliveira wrote: > Hi Ben and Bastian, > > On 06/04/2014 09:06 AM, Mauricio Faria de Oliveira wrote: > > This patch-set adds support to the ppc64el port in the linux source package. > > It includes fixes and improvements to the previous 'PATCH' submission [0]. > > Would you have any further comments/corrections/recommendations > for this submission? > > I am wondering if it is good to go, for starters, or needs more changes. I've applied: [3] ppc64el: kernel: config: split common/big-endian powerpc64 options [2] ppc64el: kernel: config: little-endian powerpc64 options [1] ppc64el: kernel: defines file [5] ppc64el: kernel: patch: temporarily disable zImage [6] ppc64el: kernel: patch: one patch for the PowerNV platform [8] ppc64el: bug: bug/include-model I have not yet applied: [4] ppc64el: kernel: config: enable HTM as common powerpc64 option because it results in a module ABI change for existing powerpc64 configurations. I could make the config change just for ppc64el at the moment, if you like. We can apply it when we have a reason to bump the ABI for all architectures (whether that's to 3.14-2 or 3.15-1). I also have not applied: [7] ppc64el: installer: new files This still adds configuration for 'powerpc' and 'ppc64' flavours, which do not exist for ppc64el! Please send a new version of this patch. Ben. -- Ben Hutchings The program is absolutely right; therefore, the computer must be wrong. signature.asc Description: This is a digitally signed message part
Re: [PATCH V2 0/9] Add ppc64el support to src:linux
Hi Ben and Bastian, On 06/04/2014 09:06 AM, Mauricio Faria de Oliveira wrote: This patch-set adds support to the ppc64el port in the linux source package. It includes fixes and improvements to the previous 'PATCH' submission [0]. Would you have any further comments/corrections/recommendations for this submission? I am wondering if it is good to go, for starters, or needs more changes. Thank you, -- Mauricio Faria de Oliveira IBM Linux Technology Center -- To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: https://lists.debian.org/5399af1d.3040...@linux.vnet.ibm.com
Re: [PATCH V2 0/9] Add ppc64el support to src:linux
On 06/04/2014 09:31 AM, Ben Hutchings wrote: I only got 8 following patches, but it looks the 9th was an accidental commit containing copies of the preceding 8 patches: Oops. Yes, exactly. I removed that file, but missed updating the cover letter. Apologies. -- Mauricio Faria de Oliveira IBM Linux Technology Center -- To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: https://lists.debian.org/538f1baa.3070...@linux.vnet.ibm.com
Re: [PATCH V2 0/9] Add ppc64el support to src:linux
On Wed, 2014-06-04 at 09:06 -0300, Mauricio Faria de Oliveira wrote: > This patch-set adds support to the ppc64el port in the linux source package. > It includes fixes and improvements to the previous 'PATCH' submission [0]. > > Thank you for the comments, discussions, and patience kindly had so far. :) > > As always, any feedback is welcome. [...] > Mauricio Faria de Oliveira (9): > ppc64el: kernel: defines file > ppc64el: kernel: config: little-endian powerpc64 options > ppc64el: kernel: config: split common/big-endian powerpc64 options > ppc64el: kernel: config: enable HTM as common powerpc64 option > ppc64el: kernel: patch: temporarily disable zImage > ppc64el: kernel: patch: one patch for the PowerNV platform > ppc64el: installer: new files > ppc64el: bug: bug/include-model > patches I only got 8 following patches, but it looks the 9th was an accidental commit containing copies of the preceding 8 patches: [...] > create mode 100644 submit-patches/-cover-letter.patch > create mode 100644 submit-patches/0001-ppc64el-kernel-defines-file.patch > create mode 100644 > submit-patches/0002-ppc64el-kernel-config-little-endian-powerpc64-option.patch > create mode 100644 > submit-patches/0003-ppc64el-kernel-config-split-common-big-endian-powerp.patch > create mode 100644 > submit-patches/0004-ppc64el-kernel-config-enable-HTM-as-common-powerpc64.patch > create mode 100644 > submit-patches/0005-ppc64el-kernel-patch-temporarily-disable-zImage.patch > create mode 100644 > submit-patches/0006-ppc64el-kernel-patch-one-patch-for-the-PowerNV-platf.patch > create mode 100644 submit-patches/0007-ppc64el-installer-new-files.patch > create mode 100644 submit-patches/0008-ppc64el-bug-bug-include-model.patch Ben. -- Ben Hutchings Experience is what causes a person to make new mistakes instead of old ones. signature.asc Description: This is a digitally signed message part
[PATCH V2 0/9] Add ppc64el support to src:linux
This patch-set adds support to the ppc64el port in the linux source package. It includes fixes and improvements to the previous 'PATCH' submission [0]. Thank you for the comments, discussions, and patience kindly had so far. :) As always, any feedback is welcome. Changes from PATCH (v1) [0] 0) Patches on top of linux 3.14.4-1 (unstable) rather than 3.15-rcN (experimental) As per maks message on debian-kernel mailing list, mentioning 3.14 long term support. This only affected 2 patches in this patchset, minorly (changes 2 and 3). 1) Patch: "ppc64el: kernel: config: little-endian powerpc64 options" Keep as common options (config-arch-64): CRASH_DUMP - disabled until further testing. PPC_TRANSACTIONAL_MEM - this goes into a new, 'add' patch, rather than the 'split configs' patch. Remove options enabled during the build process: PPC_BOOK3S_64 POWER7_CPU VSX (selected by PPC_TRANSACTIONAL_MEM) PPC_POWERNV Remove non-required options, currently: LPARCFG PPC_SPLPAR (already in config-arch-64) PPC_SMLPAR DTL Remove superfluous built-in overrides (a.k.a. run qemu w/out initrd): IBMVETH BLK_DEV_SD SCSI_IBMVSCSI EXT4_FS Remove override to HZ=100 (from HZ=250 in main config) There were no strong opinions against HZ=250, internally. 2) Patch: "ppc64el: kernel: config: split common/big-endian powerpc64 options" Keep as common options: CRASH_DUMP (see note 1) Slight change to accomodate the new 'powerpc/config.powerpc64' file for powerpc64, included in the debian/config/powerpc/defines. 3) Patch: "ppc64el: kernel: patch: one patch for the PowerNV platform" Using the original patch for linux 3.14 (rather than a forward-port). Added the 'Origin' header to the added patch file. Changes from RFC PATCH [1] (details removed, see [0]): 1) The big & little endian powerpc64 config files were split. 2) The powerpc flavour and installer subdirs/symlinks were removed. 3) Unneeded module lists were removed from installer/. 4) The bootwrapper_install target is no longer enabled for ppc64el. 5) Added a patch for booting in the PowerNV platform. Links: [0] https://lists.debian.org/debian-kernel/2014/05/msg00323.html [1] https://lists.debian.org/debian-kernel/2014/05/msg00155.html Mauricio Faria de Oliveira (9): ppc64el: kernel: defines file ppc64el: kernel: config: little-endian powerpc64 options ppc64el: kernel: config: split common/big-endian powerpc64 options ppc64el: kernel: config: enable HTM as common powerpc64 option ppc64el: kernel: patch: temporarily disable zImage ppc64el: kernel: patch: one patch for the PowerNV platform ppc64el: installer: new files ppc64el: bug: bug/include-model patches debian/config/kernelarch-powerpc/config-arch-64| 85 + debian/config/kernelarch-powerpc/config-arch-64-be | 96 + debian/config/kernelarch-powerpc/config-arch-64-le | 34 ++ debian/config/powerpc/defines |2 +- debian/config/ppc64/defines|2 +- debian/config/ppc64el/defines | 18 +- debian/installer/ppc64el/kernel-versions |2 + debian/installer/ppc64el/modules/powerpc |1 + debian/installer/ppc64el/modules/ppc64 |1 + .../installer/ppc64el/modules/ppc64el/ata-modules |1 + .../ppc64el/modules/ppc64el/btrfs-modules |1 + .../ppc64el/modules/ppc64el/cdrom-core-modules |1 + .../installer/ppc64el/modules/ppc64el/core-modules |1 + .../installer/ppc64el/modules/ppc64el/crc-modules |1 + .../ppc64el/modules/ppc64el/crypto-dm-modules |1 + .../ppc64el/modules/ppc64el/crypto-modules |1 + .../ppc64el/modules/ppc64el/event-modules |1 + .../installer/ppc64el/modules/ppc64el/ext4-modules |1 + .../ppc64el/modules/ppc64el/fancontrol-modules |1 + .../installer/ppc64el/modules/ppc64el/fat-modules |1 + .../ppc64el/modules/ppc64el/firewire-core-modules |1 + .../installer/ppc64el/modules/ppc64el/fuse-modules |1 + .../ppc64el/modules/ppc64el/hypervisor-modules |1 + .../ppc64el/modules/ppc64el/input-modules |1 + .../ppc64el/modules/ppc64el/isofs-modules |1 + .../installer/ppc64el/modules/ppc64el/jfs-modules |1 + .../installer/ppc64el/modules/ppc64el/kernel-image |1 + .../installer/ppc64el/modules/ppc64el/loop-modules |1 + .../installer/ppc64el/modules/ppc64el/md-modules |1 + .../ppc64el/modules/ppc64el/mouse-modules |1 + .../ppc64el/modu